Skip to content
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

Fix/mpl37 compat #52150

Merged
merged 3 commits into from Mar 24, 2023
Merged

Fix/mpl37 compat #52150

merged 3 commits into from Mar 24, 2023

Conversation

tacaswell
Copy link
Contributor

This was reported via matplotlib/matplotlib#25538 , I can not find an issue if @ocefpaf reported one.

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • [n/a] Added type annotations to new arguments/methods/functions.
  • [n/a] Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

As of mpl 3.7 all Axes now report as being instances of Subplots, however
ax.get_subplotspec() may now return None if the Axes is not actually in
gridspec.
The MPLPlot instance already holds a reference to the Figure, use that
instead of walking up from the first Axes.
@ocefpaf
Copy link

ocefpaf commented Mar 24, 2023

I can not find an issue if @ocefpaf reported one.

I was going to do it today. You were too fast :-)

Thanks @tacaswell!

@mroeschke mroeschke added the Visualization plotting label Mar 24, 2023
@mroeschke mroeschke added this to the 2.1 milestone Mar 24, 2023
@mroeschke mroeschke merged commit 4a2ea81 into pandas-dev:main Mar 24, 2023
43 of 44 checks passed
@mroeschke
Copy link
Member

Thanks @tacaswell

BTW, we have a branch trying to support Matplotlib 3.7 but we don't really have experienced matplotlib contributors anymore. Any insights would be appreciated: #51990

@lithomas1
Copy link
Member

@meeseeksdev backport 2.0.x

@lithomas1 lithomas1 modified the milestones: 2.1, 2.0.3 Jun 25, 2023
meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Jun 25, 2023
lithomas1 pushed a commit that referenced this pull request Jun 26, 2023
Backport PR #52150: Fix/mpl37 compat

Co-authored-by: Thomas A Caswell <tcaswell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants