Repository navigation
[Doc]: Axes.inset_axes is still experimental #27648
Description
Documentation Link
Problem
I just noticed that Axes.inset_axes is still marked as experimental. It seems pretty well established to me, even recommended as a method for placing colorbars.
Suggested improvement
Is it time to remove the "experimental" warning?
Activity
Snehaaa18 commentedon Jan 14, 2024
Please assign this issue to me
rcomer commentedon Jan 14, 2024
Hello @Snehaaa18, thank you for your enthusiasm, but we do not assign issues. Also I would like to wait for other maintainers to comment whether it should be done before anyone makes the change.
timhoffm commentedon Jan 14, 2024
I vote for dropping the experimental flag. The API is consistent and simple enough. And it's been around long enough that we can expect reasonable adoption so that any incompatible changes should better run through a deprecation cycle, whether we've still formally marked it as experimental or not.
jklymak commentedon Jan 14, 2024
There was talk of adding the ordinal locations to inset_axes. However I'm 99% certain that can be done in a backwards compatible way should someone want to do that.
timhoffm commentedon Jan 14, 2024
Yes it could. But I tend to not do it. You either don't have control over the size and margins of the inset, or the API get's really complex. I think keeping it to the an analogous API as
Figure.add_axes()is good enough.github-actions commentedon Jan 17, 2024
Good first issue - notes for new contributors
This issue is suited to new contributors because it does not require understanding of the Matplotlib internals. To get started, please see our contributing guide.
We do not assign issues. Check the Development section in the sidebar for linked pull requests (PRs). If there are none, feel free to start working on it. If there is an open PR, please collaborate on the work by reviewing it rather than duplicating it in a competing PR.
If something is unclear, please reach out on any of our communication channels.
rcomer commentedon Jan 17, 2024
I think the work here is:
subplot_mosaicat DOC: mark mosaic as no longer provisional #24891).ShivamPathak99 commentedon Jan 18, 2024
Working on it.
story645 commentedon Jan 18, 2024
@ShivamPathak99 you accidentally opened your pull request against @tacaswell 's fork of Matplotlib, so we can't merge it into main. Can you instead open your pull request against https://github.com/matplotlib/matplotlib?
https://matplotlib.org/devdocs/devel/contribute.html#open-a-pull-request-on-matplotlib
My guess is you may have forked Tom's fork instead of forking against matplotlib/matplotlib