MAINT: Updates to formatters in matplotlib.ticker
#6253
Conversation
madphysicist
force-pushed
the
ticker-formatter-maint
branch
from
dc83624 to
c77fe94
Compare
lib/matplotlib/ticker.py
Outdated
| """ | ||
| Return a short string version of the tick value. | ||
| Defaults to the position-indepedent long value. |
madphysicist
force-pushed
the
ticker-formatter-maint
branch
2 times, most recently
from
c07d694 to
7602b04
Compare
| Switch minor tick labeling on or off. | ||
| ``labelOnlyBase=True`` to turn off minor ticks. | ||
| """ |
There was a problem hiding this comment.
Does this currently work? Should it have a not in there?
There was a problem hiding this comment.
This particular method I have no idea. The formatter itself seems to work OK though. I will check right now.
There was a problem hiding this comment.
This method does nothing for this particular class since self.labelOnlyBase is not used in self.pprint_val. However, it does work for the LogFormatterExponent class, which extends this one. The old docs and the method name are indeed very misleading. labelOnlyBase is named correctly (as I have noted in the updated docs). If it is True, minor ticks will not be shown, which is exactly the opposite of what the name of the method implies. I did not find any explicit calls to this method in matplotlib. Am I free to modify the parameter name to say labelMinor and set self.labelOnlyBase = not labelMinor?
There was a problem hiding this comment.
Yes, but can you do that in a separate PR? The doc updates are non-controversial and should go in, but the API change should get more attention.
There was a problem hiding this comment.
Makes sense, will do.
There was a problem hiding this comment.
I have opened up #6264 with the code change.
tacaswell
modified the milestones:
2.1 (next point release),
2.0 (style change major release)
|
This looks good, but needs a rebase 😞 \ |
This should not break backwards compatibility since extra keywords are allowed
Moved to the top and added missing formatter classes. Probably missing some locators too, but that's for the future.
madphysicist
force-pushed
the
ticker-formatter-maint
branch
from
7602b04 to
29b2ea6
Compare
|
Fixed. I've been spoiled by Homu on numpy. |
MAINT: Updates to formatters in `matplotlib.ticker` Conflicts: .mailmap - just added the 1 new line, did not change any others lib/matplotlib/tests/test_ticker.py - whitespace vs new tests
|
backported to v2.x as 668f292 |
4 participants
This PR is a split from #6251. Most of the changes are to docs, but there are two minor code changes:
__all__has been moved to the top of the file and a couple of missing formatters were added to it.As the title suggests, only formatters were affected. Locators were left untouched.
I have retained the original commits until the PR is reviewed to make sure that I did the split correctly.