Merge pull request #14832 from sameshl/tick_formatter_limits

tacaswell · web-flow · commit ac0a423c79ac · 2019-08-22T19:21:21.000-04:00
API: make default tick formatter switch to scientific notation earlier
diff --git a/doc/api/next_api_changes/2019-07-18-SL.rst b/doc/api/next_api_changes/2019-07-18-SL.rst
@@ -0,0 +1,4 @@
+Reduced default value of `axes.formatter.limits`
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+Changed the default value of `axes.formatter.limits` from -7, 7 to -5, 6 for better readability.
diff --git a/lib/matplotlib/rcsetup.py b/lib/matplotlib/rcsetup.py
@@ -1150,7 +1150,7 @@ def _validate_linestyle(ls):
     'axes.labelpad':         [4.0, validate_float],  # space between label and axis
     'axes.labelweight':      ['normal', validate_string],  # fontsize of the x any y labels
     'axes.labelcolor':       ['black', validate_color],    # color of axis label
-    'axes.formatter.limits': [[-7, 7], validate_nseq_int(2)],
+    'axes.formatter.limits': [[-5, 6], validate_nseq_int(2)],
                                # use scientific notation if log10
                                # of the axis range is smaller than the
                                # first or larger than the second
diff --git a/lib/matplotlib/tests/test_axes.py b/lib/matplotlib/tests/test_axes.py
@@ -547,6 +547,9 @@ def test_single_point():
 
 @image_comparison(['single_date.png'], style='mpl20')
 def test_single_date():
+    # use former defaults to match existing baseline image
+    plt.rcParams['axes.formatter.limits'] = -7, 7
+
     time1 = [721964.0]
     data1 = [-65.54]
 
diff --git a/lib/matplotlib/tests/test_units.py b/lib/matplotlib/tests/test_units.py
@@ -76,6 +76,9 @@ def default_units(value, axis):
 @image_comparison(['plot_pint.png'], remove_text=False, style='mpl20',
                   tol={'aarch64': 0.02}.get(platform.machine(), 0.0))
 def test_numpy_facade(quantity_converter):
+    # use former defaults to match existing baseline image
+    plt.rcParams['axes.formatter.limits'] = -7, 7
+
     # Register the class
     munits.registry[Quantity] = quantity_converter
 
