Small comment fixups from review

doronbehar · doronbehar · commit e0f08b5afbc3 · 2024-10-17T20:01:19.000+03:00
diff --git a/galleries/examples/ticks/engformatter_offset.py b/galleries/examples/ticks/engformatter_offset.py
@@ -16,6 +16,7 @@
 
 import matplotlib.ticker as mticker
 
+# Fixing random state for reproducibility
 np.random.seed(19680801)
 
 UNIT = "Hz"
diff --git a/lib/matplotlib/tests/test_ticker.py b/lib/matplotlib/tests/test_ticker.py
@@ -1642,6 +1642,9 @@ def test_engformatter_offset_oom(
     # only makes sure that offset text and the ticks gets the correct unit
     # prefixes and the ticks.
     if formatter.offset:
+        # These prefix_ variables are used only once, so we could have inlined
+        # them all, but it is more comfortable in case of tests breakages to
+        # view their values with pytest --showlocals.
         prefix_noise_got = offset_got[2]
         prefix_noise_desired = formatter.ENG_PREFIXES[oom_noise_desired]
         prefix_center_got = offset_got[-1-len(UNIT)]
@@ -1656,7 +1659,8 @@ def test_engformatter_offset_oom(
         assert offset_got == ""
         # Make sure the ticks contain now the prefixes
         for tick in ticks_got:
-            # 0 is zero on all orders of magnitudes, no
+            # 0 is zero on all orders of magnitudes, no matter what is
+            # oom_noise_desired
             if tick[0] == "0":
                 prefix_idx = 0
             else:
diff --git a/lib/matplotlib/ticker.py b/lib/matplotlib/ticker.py
@@ -1464,8 +1464,8 @@ def set_locs(self, locs):
 
     # Simplify a bit ScalarFormatter.get_offset: We always want to use
     # self.format_data. Also we want to return a non-empty string only if there
-    # is an offset, no matter what is self.orderOfMagnitude. if there is an
-    # offset OTH, self.orderOfMagnitude is consulted. This behavior is verified
+    # is an offset, no matter what is self.orderOfMagnitude. If there _is_ an
+    # offset, self.orderOfMagnitude is consulted. This behavior is verified
     # in `test_ticker.py`.
     def get_offset(self):
         # docstring inherited
