Skip to content
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

Massive MEP move #4293

Merged
merged 23 commits into from Mar 30, 2015
Merged

Massive MEP move #4293

merged 23 commits into from Mar 30, 2015

Conversation

tacaswell
Copy link
Member

No description provided.

@tacaswell tacaswell added this to the next point release milestone Mar 29, 2015
interactivity manager has a consistent interface for the
interactivity manager to deal with. The proposed methods to add to
the artists, if they are to support interactivity, are:
* get_pixel_position_ll(self): get the pixel position of the lower
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like Travis is complaining about the indentation here.

@tacaswell
Copy link
Member Author

This is incredibly frustrating to test locally as building the docs seems to take way longer on my machine than travis (which probably means I need a new desktop...).

efiring added a commit that referenced this pull request Mar 30, 2015
@efiring efiring merged commit bf6e86d into matplotlib:master Mar 30, 2015
@tacaswell tacaswell deleted the massive_mep_move branch March 31, 2015 03:34
@fariza
Copy link
Member

fariza commented Apr 1, 2015

@tacaswell do you have the URL to see the MEP documents that are merged?

@tacaswell
Copy link
Member Author

@fariza http://matplotlib.org/devdocs/devel/MEP/index.html

Those get updated everytime we have a passing build of the master branch on travis.

@fariza
Copy link
Member

fariza commented Apr 2, 2015

We should change the wiki page, pointing to that new page, and a little explanation on how this is going to work from now on.

@tacaswell
Copy link
Member Author

Yes, that is (too far down) on my internal to-do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants