enhance test and ghactions

.github/workflows/pytest.yaml

@@ -1,9 +1,9 @@
-name: pytest-dev
+name: pytest
 
 on:
   push:
     branches:
-        - dev
+        - [main,dev]
 
 jobs:
   pytest:

.github/workflows/ruff.yaml

@@ -1,4 +1,4 @@
-name: Ruff
+name: ruff
 
 on:
   push:
@@ -23,4 +23,4 @@ jobs:
         args: format
     - uses: stefanzweifel/git-auto-commit-action@v5
       with:
-        commit_message: 'ghactions/auto fix with ruff'
+        commit_message: 'gh-actions/python format using ruff'

examples/first_steps_create_datacube.py

@@ -37,7 +37,6 @@
     intersects=geometry,
     datetime=["2022-08-01", "2022-08-09"],
     mask_with="native",  # equal to "scl" for sentinel-2
-    mask_statistics=True,
     clear_cover=50,
 )
 

tests/test_zonalstats.py

@@ -43,38 +43,24 @@ def test_numpy(self):
             self.datacube,
             self.gdf,
             all_touched=True,
-            operations=dict(mean=np.nanmean, max=np.nanmax, min=np.nanmin),
-        )
-        self.assertTrue(
-            np.all(
-                zonalstats["first_var"].sel(stats="max").values
-                == np.asarray([[8, self.constant], [23, self.constant]])
-            )
-        )
-        self.assertTrue(
-            np.all(
-                zonalstats["first_var"].sel(stats="min").values
-                == np.asarray([[0, self.constant], [9, self.constant]])
-            )
+            operations=dict(mean=np.nanmean, max=np.nanmax, min=np.nanmin, mode=np.mode),
         )
 
+        for operation in ['min','max', 'mode']:
+            self._check_results(zonalstats["first_var"].sel(stats=operation).values, operation=operation)
+
     def test_basic(self):
         zonalstats = earthdaily.earthdatastore.cube_utils.zonal_stats(
-            self.datacube, self.gdf, all_touched=True, operations=["min", "max", "mean"]
+            self.datacube, self.gdf, all_touched=True, operations=["min", "max", "mode"]
         )
-        self.assertTrue(
-            np.all(
-                zonalstats["first_var"].sel(stats="max").values
-                == np.asarray([[8, self.constant], [23, self.constant]])
-            )
-        )
-        self.assertTrue(
-            np.all(
-                zonalstats["first_var"].sel(stats="min").values
-                == np.asarray([[0, self.constant], [9, self.constant]])
-            )
-        )
-
+        for operation in ['min','max','mode']:
+            self._check_results(zonalstats["first_var"].sel(stats=operation).values, operation=operation)
+
+    def _check_results(self, stats_values,operation='min'):
+        results = {"min":np.asarray([[0, self.constant], [9, self.constant]]),
+                   "max":np.asarray([[8, self.constant], [23, self.constant]]),
+                   "mode":np.asarray([[0, self.constant], [9, self.constant]])}
+        self.assertTrue(np.all(stats_values == results[operation]))
 
 if __name__ == "__main__":
     unittest.main()