-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
COMPAT: matplotlib 3.4.0 #40718
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
COMPAT: matplotlib 3.4.0 #40718
Conversation
jbrockmendel
commented
Apr 1, 2021
•
edited by jreback
Loading
edited by jreback
- closes BUG: MatplotlibDeprecationWarning: The is_first_col function was deprecated in Matplotlib 3.4 #40714
- tests added / passed
- Ensure all linting tests pass, see here for how to run them
- whatsnew entry
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.
lgtm
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.
Thanks for looking at those failures!
@@ -760,6 +762,10 @@ def test_plot_scatter_with_c(self): | |||
|
|||
# n.b. there appears to be no public method | |||
# to get the colorbar label | |||
if mpl_ge_3_4_0: | |||
mark = pytest.mark.xfail(reason="_label attribute 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.
For recent versions, you can use ax.collections[0].colorbar.ax.get_ylabel()
instead of the private attribute
marking this for backport as well. @jbrockmendel ping when ready . |
im kinda burned out on fixing-the-ci; can someone else pick up the baton on this? |
I could pick this up sometime later today |
ping @pandas-dev/pandas-core for merge. Thanks everyone! |
thanks all! |
@meeseeksdev backport 1.2.x |
Hi, I just ran into this issue when running my matplotlib project based on pandas-input. Thanks for working on it. Please note, that matplotlib is now at 3.4.1 (3.4.0 was quickly superseded). Hth, |
Co-authored-by: jbrockmendel <[email protected]>