Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove deprecated stuff schedule for removal. #7290

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 1 commit into from
Oct 21, 2016

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Oct 17, 2016

Everything removed here had either a deprecation warning indicating
removal scheduled in 1.6, or deprecation since at least two minor
releases.

(The exception, axes_grid1.axes_grid.Colorbar, is completely
equivalent to the standard Colorbar -- it only adds a private, unused
method.)

Also adjusted an example that used a deprecated module.

@anntzer anntzer force-pushed the remove-deprecated-stuff branch from 4085cbd to badb300 Compare October 17, 2016 04:31
@tacaswell tacaswell added this to the 2.1 (next point release) milestone Oct 18, 2016
@tacaswell
Copy link
Member

Can you document the removals in api_changes?

@anntzer anntzer force-pushed the remove-deprecated-stuff branch from badb300 to db35561 Compare October 18, 2016 03:35
@anntzer
Copy link
Contributor Author

anntzer commented Oct 18, 2016

done

class and its subclasses. The ``toggle_axisline`` method has been removed from
the ``mpl_toolkits.axes_grid1.mpl_axes.Axes`` class.

Support for setting an ``Axes``' aspect to ``normal"`` has been removed, in
Copy link
Member

Choose a reason for hiding this comment

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

There is an extra tick '

Copy link
Contributor Author

@anntzer anntzer Oct 18, 2016

Choose a reason for hiding this comment

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

No, it's supposed to render as

an Axes' aspect

(well I hope that's correct in English :-))

Copy link
Member

Choose a reason for hiding this comment

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

wow… OK.
Yeah, I think that's proper english, but you know… french people and english…

Copy link
Member

Choose a reason for hiding this comment

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

There is, however, a missing double-quote on "normal".

@NelleV NelleV changed the title Remove deprecated stuff schedule for removal. [MRG+1] Remove deprecated stuff schedule for removal. Oct 18, 2016
@anntzer anntzer force-pushed the remove-deprecated-stuff branch from db35561 to f234dad Compare October 18, 2016 04:41
@anntzer
Copy link
Contributor Author

anntzer commented Oct 18, 2016

fixed

Removal of deprecated features
------------------------------

The ``matplotlib.backends.qt_compat`` module has been removed.
Copy link
Member

Choose a reason for hiding this comment

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

Isn't it qt4_compat that was removed, not qt_compat?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it is... fixed.

@anntzer anntzer force-pushed the remove-deprecated-stuff branch from f234dad to ad6e561 Compare October 18, 2016 20:48
@tacaswell
Copy link
Member

Has some overlap with #4128 which seems to need another rebase.

@anntzer
Copy link
Contributor Author

anntzer commented Oct 19, 2016

Merge one of them first and then the author of the other can do the rebase?

@tacaswell
Copy link
Member

I (maybe biasedly) have a preference for mine as it links back to the issue/commit where things were deprecated.

@anntzer
Copy link
Contributor Author

anntzer commented Oct 19, 2016

That's completely fine, can it be merged first then?

@tacaswell
Copy link
Member

@anntzer Can push the green button on #4128 ?

@NelleV
Copy link
Member

NelleV commented Oct 19, 2016

I did.

@anntzer anntzer force-pushed the remove-deprecated-stuff branch from ad6e561 to ce615e8 Compare October 19, 2016 23:08
@anntzer
Copy link
Contributor Author

anntzer commented Oct 19, 2016

Rebased; merged the changelog into yours.

@@ -1727,53 +1716,6 @@ def onRemove(self, ev):
break
h = parent

def onHilite(self, ev):
Copy link
Member

Choose a reason for hiding this comment

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

This is missing a removal note.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@@ -37,43 +34,6 @@ def _tick_only(ax, bottom_on, left_on):
ax.axis["left"].toggle(ticklabels=left_off, label=left_off)


class Colorbar(mcolorbar.Colorbar):
def _config_axes_deprecated(self, X, Y):
Copy link
Member

Choose a reason for hiding this comment

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

What is going on here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a subclass of mpl_toolkits.axes_grid1.colorbar.Colorbar (not matplotlib.Colorbar) that adds a single private method that is not even used anywhere else.

@anntzer anntzer force-pushed the remove-deprecated-stuff branch from ce615e8 to 347ebef Compare October 20, 2016 16:42
Deprecated methods
------------------

The ``set_graylevel`` and ``onHilite`` methods have been removed from the
Copy link
Member

Choose a reason for hiding this comment

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

onHilit is from FigureCanvasBase

Copy link
Contributor Author

@anntzer anntzer Oct 20, 2016

Choose a reason for hiding this comment

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

Sorry, being sloppy today... fixed.

Everything removed here had either a deprecation warning indicating
removal scheduled in 1.6, or deprecation since at least two minor
releases.

(The exception, `axes_grid1.axes_grid.Colorbar`, is completely
equivalent to the standard Colorbar -- it only adds a private, unused
method.)

Also adjusted an example that used a deprecated module.
@anntzer anntzer force-pushed the remove-deprecated-stuff branch from 347ebef to 867d6f1 Compare October 20, 2016 21:13
@QuLogic QuLogic changed the title [MRG+1] Remove deprecated stuff schedule for removal. Remove deprecated stuff schedule for removal. Oct 21, 2016
@QuLogic QuLogic merged commit 5ca283d into matplotlib:master Oct 21, 2016
@anntzer anntzer deleted the remove-deprecated-stuff branch October 21, 2016 23:56
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.

5 participants