…f the set_attrs dictionary config entries. Still need to add code to parse those entries and update the tests and documentation.

…dictionary members.

…units to the OBS_UNITS output column instead of writing a constant NA string.

…d spec parsed from the file. Also, update the process_data_plane() function to handle updating the metadata and grid defintions.

…inking again.

…ear.

…le, parse them as set_attrs.set_name instead of set_name from the set_attrs dictionary. That makes setting up the config file more flexible.

…evel_attr, and units_attr instead of name(), level_name(), and units(). This change should be made everywhere NetCDF output files are written.

…in name, level, and units.

…rs functionality.

…ttrs dictionary.

…nd units() with calls to name_attr(), level_attr(), and units_attr().

…nits() with calls to name_attr(), level_attr(), and units_attr().

… units() with calls to name_attr(), level_attr(), and units_attr().

…eplace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

…el_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

…d units() with calls to name_attr(), level_attr(), and units_attr().

…). The existing changes included *'s and don't actually match the name of the NetCDF variable.

…). The existing changes included *'s and don't actually match the name of the NetCDF variable. (#1389)

…rning about setting a header column to a null string.

…time issues I found on dakota.

…ng all of these metadata modifiers into a set_attrs dictionary, I'm now parsing them all as individual entries. See #1020 issue comments for more details.

…e user-specified long_name string in the NetCDF matched pairs file from Grid-Stat.

* Per #1020, update the VarInfo class heirarchy to store the contents of the set_attrs dictionary config entries. Still need to add code to parse those entries and update the tests and documentation.

* Per #1020, update the VarInfo class to parse and store the set_attrs dictionary members.

* Per #1020, update point_stat and ensemble_stat to write the obs_info units to the OBS_UNITS output column instead of writing a constant NA string.

* Per #1020, adds attr access functions to the VarInfo class. VarInfo::name_attr() for example returns SetAttrsName, if set, and simply Name otherwise.

* Per #1020, this is work in progress. Swapping out calls to shc set_fcst_var, set_obs_var, set_fcst_level, set_obs_level, set_fcst_units, and set_obs_units with calls to the corresponding attrs access function. Still many more updates to the code required.

* Per #1020, add support for set_attrs.set_accum to override the accumulation interval read from the data.

* Per #1020, add a set_attrs() utility function to update the metadata in a DataPlane object using the contents of a set_attr Dictionary stored in a VarInfo object.

* Per #1020, add grid.nxy() utility function to return Nx*Ny instead of always having to type out grid.nx()*grid.ny().

* Per #1020, switch VarInfo::SetAttrsGrid from a ConcatString to a Grid object. And parse that grid in the set_dict() function. The huge drawback here is that the grid may be specified as a named grid or using a grid specification string... but it cannot be defined as the path to a gridded data file. The library dependency logic is just too complex, and I'd need to move a lot of code around to make this work. For now, just leave that feature out.

* No real change, just adding a blank line that was missing.

* Per #1020, add Met2dDataFile::set_grid() function to override the grid spec parsed from the file. Also, update the process_data_plane() function to handle updating the metadata and grid defintions.

* Per #1020, had to add -lvx_color to 5 Makefile.am files to get them linking again.

* Per #1020, work on log messages to make the set_attrs parsing more clear.

* Per #1020, update logic for how to parse set_attrs entries. For example, parse them as set_attrs.set_name instead of set_name from the set_attrs dictionary. That makes setting up the config file more flexible.

* Per #1020, update Grid-Stat to write NetCDF output using name_attr, level_attr, and units_attr instead of name(), level_name(), and units(). This change should be made everywhere NetCDF output files are written.

* Per #1020, update the parsing logic to check for embedded whitespace in name, level, and units.

* Per #1020, removed unused SetAttrsEnsemble option.

* Per #1020, add new test in unit_grid_stat.xml to exercise the set_attrs functionality.

* Per #1020, update data/config/README with information about the set_attrs dictionary.

* Per #1020, in Ensemble-Stat, replace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

* Per #1020, in Grid-Diag, replace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

* Deleting stale, commented out, development code.

* Per #1020, in MODE and MTD replace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

* Per #1020, in PCP-Combine, Shift-Data-Plane, and Regrid-Data-Plane, replace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

* Per #1020, in TCRMW and Series-Analysis, replace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

* Per #1020, in Wavelet-Stat, replace calls to name(), level_name(), and units() with calls to name_attr(), level_attr(), and units_attr().

* src/tools/core/wavelet_stat/wavelet_stat.cc

* src/tools/tc_utils/tc_rmw/tc_rmw.cc

* Adding a couple of spaces to unit_grid_stat.xml to be safe.

* Per #1020, update call to set_obs_units() in point-stat to avoid a warning about setting a header column to a null string.

* Per #1020, changing the order of parsing to see if this fixes the runtime issues I found on dakota.

* Per #1020, so this represents a course correction. Rather than grouping all of these metadata modifiers into a set_attrs dictionary, I'm now parsing them all as individual entries. See #1020 issue comments for more details.

* Per #1020, respond to PR comments by updating Grid-Stat to include the user-specified long_name string in the NetCDF matched pairs file from Grid-Stat.

Co-authored-by: John Halley Gotway <johnhg@kiowa.rap.ucar.edu>