I added some minor typo fixes, and additional axes labels that I think are useful.
phobson#1

Can you please rebase and remove the extra merge commit?

👍 Looks good to me!

Hum… I was so happy to see @ivanov starting to review again that I merged before travis had time to finish. We'll see…

Thanks @phobson for the patch!

@NelleV thanks for the merge. Looks like Travis as happy & AppVeyor canceled itself after the merge (phew)

I'll take care of it as soon I can get back on GH. See my comment to Paul
above :/

On Fri, Oct 21, 2016 at 11:55 AM, Elliott Sales de Andrade <
[email protected]> wrote:

Can you please rebase and remove the extra merge commit?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#7324 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHCo2KmH79KhdYOAnQx94I3e8cnTolUks5q2QqSgaJpZM4KdbDj
.

Thanks Paul. I'm getting notification emails, but I can access github.com
or push ATM because of the Dyn DDoS
http://gizmodo.com/this-is-probably-why-half-the-internet-shut-down-today-1788062835
:/

On Fri, Oct 21, 2016 at 10:58 AM, Paul Ivanov [email protected]
wrote:

I added some minor typo fixes, and additional axes labels that I think are
useful.
phobson#1 phobson#1

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#7324 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHCoxjKBza_DNXR6GpyyV-gy_t-AxTDks5q2P05gaJpZM4KdbDj
.

I meant to. Thanks for catching that.

On Fri, Oct 21, 2016 at 10:18 AM, Paul Ivanov [email protected]
wrote:

@ivanov commented on this pull request.

In examples/statistics/boxplot_color_demo.py
#7324 (review)
:

import matplotlib.pyplot as plt
import numpy as np

Random test data

np.random.seed(123)
-all_data = [np.random.normal(0, std, 100) for std in range(1, 4)]
+all_data = [np.random.normal(0, std, size=(100, 4)) for std in range(1, 4)]

Any reason to not keep the size as just 100?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#7324 (review),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHCozgOUZMkOyQ0kzwjZEPua8Z_l1yuks5q2PPIgaJpZM4KdbDj
.

Err, those got through a bit late...