Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
ENH: make interval_multiples work for years#9841
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
d04a913
to1cd2ddc
Compare👍 in principle. Can you add a test (just setting it up and asking what the ticks are rather than an image test)? Might be worth a whats_new update, or just a callout in the some date-related documentation to make it discoverable. |
@tacaswell I updated the doc at the top (which was out of date, BTW), and also will add to#9827 where I discuss DateLocators. |
There is trailing whitespace. |
a05ac8b
to242134d
CompareThe restriction in the first place was trying to make upfor what was actually a bug in Matplotlib. It was fixed inmatplotlib/matplotlib#9841.
Uh oh!
There was an error while loading.Please reload this page.
PR Summary
If
dates.AutoTickLocator
was on the YEARLY frequency, it didn't respect theinterval_multiples kwarg. This changes that by using the YearlyLocator ifinterval_multiples=True
(Of course I still think dateticks should be overhauled, but thats a bigger project)
Fixes#9838
PR Checklist