Skip to content

Adding figratio to matplotlib to allow dynamic change of figure size based upon width/height and aspect - #29262

Closed
arkapudota2003 wants to merge 1 commit into
matplotlib:mainfrom
arkapudota2003:main
Closed

arkapudota2003 wants to merge 1 commit into
matplotlib:mainfrom
arkapudota2003:main

Conversation

@arkapudota2003

Copy link
Copy Markdown

PR summary

PR checklist

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@timhoffm

timhoffm commented Dec 9, 2024

Copy link
Copy Markdown
Member

Thanks for your willingness to contribute. I assume this should address #28758? - Please always link the related issues, as stated in the PR checklist.

The PR does not refect the state of the discussion in #28758. All we agreed upon so far was to accept None as component in the tuple given to the figsize keyword argument, with the meaning: Take that component from rcParams["figure.figsize"]; i.e. plt.figure(figsize=(None, 3) and plt.figure(figsize=(3, None)

@tacaswell tacaswell mentioned this pull request Dec 10, 2024
@timhoffm

Copy link
Copy Markdown
Member

@arkapudota2003 do you plan to continue working on this?

@timhoffm

Copy link
Copy Markdown
Member

Closing for lack of response.

@timhoffm timhoffm closed this Aug 21, 2025
@melissawm melissawm moved this from Waiting for author to Done in First Time Contributors Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants