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

Test toolkits rebase #2135

Closed

Conversation

jenshnielsen
Copy link
Member

This is a rebase of #1422 on the 1.3.x branch.

Running:

python tests.py

Will run these tests in addition to the standard mpl ones.

To run the tests from within python do:

mpl_toolkits.test()

This only runs the toolkits tests.

Note that I had to change the baseline images. Some of these are simply due to changes in freetype but some also seems to involve changes to axis lims. Not sure why.

All tests are at this stage for axisgrid1 and borrowed from the mpl documentation.

@jenshnielsen
Copy link
Member Author

Do you want this in the 1.3.0 release or should I just rebase in on master?

@mdboom
Copy link
Member

mdboom commented Jun 24, 2013

Lets just do master for now -- master and 1.3.x are not so different that it would be too hard to cherry-pick this onto 1.3.x if we decide that later. Mainly, I'd like to get Travis running over this and do some of my own testing and kicking of the tires...

mdboom and others added 26 commits August 26, 2013 15:44
fix draw_idle reference in  NavigationToolbar2
Upload test result images to Amazon S3
changed colorbar outline from a Line2D object to a Polygon object
Conflicts:
	.travis.yml
	lib/matplotlib/backends/qt4_editor/formlayout.py
Better axis limits when using shared axes and empty subplots
…late

[bug correction] trirefine is now independant of triangulation numbering
make sure we only perform absolute imports on loading a backend
Stop relying on 2to3 and use `six.py` for compatibility instead
@tacaswell
Copy link
Member

@tbekolay Is this still a live PR? If so, it needs a re-base.

@tbekolay
Copy link
Contributor

This is @jenshnielsen's PR, I just followed the same directory structure. I can do the rebase, though, if @jenshnielsen is busy.

@tacaswell
Copy link
Member

@tbekolay Sorry! I apparently can't read.

@tbekolay
Copy link
Contributor

@tacaswell 😄 No worries!

@tacaswell
Copy link
Member

@jenshnielsen Do you have time to rebase in the near future?

@jenshnielsen
Copy link
Member Author

Yes. If someone is actually interested in merging it. The last time I did a rebase nothing happened after that.

@jenshnielsen
Copy link
Member Author

Closing this to reopen agains master.

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

Successfully merging this pull request may close these issues.

None yet