-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Feature Request: Allow setting default AutoMinorLocator #14233
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
Comments
I would like to work on this issue. please guide me @dstansby @timhoffm what need to be done since it is my first contribution. From above i understood that rcParams is a global object which can be used across the module. In this we need to add the method to set a default value for ticker module and in the ticker module's AutoMinorLocator method we need to check if the default exist in the rcParams and use it. Please let me know if i understood it right or not and suggest if any corrections. |
Welcome to matplotlib! General information how to contribute and setup your local development environment can be found at https://matplotlib.org/devel/contributing.html. For this specific feature:
If you have questions, you can always ask here, or if it's easier also start with an incomplete PR and ask there in the context of your proposed code. |
I would like to work on this issue if it is available. I was thinking the best way to tackle it would be to add an rcParam for the tick count, or use one of the existing one (i don't think any of them tell us this though). Is it okay to add rcParams, or should I try a different approach? |
Hi everyone, is this still a relevant issue? I think I gained a common sense from the provided comments and reading the code for |
@IlievskiV There is an open PR for this at #16762. Perhaps ping the author there if you are interested in taking this up? |
@jklymak sure, I just did it. Thanks for the suggestion! |
Fixes #14233 Co-authored-by: Vladimir <vladimir@igenius.net> Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Fixes matplotlib#14233 Co-authored-by: Vladimir <vladimir@igenius.net> Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Fixes matplotlib#14233 Co-authored-by: Vladimir <vladimir@igenius.net> Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Via
rcParams
you can turn on minor ticks/grid for all your plots but there is currently no way to give a default interval for minor ticks to be shown.AutoMinorLocator defaults to 4 or 5 so potentially it could check to see if this value is set in rcParams and use that first instead?
The text was updated successfully, but these errors were encountered: