8000 LogFormatter bugfix, docs, support classic by efiring · Pull Request #7594 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

LogFormatter bugfix, docs, support classic #7594

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

Merged
merged 1 commit into from
Dec 9, 2016

Conversation

efiring
Copy link
Member
@efiring efiring commented Dec 8, 2016

Closes #7590.
Classic style, with no minor tick labeling on a log axis,
is supported via rcParams['_internal.classic'].
LogFormatter changes are documented in api_changes and
dflt_style_changes.
An implicit boolean is made explicit in scale.py.
A LogFormatter attribute that was sometimes, but not always,
set by set_locs, but used only in call, is replaced by
a local variable calculated in call. (set_locs doesn't
have to be called prior to call, so this would have been
a bug even if the attribute were always calculated by set_locs.)

Closes matplotlib#7590.
Classic style, with no minor tick labeling on a log axis,
is supported via rcParams['_internal.classic']
LogFormatter changes are documented in api_changes and
dflt_style_changes.
An implicit boolean is made explicit in scale.py.
A LogFormatter attribute that was sometimes, but not always,
set by set_locs, but used only in __call__, is replaced by
a local variable calculated in __call__.  (set_locs doesn't
have to be called prior to __call__, so this would have been
a bug even if the attribute were always calculated by set_locs.)
@efiring efiring changed the title LogFormatter bugfix, docs, support classic [MRG] LogFormatter bugfix, docs, support classic Dec 9, 2016
@efiring efiring added this to the 2.0 (style change major release) milestone Dec 9, 2016
@QuLogic QuLogic changed the title [MRG] LogFormatter bugfix, docs, support classic [MRG+1] LogFormatter bugfix, docs, support classic Dec 9, 2016
Copy link
Contributor
@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dopplershift dopplershift merged commit 1b58fcd into matplotlib:v2.x Dec 9, 2016
@dopplershift dopplershift changed the title [MRG+1] LogFormatter bugfix, docs, support classic LogFormatter bugfix, docs, support classic Dec 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0