-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
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
Conversation
4085cbd
to
badb300
Compare
Can you document the removals in api_changes? |
badb300
to
db35561
Compare
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 |
There was a problem hiding this comment.
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 '
There was a problem hiding this comment.
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 :-))
There was a problem hiding this comment.
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…
There was a problem hiding this comment.
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".
db35561
to
f234dad
Compare
fixed |
Removal of deprecated features | ||
------------------------------ | ||
|
||
The ``matplotlib.backends.qt_compat`` module has been removed. |
There was a problem hiding this comment.
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
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is... fixed.
f234dad
to
ad6e561
Compare
Has some overlap with #4128 which seems to need another rebase. |
Merge one of them first and then the author of the other can do the rebase? |
I (maybe biasedly) have a preference for mine as it links back to the issue/commit where things were deprecated. |
That's completely fine, can it be merged first then? |
I did. |
ad6e561
to
ce615e8
Compare
Rebased; merged the changelog into yours. |
@@ -1727,53 +1716,6 @@ def onRemove(self, ev): | |||
break | |||
h = parent | |||
|
|||
def onHilite(self, ev): |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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): |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
ce615e8
to
347ebef
Compare
Deprecated methods | ||
------------------ | ||
|
||
The ``set_graylevel`` and ``onHilite`` methods have been removed from the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
onHilit
is from FigureCanvasBase
There was a problem hiding this comment.
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.
347ebef
to
867d6f1
Compare
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 completelyequivalent to the standard Colorbar -- it only adds a private, unused
method.)
Also adjusted an example that used a deprecated module.