Apply suggestions from code review

MetOffice · Jan 21, 2025 · e480d36 · e480d36
1 parent 37a8dd6
commit e480d36
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/CSET/operators/collapse.py b/src/CSET/operators/collapse.py
@@ -108,7 +108,8 @@ def collapse_by_hour_of_day(
     Collapsing of the cube is around the 'time' coordinate. The coordinates are
     first grouped by the hour of day, and then aggregated by the hour of day
     to create a diurnal cycle. This operator is applicable for both single
-    forecasts and for multiple forecasts.
+    forecasts and for multiple forecasts. The hour used is based on the units of
+    the time coordinate. If the time coordinate is in UTC, hour will be in UTC.
 
     To apply this operator successfully there must only be one time dimension.
     Should a MultiDim exception be raised the user first needs to apply the

diff --git a/tests/operators/test_collapse.py b/tests/operators/test_collapse.py
@@ -25,7 +25,7 @@
 def long_forecast() -> iris.cube.Cube:
     """Get long_forecast to run tests on."""
     return iris.load_cube(
-        "tests/test_data/long_forecast_air_temperature.nc", "air_temperature"
+        "tests/test_data/long_forecast_air_temp_fcst_1.nc", "air_temperature"
     )
 
 

diff --git a/...est_data/long_forecast_air_temperature.nc → ...est_data/long_forecast_air_temp_fcst_1.nc b/...est_data/long_forecast_air_temperature.nc → ...est_data/long_forecast_air_temp_fcst_1.nc