Apply suggestions from code review

Co-authored-by: Pascal Bourgault <[email protected]>
Ouranosinc · Jan 10, 2025 · 9a1760d · 9a1760d
1 parent 2484976
commit 9a1760d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 14 deletions.
diff --git a/src/xclim/data/fr.json b/src/xclim/data/fr.json
@@ -1034,16 +1034,16 @@
     "abstract": "Nombre de jours où la neige est entre une borne inférieure et supérieure."
   },
   "HOLIDAY_SNOW_DAYS": {
-    "long_name": "Nombre de jours de neige durant les jours fériés",
-    "description": "Nombre de jours de neige durant les jours fériés.",
-    "title": "Jours de neige durant les jours fériés",
-    "abstract": "Nombre de jours de neige durant les jours fériés."
+    "long_name": "Nombre de jours de neige durant les jours de Noël",
+    "description": "Nombre de jours de neige durant les jours de Noël.",
+    "title": "Jours de neige durant les jours de Noël",
+    "abstract": "Nombre de jours de neige durant les jours de Noël."
   },
   "HOLIDAY_SNOW_AND_SNOWFALL_DAYS": {
-    "long_name": "Nombre de jours de neige et de jours de chute de neige durant les jours fériés",
-    "description": "Nombre de jours de neige et de jours de chute de neige durant les jours fériés.",
-    "title": "Jours de neige et de chute de neige durant les jours fériés",
-    "abstract": "Nombre de jours de neige et de jours de chute de neige durant les jours fériés."
+    "long_name": "Nombre de jours de neige et de jours de chute de neige durant les jours de Noël",
+    "description": "Nombre de jours de neige et de jours de chute de neige durant les jours de Noël.",
+    "title": "Jours de neige et de chute de neige durant les jours de Noël",
+    "abstract": "Nombre de jours de neige et de jours de chute de neige durant les jours de Noël."
   },
   "SND_SEASON_LENGTH": {
     "long_name": "Durée de couvert de neige",

diff --git a/src/xclim/indices/_threshold.py b/src/xclim/indices/_threshold.py
@@ -3687,15 +3687,14 @@ def holiday_snow_days(
     """
     snd_constrained = select_time(
         snd,
-        drop=True,
         date_bounds=(date_start, date_start if date_end is None else date_end),
     )
 
     xmas_days = count_occurrences(
         snd_constrained, snd_thresh, freq, op, constrain=[">=", ">"]
     )
 
-    xmas_days = xmas_days.assign_attrs({"units": "days"})
+    xmas_days = to_agg_units(xmas_days, snd,  "count")
     return xmas_days
 
 
@@ -3709,7 +3708,7 @@ def holiday_snow_and_snowfall_days(
     snd: xarray.DataArray,
     prsn: xarray.DataArray | None = None,
     snd_thresh: Quantified = "20 mm",
-    prsn_thresh: Quantified = "1 cm",
+    prsn_thresh: Quantified = "1 mm",
     snd_op: str = ">=",
     prsn_op: str = ">=",
     date_start: str = "12-25",
@@ -3755,7 +3754,6 @@ def holiday_snow_and_snowfall_days(
     """
     snd_constrained = select_time(
         snd,
-        drop=True,
         date_bounds=(date_start, date_start if date_end is None else date_end),
     )
 
@@ -3764,7 +3762,6 @@ def holiday_snow_and_snowfall_days(
     )
     prsn_mm_constrained = select_time(
         prsn_mm,
-        drop=True,
         date_bounds=(date_start, date_start if date_end is None else date_end),
     )
 
@@ -3781,5 +3778,5 @@ def holiday_snow_and_snowfall_days(
         constrain_var2=[">=", ">"],
     )
 
-    perfect_xmas_days = perfect_xmas_days.assign_attrs({"units": "days"})
+    perfect_xmas_days = to_agg_units(perfect_xmas_days, snd, "count")
     return perfect_xmas_days