From 9d5392d0c5db7f4b71f066c0910d910405c185cb Mon Sep 17 00:00:00 2001
From: Zeitsperre <10819524+Zeitsperre@users.noreply.github.com>
Date: Fri, 31 May 2024 11:30:25 -0400
Subject: [PATCH] clip minimum values for tp and sf

---
 CHANGES.rst                              |  1 +
 miranda/convert/data/ecmwf_cf_attrs.json | 12 ++++++++++++
 2 files changed, 13 insertions(+)

diff --git a/CHANGES.rst b/CHANGES.rst
index 91ee6e6f..3141c5d3 100644
--- a/CHANGES.rst
+++ b/CHANGES.rst
@@ -16,6 +16,7 @@ New features
 ^^^^^^^^^^^^
 * Aggregation operations now support more variables (`hur`, `hurs`, `huss`, `rlds`, `ta`, `tdp`, `ua`, `uas`, `va`, `vas`)
 * `RDRDv21` has been added as a dataset to be converted.
+* Minimum values of `0 kg m2 s-1` has been set for both `tp` and `sf` variables in ERA5 and ERA5-Land projects.
 
 Bug fixes
 ^^^^^^^^^
diff --git a/miranda/convert/data/ecmwf_cf_attrs.json b/miranda/convert/data/ecmwf_cf_attrs.json
index 1a0afa83..1c080ac3 100644
--- a/miranda/convert/data/ecmwf_cf_attrs.json
+++ b/miranda/convert/data/ecmwf_cf_attrs.json
@@ -201,6 +201,12 @@
     },
     "sf": {
       "_cf_variable_name": "prsn",
+      "_clip_values": {
+        "all": {
+          "context": "hydro",
+          "min": "0 kg m-2 s-1"
+        }
+      },
       "_corrected_units": "m",
       "_invert_sign": false,
       "_offset_time": {
@@ -436,6 +442,12 @@
     },
     "tp": {
       "_cf_variable_name": "pr",
+      "_clip_values": {
+        "all": {
+          "context": "hydro",
+          "min": "0 kg m-2 s-1"
+        }
+      },
       "_corrected_units": false,
       "_invert_sign": false,
       "_offset_time": true,