Merge 937196e into 95b4408

regions/core/core.py

@@ -161,6 +161,36 @@ def to_shapely(self):
         """
         raise NotImplementedError("")
 
+    @abc.abstractmethod
+    def as_patch(self, **kwargs):
+        """Convert to mpl patch
+
+        Returns
+        -------
+        patch : `~matplotlib.patches.Patch`
+            Matplotlib patch
+        """
+        raise NotImplementedError
+
+    def plot(self, ax=None, **kwargs):
+        """
+        Calls as_patch method forwarding all kwargs and adds patch
+        to given axis.
+
+        Parameters
+        ----------
+        ax : `~matplotlib.axes`, optional
+            Axis
+        """
+        import matplotlib.pyplot as plt
+
+        ax = plt.gca() if ax is None else ax
+
+        patch = self.as_patch(**kwargs)
+        ax.add_patch(patch)
+
+        return ax
+
 
 @six.add_metaclass(abc.ABCMeta)
 class SkyRegion(Region):
@@ -244,3 +274,34 @@ def to_pixel(self, wcs, mode='local', tolerance=None):
             The tolerance for the ``'full'`` mode described above.
         """
         raise NotImplementedError("")
+
+    @abc.abstractmethod
+    def as_patch(self, ax, **kwargs):
+        """Convert to mpl patch using a given wcs axis
+
+        Parameters
+        ----------
+        ax : `~astropy.wcsaxes.WCSAxes`
+            WCS axis object
+
+        Returns
+        -------
+        patch : `~matplotlib.patches.Patch`
+            Matplotlib patch
+        """
+        raise NotImplementedError
+
+    def plot(self, ax=None, **kwargs):
+        """
+        Calls as_patch method forwarding all kwargs and adds patch
+        to given wcs axis.
+
+        Parameters
+        ----------
+        ax : `~astropy.wcsaxes.WCSAxes`
+            WCS axis object
+        """
+        patch = self.as_patch(ax, **kwargs)
+        ax.add_patch(patch)
+
+        return ax

regions/shapes/circle.py

@@ -50,14 +50,7 @@ def to_mask(self, mode='center'):
         # TODO: needs to be implemented
         raise NotImplementedError("")
 
-    def to_mpl_patch(self, **kwargs):
-        """Convert to mpl patch.
-
-        Returns
-        -------
-        patch : `~matplotlib.mpatches.Circle`
-            Matplotlib patch
-        """
+    def as_patch(self, **kwargs):
         import matplotlib.patches as mpatches
 
         patch = mpatches.Circle(self.center, self.radius, **kwargs)
@@ -134,22 +127,7 @@ def to_pixel(self, mywcs, mode='local', tolerance=None):
 
         return CirclePixelRegion(pixel_positions, radius_pix)
 
-    def to_mpl_patch(self, ax, **kwargs):
-        """Convert to mpl patch using a given wcs axis
-
-        Parameters
-        ----------
-        ax : `~astropy.wcsaxes.WCSAxes`
-            WCS axis object
-        kwargs : dict
-            kwargs are forwarded to mpatches.Circle
-
-        Returns
-        -------
-        patch : `~matplotlib.mpatches.Circle`
-            Matplotlib patch
-        """
-
+    def as_patch(self, ax, **kwargs):
         import matplotlib.patches as mpatches
 
         val = self.center.icrs

regions/shapes/ellipse.py

@@ -52,6 +52,9 @@ def to_mask(self, mode='center'):
         # TODO: needs to be implemented
         raise NotImplementedError("")
 
+    def as_patch(self, **kwargs):
+        # TODO: needs to be implemented
+        raise NotImplementedError("")
 
 
 class EllipseSkyRegion(SkyRegion):
@@ -92,3 +95,7 @@ def __contains__(self, skycoord):
     def to_pixel(self, wcs, mode='local', tolerance=None):
         # TODO: needs to be implemented
         raise NotImplementedError("")
+
+    def as_patch(self, **kwargs):
+        # TODO: needs to be implemented
+        raise NotImplementedError("")

regions/shapes/point.py

@@ -40,6 +40,10 @@ def to_mask(self, mode='center'):
         # TODO: needs to be implemented
         raise NotImplementedError("")
 
+    def as_patch(self, **kwargs):
+        # TODO: needs to be implemented
+        raise NotImplementedError("")
+
 
 
 class PointSkyRegion(SkyRegion):
@@ -68,3 +72,8 @@ def __contains__(self, skycoord):
     def to_pixel(self, wcs, mode='local', tolerance=None):
         # TODO: needs to be implemented
         raise NotImplementedError("")
+
+    def as_patch(self, **kwargs):
+        # TODO: needs to be implemented
+        raise NotImplementedError("")
+

regions/shapes/polygon.py

@@ -38,6 +38,9 @@ def to_mask(self, mode='center'):
         # TODO: needs to be implemented
         raise NotImplementedError("")
 
+    def as_patch(self, **kwargs):
+        # TODO: needs to be implemented
+        raise NotImplementedError("")
 
 
 class PolygonSkyRegion(SkyRegion):
@@ -68,3 +71,7 @@ def __contains__(self, skycoord):
     def to_pixel(self, wcs, mode='local', tolerance=None):
         # TODO: needs to be implemented
         raise NotImplementedError("")
+
+    def as_patch(self, **kwargs):
+        # TODO: needs to be implemented
+        raise NotImplementedError("")

regions/shapes/rectangle.py

@@ -71,7 +71,7 @@ class RectangleSkyRegion(SkyRegion):
     def __init__(self, center, height, width, angle=0 * u.deg, meta=None, visual=None):
         # TODO: use quantity_input to check that height, width, and angle are angles
         self.center = center
-        self.height = minor
+        self.height = height
         self.width = width
         self.angle = angle
         self.meta = meta or {}

regions/shapes/tests/test_circle.py

@@ -38,9 +38,10 @@ def test_plot():
     wcs = WCS(h)
     fig = plt.figure()
     ax = fig.add_axes([0.1, 0.1, 0.8, 0.8], projection=wcs)
-    p = c.to_mpl_patch(ax, alpha=0.6)
-    ax.add_patch(p)
+    p = c.as_patch(ax, alpha=0.6)
 
     assert_allclose(p.center[0], skycoord.icrs.ra.value)
     assert_allclose(p.center[1], skycoord.icrs.dec.value)
     assert p.get_facecolor() == (1, 0, 0, 0.6)
+
+    c.plot(ax, alpha=0.6)