-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Log Colorbar minorticks_off reverted if ticks set... #13339
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
Actually do not think connected to that PR, not sure why I thought it was. The cause is straight forward when looking at the code ( |
Can you give it a try and run the tests, and submit a PR if that is a reasonable solution? Seems you found something that works, so feel free to give it a try... OTOH, did you test if the same thing happens for log-axes on a normal axes instead of a colorbar? The point of the colorbar changes recently were to make the colorbar behave more like a normal axis, so, anything we can do to further that end is welcome. |
If the following is sufficient to test normal axes, then the same issue does not appear for them: t = np.arange(0.01, 20.0, 0.01)
ax = plt.subplot(111)
plt.loglog(t, 20*np.exp(-t/10.0), basex=2)
ax.minorticks_off()
ax.set_yticks([3, 4, 6, 10, 20])
plt.show() I am actually not sure how to submit a PR when the per-requisite PR has not been merged. |
Right - may as well wait for #13234 to go in, though you can make a PR against that branch and then rebas when its merged, its a bit fragile to do that. |
Posted by @LevN0
Apologies if not the right place. This PR has the following side effect:
Prior to this PR, minor ticks would remain off after
set_ticks
when they had been turned off. After this PR, they come back on.Originally posted by @LevN0 in #12909 (comment)
The text was updated successfully, but these errors were encountered: