Make compose nrow & ncol attributes

Author: has2k1

plotnine/composition/_beside.py

@@ -30,13 +30,9 @@ class Beside(Compose):
     plotnine.composition.Compose : For more on composing plots
     """
 
-    @property
-    def nrow(self) -> int:
-        return 1
-
-    @property
-    def ncol(self) -> int:
-        return len(self)
+    def __post_init__(self):
+        super().__post_init__()
+        self.ncol = len(self)
 
     def __or__(self, rhs: ggplot | Compose) -> Compose:
         """

plotnine/composition/_compose.py

@@ -109,6 +109,8 @@ def __post_init__(self):
             op if isinstance(op, Compose) else deepcopy(op)
             for op in self.items
         ]
+        self.nrow = 1
+        self.ncol = 1
 
     def __repr__(self):
         """
@@ -269,20 +271,6 @@ def _repr_mimebundle_(self, include=None, exclude=None) -> MimeBundle:
         figure_size_px = self.last_plot.theme._figure_size_px
         return get_mimebundle(buf.getvalue(), format, figure_size_px)
 
-    @property
-    def nrow(self) -> int:
-        """
-        Number of rows in the composition
-        """
-        return 0
-
-    @property
-    def ncol(self) -> int:
-        """
-        Number of cols in the composition
-        """
-        return 0
-
     @property
     def last_plot(self) -> ggplot:
         """

plotnine/composition/_stack.py

@@ -30,13 +30,9 @@ class Stack(Compose):
     plotnine.composition.Compose : For more on composing plots
     """
 
-    @property
-    def nrow(self) -> int:
-        return len(self)
-
-    @property
-    def ncol(self) -> int:
-        return 1
+    def __post_init__(self):
+        super().__post_init__()
+        self.nrow = len(self)
 
     def __truediv__(self, rhs: ggplot | Compose) -> Compose:
         """