Fix DR #84

changelog/74.fix.md

@@ -0,0 +1 @@
+Updated the cell methods and description of `esn_tavg-u-hxy-sn` to clarify the meaning (see [DR #84](https://github.com/CMIP-Data-Request/Harmonised-Public-Consultation/issues/84))

scripts/84-dr-fixes.py

@@ -0,0 +1,39 @@
+"""
+Fixes from DR #84
+"""
+
+from __future__ import annotations
+
+import json
+from pathlib import Path
+
+
+def main():
+    """
+    Apply the fixes
+    """
+    REPO_ROOT = Path(__file__).parents[1]
+
+    for branded_variable, new_cell_methods, new_description in (
+        (
+            "esn_tavg-u-hxy-sn",
+            "area: mean where snow (on land only) time: mean",
+            "Evapotranspiration flux due to conversion of liquid or solid water into vapor at the surface where there is snow on land",
+        ),
+    ):
+        with open(
+            REPO_ROOT / "src-data" / "variable" / f"{branded_variable}.json"
+        ) as fh:
+            working = json.load(fh)
+
+        working["cell-methods"] = new_cell_methods
+        working["description"] = new_description
+
+        with open(
+            REPO_ROOT / "src-data" / "variable" / f"{branded_variable}.json", "w"
+        ) as fh:
+            json.dump(working, fh, indent=4)
+
+
+if __name__ == "__main__":
+    main()

src-data/variable/esn_tavg-u-hxy-sn.json

@@ -2,10 +2,10 @@
     "id": "esn_tavg-u-hxy-sn",
     "validation-key": "esn_tavg-u-hxy-sn",
     "ui-label": "Snow Evaporation",
-    "description": "liquid\\_water\\_evaporation\\_flux\\_from\\_surface\\_snow",
+    "description": "Evapotranspiration flux due to conversion of liquid or solid water into vapor at the surface where there is snow on land",
     "area-label": "sn",
     "cell-measures": "area: areacella",
-    "cell-methods": "area: mean where land time: mean",
+    "cell-methods": "area: mean where snow (on land only) time: mean",
     "dimensions": [
         "longitude",
         "latitude",