Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

REV: remove default formatter on log minor ticks#7000

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

Conversation

tacaswell
Copy link
Member

Reverts part of#5161

original commit merged to master as947e6eb
the merge was backported to v2.x asfb45c4a

Reverts part ofmatplotlib#5161original commit merged to master as947e6ebthe merge was backported to v2.x asfb45c4a
@tacaswelltacaswell added this to the2.0 (style change major release) milestoneAug 29, 2016
@tacaswell
Copy link
MemberAuthor

This was causing these to be in the docs:

image

image

The issues are the overstriking, the wiggly baseline, and the difficulty in telling which tick the intermediate labels apply to.

The smarter formatter is still there and can be used, but is disabled by default.

@WeatherGod
Copy link
Member

This is certainly an improvement, and I am willing to merge as-is, but it feels like we are papering over some deeper issues. Do we have issues filed detailing the wiggly baseline?

Also, I feel that there is too much blank space between the tick labels now, but at least they aren't overlapping anymore, right?

@tacaswell
Copy link
MemberAuthor

Ticks labels are placed at a fixed pad away from the end of the tick mark and thenva='top' which explains the wiggles in the baseline.

#6549 and#7001 track the opposite case where we have too many log ticks.

@WeatherGodWeatherGod merged commite3be57e intomatplotlib:v2.xAug 30, 2016
@tacaswelltacaswell deleted the api_revert_log_minor_tick_labels branchAugust 30, 2016 21:11
efiring added a commit to efiring/matplotlib that referenced this pull requestNov 7, 2016
This partly restores the functionality that was added inPRmatplotlib#5161 and partly removed inmatplotlib#7000.  The "partly" is becausenow the labeling of minor log ticks is turned on only whennumdecs (the axis range in powers of the log base) is lessthan or equal to one, rather than 3.This also fixes a bug that was causing double labeling with abase of 2; minor ticks were coinciding with major ticks, andboth were being labeled.
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.0.0
Development

Successfully merging this pull request may close these issues.

3 participants
@tacaswell@WeatherGod@mdboom

[8]ページ先頭

©2009-2025 Movatter.jp