Remove deprecated stuff schedule for removal.#7290
Conversation
anntzer
force-pushed
the
remove-deprecated-stuff
branch
from
4085cbd to
badb300
Compare
|
Can you document the removals in api_changes? |
anntzer
force-pushed
the
remove-deprecated-stuff
branch
from
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.
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.
wow… OK.
Yeah, I think that's proper english, but you know… french people and english…
There was a problem hiding this comment.
There is, however, a missing double-quote on "normal".
NelleV
changed the title
anntzer
force-pushed
the
remove-deprecated-stuff
branch
from
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.
Isn't it qt4_compat that was removed, not qt_compat?
anntzer
force-pushed
the
remove-deprecated-stuff
branch
from
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. |
anntzer
force-pushed
the
remove-deprecated-stuff
branch
from
ad6e561 to
ce615e8
Compare
|
Rebased; merged the changelog into yours. |
| break | ||
| h = parent | ||
|
|
||
| def onHilite(self, ev): |
There was a problem hiding this comment.
This is missing a removal note.
|
|
||
|
|
||
| class Colorbar(mcolorbar.Colorbar): | ||
| def _config_axes_deprecated(self, X, Y): |
There was a problem hiding this comment.
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
force-pushed
the
remove-deprecated-stuff
branch
from
ce615e8 to
347ebef
Compare
doc/api/api_changes/code_removal.rst
Outdated
| Deprecated methods | ||
| ------------------ | ||
|
|
||
| The ``set_graylevel`` and ``onHilite`` methods have been removed from the |
There was a problem hiding this comment.
onHilit is from FigureCanvasBase
There was a problem hiding this comment.
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
force-pushed
the
remove-deprecated-stuff
branch
from
347ebef to
867d6f1
Compare
QuLogic
changed the title
5 participants
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.