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

reverse changes to imports in embedding_webagg #6

Closed
wants to merge 1 commit into from
Closed

reverse changes to imports in embedding_webagg #6

wants to merge 1 commit into from

Conversation

mikofski
Copy link

As describted in matplotlib#3204 and matplotlib#3205, changes to embedding_webagg.py were not meant to be in this pull request.

I tested this on Windows-7 (x64) and Ubuntu-14.14 in a virtualenv with e9c7dcb installed by calling:

$ python ~/matplotlib/examples/user_interfaces/embedding_webagg.py

Prior to the patch running embedding_webagg.py raised exceptions, because FigureManagerWebAgg.get_javascript() calls FigureCanvasWebAggCore.get_default_filetype() and FigureCanvasWebAggCore.get_supported_filetypes_grouped() as if they were class methods.

@mikofski
Copy link
Author

I was wrong, this change has already been made in master, backend_bases.FigureCanvasBase.get_supported_filetypes_grouped() and backend_bases.FigureCanvasBase.get_default_filetype() have already been changed to class methods.

@mikofski mikofski closed this Jul 10, 2014
tacaswell pushed a commit that referenced this pull request Sep 12, 2015
tacaswell pushed a commit that referenced this pull request Mar 7, 2017
ENH: make texFontMap a property
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant