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

get_subplot_params behavior doesn't match docstring #13109

Closed
anntzer opened this issue Jan 5, 2019 · 1 comment · Fixed by #25992
Closed

get_subplot_params behavior doesn't match docstring #13109

anntzer opened this issue Jan 5, 2019 · 1 comment · Fixed by #25992
Milestone

Comments

@anntzer
Copy link
Contributor

anntzer commented Jan 5, 2019

Extracted from #13105:

In gridspec.py, for the GridSpec class:

    def get_subplot_params(self, figure=None, fig=None):
        """
        Return a dictionary of subplot layout parameters. The default
        parameters are from rcParams unless a figure attribute is set.
        """

but the implementation clearly always drops the figure attributes; IOW it always returns a copy of self completely ignoring figure.

Conversely, GridSpecFromSubplotSpec.get_subplot_params does check for None attributes (but isn't documented to do so).

What a mess, as usual.

@github-actions
Copy link

This issue has been marked "inactive" because it has been 365 days since the last comment. If this issue is still present in recent Matplotlib releases, or the feature request is still wanted, please leave a comment and this label will be removed. If there are no updates in another 30 days, this issue will be automatically closed, but you are free to re-open or create a new issue if needed. We value issue reports, and this procedure is meant to help us resurface and prioritize issues that have not been addressed yet, not make them disappear. Thanks for your help!

@github-actions github-actions bot added the status: inactive Marked by the “Stale” Github Action label May 28, 2023
@QuLogic QuLogic added this to the v3.7.2 milestone May 29, 2023
@QuLogic QuLogic removed the status: inactive Marked by the “Stale” Github Action label May 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants