Update main_v9.1-ref after #1730

.github/pull_request_template.md

@@ -4,7 +4,9 @@
 
 - [ ] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:</br>
 
-- [ ] Do these changes include sufficient documentation and testing updates? **[Yes or No]**
+- [ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? **[Yes or No]**
+
+- [ ] Do these changes include sufficient testing updates? **[Yes or No]**
 
 - [ ] Will this PR result in changes to the test suite? **[Yes or No]**</br>
 If **yes**, describe the new output and/or changes to the existing output:</br>

met/data/config/README

@@ -53,7 +53,7 @@ The configuration file language supports the following data types:
          - In the Gen-Vx-Mask "-thresh" command line option.
       - The following percentile threshold types are supported:
          - "SFP" for a percentile of the sample forecast values.
-           e.g. ">SFP50" means greater than the 50-th forecast percentile.
+           e.g. ">SFP33.3" means greater than the 33.3-rd forecast percentile.
          - "SOP" for a percentile of the sample observation values.
            e.g. ">SOP75" means greater than the 75-th observation percentile.
          - "SCP" for a percentile of the sample climatology values.

met/docs/Users_Guide/release-notes.rst

@@ -5,8 +5,15 @@ When applicable, release notes are followed by the GitHub issue number which
 describes the bugfix, enhancement, or new feature:
 https://github.com/dtcenter/MET/issues
 
+Version 9.1.3 release notes (20210319)
+--------------------------------------
+
+- See complete list of `GitHub Issues <https://github.com/NCAR/MET/milestone/71?closed=1>`_.
+- Fix PB2NC segfault when deriving PBL (`#1715 <http://github.com/dtcenter/MET/issues/1715>`_).
+- Fix parsing error for floating point percentile thresholds, like >SFP33.3 (`#1716 <http://github.com/dtcenter/MET/issues/1716>`_).
+
 Version 9.1.2 release notes (20210310)
-------------------------------------------------
+--------------------------------------
 
 - See complete list of `GitHub Issues <https://github.com/NCAR/MET/milestone/70?closed=1>`_.
 - Enable the MET documentation to be published via `Read the Docs <https://met.readthedocs.io/en/latest/>`_.
@@ -17,7 +24,7 @@ Version 9.1.2 release notes (20210310)
 - The 1D histograms reported by Grid-Diag are incorrect when reading the same variable name from multiple data sources (`#1694 <http://github.com/dtcenter/MET/issues/1694>`_).
 
 Version 9.1.1 release notes (20201118)
-------------------------------------------------
+--------------------------------------
 
 - See complete list of `GitHub Issues <https://github.com/NCAR/MET/milestone/68?closed=1>`_.
 - Fix Grid-Diag to correctly ignore missing data values (`#1562 <http://github.com/dtcenter/MET/issues/1562>`_).

met/docs/conf.py

@@ -20,11 +20,11 @@
 project = 'MET'
 author = 'UCAR/NCAR, NOAA, CSU/CIRA, and CU/CIRES'
 author_list = 'Brown, B., Bullock, R., Fowler, T., Halley Gotway, J., Newman, K., Jensen, T.'
-version = '9.1.2'
+version = '9.1.3'
 verinfo = version
 release = f'{version}'
 release_year = '2021'
-release_date = f'{release_year}0310'
+release_date = f'{release_year}0319'
 copyright = f'{release_year}, {author}'
 
 # -- General configuration ---------------------------------------------------

met/src/basic/vx_config/my_config_scanner.cc

@@ -169,6 +169,8 @@ static bool replace_env(ConcatString &);
 
 static bool is_fort_thresh_no_spaces();
 
+static bool is_simple_perc_thresh();
+
 static int  do_simple_perc_thresh();
 
 
@@ -370,6 +372,8 @@ if ( is_float_v2() )  { if ( do_float() )  return ( token(FLOAT) ); }
 
 if ( is_fort_thresh_no_spaces() )  { return ( do_fort_thresh() ); }
 
+if ( is_simple_perc_thresh() )  { return ( do_simple_perc_thresh() ); }
+
 int t;
 
 if ( is_id() )  { t = do_id();  return ( token(t) ); }
@@ -533,7 +537,6 @@ if ( is_lhs )  { strncpy(configlval.text, configtext, max_id_length);  return (
 
 if ( strcmp(configtext, "print"  ) == 0 )  { return ( PRINT ); }
 
-
    //
    //  boolean?
    //
@@ -554,17 +557,13 @@ for (j=0; j<n_fort_thresh_strings; ++j)  {
 
 }
 
-
-
-
-
    //
    //  builtin ?
    //
 
 int index;
 
- if ( (! is_lhs) && is_builtin((string)configtext, index) )  { configlval.index = index;  return ( BUILTIN ); }
+if ( (! is_lhs) && is_builtin((string)configtext, index) )  { configlval.index = index;  return ( BUILTIN ); }
 
    //
    //  local variable ?   //  ie, in argument list
@@ -580,11 +579,6 @@ const DictionaryEntry * e = dict_stack->lookup(configtext);
 
 if ( e && (e->is_number()) && (! is_lhs) )  {
 
-   // cout << "=================== id = \"" << configtext << "\"    is_lhs = " << (is_lhs ? "true" : "false") << "\n";
-
-   // cout << "do_id() -> \n";
-   // e->dump(cout);
-
    if ( e->type() == IntegerType )  {
 
       set_int(configlval.nval, e->i_value());
@@ -613,28 +607,20 @@ if ( e && (! is_lhs) && (e->type() == UserFunctionType) )  {
 
 }
 
-
     ///////////////////////////////////////////////////////////////////////
 
-
-
-
-
    //
    //  fortran threshold without spaces?  (example: "le150")
    //
 
-if ( (strncmp(configtext, "lt", 2) == 0) && is_number(configtext + 2, max_id_length - 2) )  { return ( do_fort_thresh() ); }
-
 for (j=0; j<n_fort_thresh_strings; ++j)  {
 
    if (    (strncmp(configtext, fort_thresh_string[j], 2) == 0)
-        && (is_number(configtext + 2, max_id_length - 2))
-        )  { configlval.cval = thresh_lt;  return ( do_fort_thresh() ); }
+        && (is_number(configtext + 2, max_id_length - 2))  )
+           { configlval.cval = thresh_lt;  return ( do_fort_thresh() ); }
 
 }
 
-
    //
    //  simple percentile threshold?  (example: "SOP50")
    //
@@ -649,11 +635,8 @@ for (j=0; j<n_perc_thresh_infos; ++j)  {
 
 }
 
-
-
     ///////////////////////////////////////////////////////////////////////
 
-
    //
    //  nope
    //
@@ -863,7 +846,6 @@ while ( 1 )  {
 
    c1 = c2;
 
-   // c2 = nextchar();
    c2 = fgetc(configin);
 
 }
@@ -891,7 +873,6 @@ while ( 1 )  {
 
    if ( feof (configin) )  break;
 
-   // c = nextchar();
    c = fgetc(configin);
 
    if ( (c == eof) || (c == '\n') )  break;
@@ -1435,7 +1416,38 @@ int j;
 
 for (j=0; j<n_fort_thresh_strings; ++j)  {
 
-   if ( (strncmp(configtext, fort_thresh_string[j], 2) == 0) && (is_number(configtext + 2, max_id_length - 2)) )    return ( true );
+   if (    (strncmp(configtext, fort_thresh_string[j], 2) == 0)
+        && (is_number(configtext + 2, max_id_length - 2))  )
+           { return ( true ); }
+
+}
+
+
+return ( false );
+
+}
+
+
+////////////////////////////////////////////////////////////////////////
+
+
+bool is_simple_perc_thresh()
+
+{
+
+int j, k;
+
+   //
+   //  simple percentile threshold?  (example: "SOP50.0")
+   //
+
+for (j=0; j<n_perc_thresh_infos; ++j)  {
+
+   k = perc_thresh_info[j].short_name_length;
+
+   if (    (strncmp(configtext, perc_thresh_info[j].short_name, k) == 0)
+        && (is_number(configtext + k, max_id_length - k))  )
+           { return ( do_simple_perc_thresh() ); }
 
 }
 
@@ -1472,7 +1484,6 @@ for (j=0; j<n_perc_thresh_infos; ++j)  {
 
 }
 
-
 if ( index < 0 )   {
 
    mlog << Error << "\ndo_simple_perc_thresh() -> "
@@ -1482,11 +1493,8 @@ if ( index < 0 )   {
 
 }
 
-
 configlval.pc_info.perc_index = index;
-// configlval.pc_info.is_simple  = true;
-configlval.pc_info.value     = value;
-// configlval.pc_info.value2     = bad_data_double;;
+configlval.pc_info.value      = value;
 
 
 return ( SIMPLE_PERC_THRESH );
@@ -1495,9 +1503,3 @@ return ( SIMPLE_PERC_THRESH );
 
 
 ////////////////////////////////////////////////////////////////////////
-
-
-
-
-
-

met/src/basic/vx_util/util_constants.h

@@ -18,6 +18,7 @@
 ////////////////////////////////////////////////////////////////////////
 
 // Released versions of MET
+static const char met_version_9_1_3[] = "V9.1.3";
 static const char met_version_9_1_2[] = "V9.1.2";
 static const char met_version_9_1_1[] = "V9.1.1";
 static const char met_version_9_1[]   = "V9.1";
@@ -40,7 +41,7 @@ static const char met_version_1_1[]   = "V1.1";
 
 ////////////////////////////////////////////////////////////////////////
 
-static const char * const met_version    = met_version_9_1_2;
+static const char * const met_version    = met_version_9_1_3;
 static const char default_met_data_dir[] = "MET_BASE";
 static const char txt_file_ext[]         = ".txt";
 static const char stat_file_ext[]        = ".stat";