From 82720fd5409a97db241c66cbd5345efb50b8012d Mon Sep 17 00:00:00 2001 From: George McCabe <23407799+georgemccabe@users.noreply.github.com> Date: Mon, 6 Jun 2022 13:23:56 -0600 Subject: [PATCH 1/3] per #1641, remove list variables from list of items that are formatted as a string. These values are found in the above list of items that are formatted as a list, but the correct env var values are overwritten since they are included in this list incorrectly --- metplus/wrappers/stat_analysis_wrapper.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/metplus/wrappers/stat_analysis_wrapper.py b/metplus/wrappers/stat_analysis_wrapper.py index 4ff079b6e..c7afbb3f4 100755 --- a/metplus/wrappers/stat_analysis_wrapper.py +++ b/metplus/wrappers/stat_analysis_wrapper.py @@ -1793,9 +1793,6 @@ def run_stat_analysis_job(self, runtime_settings_dict_list): 'FCST_INIT_END', 'OBS_INIT_BEG', 'OBS_INIT_END', - 'DESC', - 'OBTYPE', - 'FCST_LEAD' ] for mp_item in mp_items: if not runtime_settings_dict.get(mp_item, ''): From 79e2f75f1b86f6e83a7272d16a6e4cbe42ae8a17 Mon Sep 17 00:00:00 2001 From: George McCabe <23407799+georgemccabe@users.noreply.github.com> Date: Mon, 6 Jun 2022 13:24:29 -0600 Subject: [PATCH 2/3] moved variable name to match the order found in the MET config file for clarity --- metplus/wrappers/stat_analysis_wrapper.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/metplus/wrappers/stat_analysis_wrapper.py b/metplus/wrappers/stat_analysis_wrapper.py index c7afbb3f4..f551bc3d6 100755 --- a/metplus/wrappers/stat_analysis_wrapper.py +++ b/metplus/wrappers/stat_analysis_wrapper.py @@ -1756,7 +1756,6 @@ def run_stat_analysis_job(self, runtime_settings_dict_list): # with other wrappers mp_lists = ['MODEL', 'DESC', - 'OBTYPE', 'FCST_LEAD', 'OBS_LEAD', 'FCST_VALID_HOUR', @@ -1769,6 +1768,7 @@ def run_stat_analysis_job(self, runtime_settings_dict_list): 'OBS_UNITS', 'FCST_LEVEL', 'OBS_LEVEL', + 'OBTYPE', 'VX_MASK', 'INTERP_MTHD', 'INTERP_PNTS', From b824da7924b545455b958cb45d51502b2332fe5e Mon Sep 17 00:00:00 2001 From: George McCabe <23407799+georgemccabe@users.noreply.github.com> Date: Mon, 6 Jun 2022 13:24:42 -0600 Subject: [PATCH 3/3] renamed incorrect variable name from conv_thresh to cov_thresh --- metplus/wrappers/stat_analysis_wrapper.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/metplus/wrappers/stat_analysis_wrapper.py b/metplus/wrappers/stat_analysis_wrapper.py index f551bc3d6..8a7ceee31 100755 --- a/metplus/wrappers/stat_analysis_wrapper.py +++ b/metplus/wrappers/stat_analysis_wrapper.py @@ -1774,7 +1774,7 @@ def run_stat_analysis_job(self, runtime_settings_dict_list): 'INTERP_PNTS', 'FCST_THRESH', 'OBS_THRESH', - 'CONV_THRESH', + 'COV_THRESH', 'ALPHA', 'LINE_TYPE' ]