Per #2250, update logic in get_att_accum_time() function. If no accum…

… time attribute is present, report the value as 0 instead of bad data.
dtcenter · Sep 1, 2022 · e49e7ce · e49e7ce
1 parent 71c1939
commit e49e7ce
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/libcode/vx_data2d_nc_met/met_file.cc b/src/libcode/vx_data2d_nc_met/met_file.cc
@@ -212,11 +212,11 @@ for (j=0; j<Ndims; ++j)  {
       //
       //  parse the variable attributes
       //
-      get_att_name     ( Var[j],     Var[j].name_att      );
-      get_var_long_name( Var[j].var, Var[j].long_name_att );
-      get_att_level    ( Var[j],     Var[j].level_att     );
-      get_var_units    ( Var[j].var, Var[j].units_att     );
-      get_att_accum_time( Var[j],    Var[j].AccumTime     );
+      get_att_name      ( Var[j],     Var[j].name_att      );
+      get_var_long_name ( Var[j].var, Var[j].long_name_att );
+      get_att_level     ( Var[j],     Var[j].level_att     );
+      get_var_units     ( Var[j].var, Var[j].units_att     );
+      get_att_accum_time( Var[j],     Var[j].AccumTime     );
 
       get_att_unixtime( Var[j], init_time_ut_att_name,  ill);
       get_att_unixtime( Var[j], valid_time_ut_att_name, vll);

diff --git a/src/libcode/vx_nc_util/nc_var_info.cc b/src/libcode/vx_nc_util/nc_var_info.cc
@@ -394,7 +394,9 @@ bool get_att_int(const NcVarInfo &info, const ConcatString att_name,
 ////////////////////////////////////////////////////////////////////////
 
 bool get_att_accum_time(const NcVarInfo &info, int &att_value) {
-   return get_att_int(info, accum_time_sec_att_name, att_value);
+   bool status = get_att_int(info, accum_time_sec_att_name, att_value);
+   if (is_bad_data(att_value))  att_value = 0;
+   return status;
 }
 
 ////////////////////////////////////////////////////////////////////////