[Bug]: cm fontset in log scale does not use Unicode minus #21540

Activity

1. QuLogic commented on Nov 4, 2021

   

   QuLogic

   on Nov 4, 2021

   Member

   More actions

   This looks fine to me:
   
   Maybe it's your theme?
2. dstansby commented on Nov 6, 2021

   

   dstansby

   on Nov 6, 2021

   Member

   More actions

   Yes, works fine for me on v3.4.3 and the following code:

   from matplotlib.pyplot import *
   
   plot([-10, 10], [-10, 10])
   yscale('symlog', linthresh=.1)
   show()
   ``
   
   I'm going to close as not an issue as two of us have checked - @Atcold feel free to comment/re-open if you think we've missed something.
3. dstansby

   closed this as completedon Nov 6, 2021
4. Atcold commented on Nov 19, 2021

   

   Atcold

   on Nov 19, 2021

   Last edited by Atcold

   Author

   More actions

   Adding style.use(['bmh']) will show the bug.
   So the theme is breaking it.
   Not sure how to reopen this issue.

   @QuLogic that didn't really look too fine to me, haha.
   
5. QuLogic commented on Nov 20, 2021

   

   QuLogic

   on Nov 20, 2021

   Member

   More actions

   Ah, bmh sets the cm fontset, and that doesn't appear to support the minus sign properly. Also, it doesn't seem like the Log formatters do any minus sign glyph substitution, but I may have missed it.

   You can replicate this more easily by setting mpl.rcParams['mathtext.fontset'] = 'cm', and if you don't mind going back to DejaVu Sans, can work around it by setting mpl.rcParams['mathtext.fontset'] = 'dejavusans' after setting the style.
6. QuLogic

   reopened this on Nov 20, 2021
7. QuLogic

   changed the title [-][Bug]: minuses are shown as dashes when choosing the log scale[/-] [+][Bug]: cm fontset in log scale does not use Unicode minus[/+] on Nov 20, 2021
8. QuLogic

   added

   topic: text/fonts

   topic: ticks axis labels

   on Nov 20, 2021
9. QuLogic commented on Nov 20, 2021

   

   QuLogic

   on Nov 20, 2021

   Member

   More actions

   A possible fix may be to do:

   diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
   index 6d8fa5419b..c759403bbf 100644
   --- a/lib/matplotlib/ticker.py
   +++ b/lib/matplotlib/ticker.py
   @@ -1066,7 +1066,7 @@ class LogFormatterMathtext(LogFormatter):
            if x == 0:  # Symlog
                return r'$\mathdefault{0}$'
    
   -        sign_string = '-' if x < 0 else ''
   +        sign_string = self.fix_minus('-') if x < 0 else ''
            x = abs(x)
            b = self._base

   though I'm not sure if this should be applied to LogFormatter as well (probably it should).
10. anntzer commented on Nov 20, 2021

    

    anntzer

    on Nov 20, 2021

    Last edited by anntzer

    Contributor

    More actions

    Agreed that we should use fix_minus in as many places as possible. (But see also #21687.)

    👍React with 👍1Reacted by Alfredo Canziani
11. Atcold commented on Dec 3, 2021

    

    Atcold

    on Dec 3, 2021

    Author

    More actions

    Thanks for going to the bottom of this.
12. QuLogic

    added a commit that references this issue on Jan 7, 2022

    Ensure log formatters use Unicode minus

    ...

    This issue will close once commit 52003e4 is merged into the 'main' branch.

    52003e4
13. dstansby

    closed this as completedin #22140on Jan 9, 2022
14. QuLogic

    added this to the v3.6.0 milestone on Jan 11, 2022
15. stanleyjs

    added a commit that references this issue on Jan 26, 2022

    subfigure _axstack property

    ...

    This issue will close once commit df85ec0 is merged into the 'main' branch.

    df85ec0