Skip to content

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

Merged
merged 3 commits into from
Jul 19, 2015
Merged

mep12 on scatter_profile.py #4734

merged 3 commits into from
Jul 19, 2015

Conversation

ericmjl
Copy link
Contributor

@ericmjl ericmjl commented Jul 17, 2015

No description provided.

@tacaswell tacaswell added this to the proposed next point release milestone Jul 17, 2015
@jenshnielsen jenshnielsen added the MEP: MEP12 gallery and examples improvements label Jul 17, 2015
y = 0.9*pylab.rand(N)
s = 20*pylab.rand(N)
pylab.scatter(x, y, s)
x = 0.9*np.rand(N)
Copy link
Member

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)
Copy link
Member

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

Copy link
Contributor Author

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

jenshnielsen added a commit that referenced this pull request Jul 19, 2015
@jenshnielsen jenshnielsen merged commit d2f91a1 into matplotlib:master Jul 19, 2015
@jenshnielsen
Copy link
Member

Thanks @ericmjl Do you have the time to fix #4646 and #4631 too?

@ericmjl
Copy link
Contributor Author

ericmjl commented Jul 19, 2015

Just did it! 😄

@ericmjl ericmjl deleted the mep12-scatter_profile.py branch July 19, 2015 22:56
@QuLogic QuLogic modified the milestones: v1.5.0, proposed next point release (2.1) Nov 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MEP: MEP12 gallery and examples improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants