-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
mep12 on scatter_profile.py #4734
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mep12 on scatter_profile.py #4734
Conversation
y = 0.9*pylab.rand(N) | ||
s = 20*pylab.rand(N) | ||
pylab.scatter(x, y, s) | ||
x = 0.9*np.rand(N) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be np.random.rand
y = 0.9*pylab.rand(N) | ||
s = 20*pylab.rand(N) | ||
pylab.scatter(x, y, s) | ||
x = 0.9*np.random.random(N) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry it has to be np.random.rand
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack! I will get that changed as soon as I get back to my computer!
Cheers,Eric
On Sun, Jul 19, 2015 at 1:53 PM, Jens Hedegaard Nielsen
notifications@github.com wrote:
for N in (20, 100, 1000, 10000, 50000):
tstart = time.time()
- x = 0.9*pylab.rand(N)
- y = 0.9*pylab.rand(N)
- s = 20*pylab.rand(N)
- pylab.scatter(x, y, s)
- x = 0.9*np.random.random(N)
Sorry it has to be np.random.rand
Reply to this email directly or view it on GitHub:
https://github.com/matplotlib/matplotlib/pull/4734/files#r34960322
mep12 on scatter_profile.py
Just did it! 😄 |
No description provided.