fix units

Signed-off-by: Trevor James Smith <[email protected]>
Ouranosinc · Jan 8, 2025 · 04787cb · 04787cb
1 parent 86554e4
commit 04787cb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/xclim/indices/_threshold.py b/src/xclim/indices/_threshold.py
@@ -3690,7 +3690,7 @@ def holiday_snow_days(
 
     xmas_days = (snow_depth_constrained >= snow_depth_thresh).resample(time=freq).sum()
 
-    xmas_days = xmas_days.assign_attrs({"units": "1"})
+    xmas_days = xmas_days.assign_attrs({"units": "days"})
     return xmas_days
 
 
@@ -3769,5 +3769,5 @@ def holiday_snow_and_snowfall_days(
         .sum()
     )
 
-    perfect_xmas_days = perfect_xmas_days.assign_attrs({"units": "1"})
+    perfect_xmas_days = perfect_xmas_days.assign_attrs({"units": "days"})
     return perfect_xmas_days
diff --git a/tests/test_snow.py b/tests/test_snow.py
@@ -135,7 +135,7 @@ def test_xmas_days_simple(self, nimbus):
 
         out = land.holiday_snow_days(snd)
 
-        assert out.units == "1"
+        assert out.units == "days"
         assert out.long_name == "Number of holiday days with snow"
         np.testing.assert_array_equal(
             out.sum(dim="time"),
@@ -166,7 +166,7 @@ def test_perfect_xmas_days_simple(self, nimbus):
 
         out = land.holiday_snow_and_snowfall_days(snd, prsn)
 
-        assert out.units == "1"
+        assert out.units == "days"
         assert out.long_name == "Number of holiday days with snow and snowfall"
         np.testing.assert_array_equal(
             out.sum(dim="time"),