Thanks to visit codestin.com
Credit goes to github.com

Skip to content

BUG: ax.tick_params change in tick length does not adjust tick labels #7114

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

Closed
efiring opened this issue Sep 14, 2016 · 3 comments
Closed

BUG: ax.tick_params change in tick length does not adjust tick labels #7114

efiring opened this issue Sep 14, 2016 · 3 comments
Assignees
Labels
Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. status: confirmed bug
Milestone

Comments

@efiring
Copy link
Member

efiring commented Sep 14, 2016

The problem was found by @QuLogic in #7113.

import matplotlib.pyplot as plt
fig, ax = plt.subplots()
ax.tick_params(length=20)
plt.show()
@efiring efiring added status: confirmed bug Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. labels Sep 14, 2016
@efiring efiring added this to the 2.0 (style change major release) milestone Sep 14, 2016
@efiring efiring self-assigned this Sep 15, 2016
@WeatherGod
Copy link
Member

Not entirely convinced that it is a bug. With inward facing ticks, I
wouldn't want the tick labels to move just be cause I set a long tick line.
Tick padding is a separate parameter.

On Wed, Sep 14, 2016 at 6:59 PM, Eric Firing [email protected]
wrote:

The problem was found by @QuLogic https://github.com/QuLogic in #7113
#7113.

import matplotlib.pyplot as plt
fig, ax = plt.subplots()
ax.tick_params(length=20)
plt.show()


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#7114, or mute the thread
https://github.com/notifications/unsubscribe-auth/AARy-HYAXXfdXeRjmDuX8wvbHPpjObuQks5qqHxvgaJpZM4J9Vgu
.

@efiring
Copy link
Member Author

efiring commented Sep 15, 2016

@WeatherGod, it's a bug. The logic is in place to calculate the padding based on tick length and a _base_pad, taking into account whether the ticks are inward, outward, or centered. The calculation just wasn't being updated when the tick length was changed.

@ghost ghost assigned NelleV Sep 16, 2016
@ghost ghost added the In Progress label Sep 16, 2016
@NelleV
Copy link
Member

NelleV commented Sep 16, 2016

eh… Thanks @wafflebot[bot] !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. status: confirmed bug
Projects
None yet
Development

No branches or pull requests

4 participants