-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add feature: adapt_limit argument #26381
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
Conversation
Thank you for your work on this, however I am not sure that this is the right approach. We have a system for managing the margins / autolimiting behivor (see https://matplotlib.org/stable/gallery/subplots_axes_and_figures/axes_margins.html#sphx-glr-gallery-subplots-axes-and-figures-axes-margins-py). The best option would be integrate hist2d with this system (maybe just dropping the |
@niranjank2022 Reading this a bit more carefully and understanding the original bug, I do not think that this is a feature we want to take. Thank you for your effort and I hope we hear from you again! |
@niranjank2022 Thanks very much for opening this PR! I have had a lengthy debate with @tacaswell whether to include the feature and he is very difficult to persuade :-) I can see you are a young student who wants to contribute to open source, so I thought I would give a few (hopefully) helpful comments as a thanks for your help.
Hopefully the core devs will help you finish this PR, and you will be encouraged to contribute more code in the future! |
Thanks @Hvass-Labs for your valuable comments. It means a lot to me. I will definitely keep these in mind and continue to find ways to contribute! :) |
PR summary
PR checklist
This PR is to address issue #26288.