TST

Author: jklymak

lib/matplotlib/path.py

@@ -986,22 +986,20 @@ def arc(cls, theta1, theta2, n=None, is_wedge=False, wrap=True):
 
         Notes
         -----
-           The arc is approximated using cubic Bézier curves, as described in
-           Masionobe, L.  2003.  `Drawing an elliptical arc using polylines,
-           quadratic or cubic Bezier curves
-           <https://web.archive.org/web/20190318044212/http://www.spaceroots.org/documents/ellipse/index.html>`_.
+        The arc is approximated using cubic Bézier curves, as described in
+        Masionobe, L.  2003.  `Drawing an elliptical arc using polylines,
+        quadratic or cubic Bezier curves
+        <https://web.archive.org/web/20190318044212/http://www.spaceroots.org/documents/ellipse/index.html>`_.
         """
 
         eta1 = theta1
         if wrap:
             # Wrap theta2 to 0-360 degrees from theta1.
             eta2 = np.mod(theta2 - theta1, 360.0) + theta1
-            print('Eta1, Eta20', eta1, eta2)
             # Ensure 360-deg range is not flattened to 0 due to floating-point
             # errors, but don't try to expand existing 0 range.
             if theta2 != theta1 and eta2 <= eta1:
                 eta2 += 360
-            print('Eta1, Eta2', eta1, eta2)
         else:
             eta2 = theta2
         eta1, eta2 = np.deg2rad([eta1, eta2])
@@ -1012,9 +1010,8 @@ def arc(cls, theta1, theta2, n=None, is_wedge=False, wrap=True):
                 # this doesn't need to grow exponentially, but we have left
                 # this way for back compatibility
                 n = int(2 ** np.ceil(2 * np.abs(eta2 - eta1) / np.pi))
-                print('Here')
             else:
-                # this will not grow exponentially if we allow wrapping arcs:
+                # this will grow linearly if we allow wrapping arcs:
                 n = int(2 * np.ceil(2 * np.abs(eta2 - eta1) / np.pi))
         if n < 1:
             raise ValueError("n must be >= 1 or None")

lib/matplotlib/tests/baseline_images/test_path/arc_wrap_false.png

@@ -481,7 +481,7 @@ def test_full_arc(offset):
 @image_comparison(['arc_close360'], style='default', remove_text=True,
                   extensions=['png'])
 def test_arc_close360():
-    fig, ax = plt.subplots(ncols=3)
+    _, ax = plt.subplots(ncols=3)
     ax[0].add_patch(patches.PathPatch(Path.arc(theta1=-90 - 1e-14, theta2=270)))
     #ax[0].set_title("arc(-90-1e-14, 270), should be a circle")
     ax[1].add_patch(patches.PathPatch(Path.arc(theta1=-90, theta2=270)))
@@ -494,6 +494,26 @@ def test_arc_close360():
         a.set_aspect("equal")
 
 
+@image_comparison(['arc_wrap_false'], style='default', remove_text=True,
+                  extensions=['png'])
+def test_arc_wrap_false():
+    _, ax = plt.subplots(2, 2)
+    ax = ax.flatten()
+    ax[0].add_patch(patches.PathPatch(Path.arc(theta1=10, theta2=20,
+                                               is_wedge=True, wrap=True)))
+    ax[1].add_patch(patches.PathPatch(Path.arc(theta1=10, theta2=380,
+                                               is_wedge=True, wrap=True)))
+    ax[2].add_patch(patches.PathPatch(Path.arc(theta1=10, theta2=20,
+                                               is_wedge=True, wrap=False)))
+    ax[3].add_patch(patches.PathPatch(Path.arc(theta1=10, theta2=740,
+                                               is_wedge=True, wrap=False)))
+    for a in ax:
+        a.set_xlim(-1, 1)
+        a.set_ylim(-1, 1)
+        a.set_aspect("equal")
+
+
+
 def test_disjoint_zero_length_segment():
     this_path = Path(
         np.array([