René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[Bug]: LogFormatter minor ticks with `minor_thresholds` of (0,0) does not behave as documented","articleBody":"### Bug summary\n\nIn investigating #25894 I discovered that despite being documented as:\r\n\r\n\u003e To disable labeling of minor ticks when 'labelOnlyBase' is False, use `minor_thresholds=(0, 0)`. This is the default for the \"classic\" style.\r\n\r\nIt actually only removes some minor tick labels  (e.g. with base of 10: 11-14 will get labels, but 16-19 (15 is edge case, dependent on floating point rounding, 1.4, or 140 also similar))\n\n### Code for reproduction\n\n```python\nimport matplotlib.pyplot as plt\r\nimport matplotlib.ticker as mticker\r\n\r\nfig, ax = plt.subplots()\r\nax.set_yscale(\"log\")\r\n\r\nformatter = mticker.LogFormatter(minor_thresholds=(0,0))\r\nax.yaxis.set_minor_formatter(formatter)\r\n\r\nax.set_ylim(10, 20)\r\nplt.show()\n```\n\n\n### Actual outcome\n\n\r\n![Figure_2](https://github.com/matplotlib/matplotlib/assets/2501846/ea3f8e41-0872-42d4-8e5d-c3a479a4f925)\r\n\r\n\r\n```python\r\nformatter(10) # '10'\r\nformatter(11) # '11'\r\nformatter(12) # '12'\r\nformatter(13) # '13'\r\nformatter(14) # '14'\r\nformatter(15) # '15'\r\nformatter(16) # ''\r\n```\n\n### Expected outcome\n\n\r\n![Figure_2](https://github.com/matplotlib/matplotlib/assets/2501846/874d28bb-2f8e-43e1-a3dd-b76dfad64bbf)\r\n\r\n\r\n```python\r\nformatter(10) # ''\r\nformatter(11) # ''\r\nformatter(12) # ''\r\nformatter(13) # ''\r\nformatter(14) # ''\r\nformatter(15) # ''\r\nformatter(16) # ''\r\n```\n\n### Additional information\n\nOnly really affects plots that are zoomed in to much less than one decade, which arguably doesn't make sense to use a log scale, but still seems incongruous with the documented \"this turns off minor ticks\".\r\n\r\nHas to do with when `coeff` rounds down to `1` in:\r\n\r\nhttps://github.com/matplotlib/matplotlib/blob/f051d9449820b5e7c01dcc1361d161517bff179f/lib/matplotlib/ticker.py#L980-L988\r\n\r\nI'm not _quite_ sure what the proper solution is, perhaps making `self._sublabels=set()` instead?\n\n### Operating system\n\nLinux\n\n### Matplotlib Version\n\n3.8 (main)\n\n### Matplotlib Backend\n\n_No response_\n\n### Python version\n\n3.11\n\n### Jupyter version\n\n_No response_\n\n### Installation\n\ngit checkout","author":{"url":"https://github.com/ksunden","@type":"Person","name":"ksunden"},"datePublished":"2023-05-16T21:31:35.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/25896/matplotlib/issues/25896"}