Merge pull request #2713 from CliMA/js/diags

overwrite duplicate diagnostics
CliMA · Feb 27, 2024 · 6e42940 · 6e42940
2 parents 30b3cd8 + de58d31
commit 6e42940
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/diagnostics/diagnostic.jl b/src/diagnostics/diagnostic.jl
@@ -152,8 +152,14 @@ function add_diagnostic_variable!(;
     comments = "",
     compute!,
 )
-    haskey(ALL_DIAGNOSTICS, short_name) &&
-        error("diagnostic $short_name already defined")
+    haskey(ALL_DIAGNOSTICS, short_name) && @warn(
+        "overwriting diagnostic `$short_name` entry containing fields\n" *
+        "$(map(
+            field -> "$(getfield(ALL_DIAGNOSTICS[short_name], field))",
+            filter(field -> field != :compute!, fieldnames(DiagnosticVariable)),
+        ))"
+    )
+
 
     ALL_DIAGNOSTICS[short_name] = DiagnosticVariable(;
         short_name,