Changes to comments.

ufs-community · Nov 25, 2023 · fda85e8 · fda85e8
1 parent 7305347
commit fda85e8
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 26 deletions.
diff --git a/parm/metplus/EnsembleStat_APCP.conf b/parm/metplus/EnsembleStat_APCP.conf
@@ -179,7 +179,7 @@ ENSEMBLE_STAT_ENSEMBLE_FLAG_NMEP = FALSE
 ENSEMBLE_STAT_ENSEMBLE_FLAG_RANK = FALSE
 ENSEMBLE_STAT_ENSEMBLE_FLAG_WEIGHT = FALSE
 #
-# Precipitation accumulation interval in 2-digit hours.
+# Accumulation interval in 2-digit hours.
 #
 ACCUM_HH = {{accum_hh}}
 #

diff --git a/parm/metplus/EnsembleStat_ASNOW.conf b/parm/metplus/EnsembleStat_ASNOW.conf
@@ -179,7 +179,7 @@ ENSEMBLE_STAT_ENSEMBLE_FLAG_NMEP = FALSE
 ENSEMBLE_STAT_ENSEMBLE_FLAG_RANK = FALSE
 ENSEMBLE_STAT_ENSEMBLE_FLAG_WEIGHT = FALSE
 #
-# Precipitation accumulation interval in 2-digit hours.
+# Accumulation interval in 2-digit hours.
 #
 ACCUM_HH = {{accum_hh}}
 #

diff --git a/parm/metplus/GridStat_APCP.conf b/parm/metplus/GridStat_APCP.conf
@@ -113,6 +113,10 @@ GRID_STAT_DESC = NA
 
 GRID_STAT_MET_CONFIG_OVERRIDES = cat_thresh = [NA]; cnt_thresh = [NA]; cnt_logic = UNION; wind_thresh = [NA]; wind_logic = UNION; ci_alpha = [0.05]; rank_corr_flag = FALSE;
 #
+# Accumulation interval in 2-digit hours.
+#
+ACCUM_HH = {{accum_hh}}
+#
 # List of forecast and corresponding observation fields to process.
 #
 # Note on use of set_attr_lead and ensemble member time-lagging:
@@ -130,7 +134,6 @@ GRID_STAT_MET_CONFIG_OVERRIDES = cat_thresh = [NA]; cnt_thresh = [NA]; cnt_logic
 # cause confusion, so here, we always use lead values with zero lead
 # corresponding to the nominal cdate.
 #
-ACCUM_HH = {{accum_hh}}
 FCST_VAR1_NAME = {{fieldname_in_met_output}}
 FCST_VAR1_LEVELS = A{ACCUM_HH}
 FCST_VAR1_THRESH = {{field_thresholds}}

diff --git a/parm/metplus/GridStat_ASNOW.conf b/parm/metplus/GridStat_ASNOW.conf
@@ -93,6 +93,10 @@ OBTYPE = {{obtype}}
 # Value to enter under the DESC column in the output stat file.
 #
 GRID_STAT_DESC = NA
+#
+# Accumulation interval in 2-digit hours.
+#
+ACCUM_HH = {{accum_hh}}
 
 # List of variables to compare in GridStat - FCST_VAR1 variables correspond
 #  to OBS_VAR1 variables
@@ -104,7 +108,6 @@ GRID_STAT_MET_CONFIG_OVERRIDES = cat_thresh = [NA]; cnt_thresh = [NA]; cnt_logic
 # String to search for in the foreast input files for forecast variable
 # 1.
 #
-ACCUM_HH = {{accum_hh}}
 FCST_VAR1_NAME = {{fieldname_in_met_output}}
 FCST_VAR1_LEVELS = A{ACCUM_HH}
 FCST_VAR1_THRESH = {{field_thresholds}}

diff --git a/parm/metplus/GridStat_ensmean_APCP.conf b/parm/metplus/GridStat_ensmean_APCP.conf
@@ -101,7 +101,7 @@ GRID_STAT_DESC = NA
 
 GRID_STAT_MET_CONFIG_OVERRIDES = cat_thresh = [NA]; cnt_thresh = [NA]; cnt_logic = UNION; wind_thresh = [NA]; wind_logic = UNION; ci_alpha = [0.05]; rank_corr_flag = FALSE;
 #
-# Precipitation accumulation interval in 2-digit hours.
+# Accumulation interval in 2-digit hours.
 #
 ACCUM_HH = {{accum_hh}}
 #

diff --git a/parm/metplus/GridStat_ensmean_ASNOW.conf b/parm/metplus/GridStat_ensmean_ASNOW.conf
@@ -106,28 +106,11 @@ GRID_STAT_DESC = NA
 
 GRID_STAT_MET_CONFIG_OVERRIDES = cat_thresh = [NA]; cnt_thresh = [NA]; cnt_logic = UNION; wind_thresh = [NA]; wind_logic = UNION; ci_alpha = [0.05]; rank_corr_flag = FALSE;
 #
-# String to search for in the foreast input files for forecast variable
-# 1.
+# Accumulation interval in 2-digit hours.
 #
-# Note:
-# This is the name of the field in the NetCDF file(s) created by MET's
-# gen_ens_prod tool.  This tool reads in the grib2 file(s) (in this case
-# of forecasts) and outputs NetCDF file(s) in which the array names
-# consist of the value of fieldname_in_met_output plus a suffix that
-# specifies additional properties of the data in the array such as the
-# level (which in this case is the accumulation period), the type of
-# statistic, etc.  In this case, this suffix is "_A01_ENS_MEAN".  Thus,
-# below, FCST_VAR1_NAME must be set to the value of fieldname_in_met_output
-# with "_A01_ENS_MEAN" appended to it.
-#
-# Question:
-# Why is the name of the array in the input NetCDF file (created by
-# GenEnsProd) that this task reads contains ..._A01_ENS_MEAN instead of
-# ..._A1_ENS_MEAN, as is the case for greater than 1-hour accumulation?
-# If that were the case, we could use ACCUM_NO_PAD here instead of hard-
-# coding the "A01", which would make it consistent with what is done in
-# the METplus conf files for accumulation > 1 hour (e.g. 3, 6, and 24
-# hour).
+ACCUM_HH = {{accum_hh}}
+#
+# List of forecast and corresponding observation fields to process.
 #
 ACCUM_HH = {{accum_hh}}
 FCST_VAR1_NAME = {{fieldname_in_met_output}}_A{{accum_no_pad}}_ENS_MEAN