Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
backend_qt4agg: remove redundant classes. Closes #1151.#2629
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
The It looks like it got copied over from the gtk backends when the QT ones were created. In gtk this function is used to set up the subplot tool. travis failures look to all be pep8 blank-line issues. |
backend_qt4agg: remove redundant classes.Closes#1151.
This probably should get an entry in api_changes. Do we want to provide an alias to support 3rd party users who might be using it? |
@tacaswell, good point that it should have an api_changes entry; but I hope we can avoid putting in aliases. |
never mind, I can't read. |
Sorry for the noise, I miss read. |
And I was looking at the wrong example. Wedo use it in an example,http://matplotlib.org/examples/user_interfaces/embedding_in_qt4_wtoolbar.html , at least the toolbar should be depreciated gracefully. |
@tacaswell Thanks for catching that; I was too hasty with the original PR. Both classes can be deprecated by subclassing, using warnings.warn with a mplDeprecation argument in theinit. It doesn't look like we have anything fancier than this for deprecating classes. |
Added NavigationToolbar2QTAgg back with a deprecation warningAdded entry to api_changes.rst
No description provided.