Skip to content

Conversation

JohnHalleyGotway
Copy link
Collaborator

@JohnHalleyGotway JohnHalleyGotway commented Jan 22, 2021

Pull Request Testing

  • Describe testing already performed for these changes:

    Merging the latest version of develop into develop-ref. Here's the diffs from the NB20210122 regression test:
ERROR: folder MET-develop-ref/test_output missing 4 files
    climatology/stat_analysis_WMO_1.5DEG_FILTER_CDF_dump.stat 
    climatology/stat_analysis_WMO_1.5DEG_MPR_to_CTC_out.stat 
    plot_point_obs/nam_and_ndas.20120409.t12z.prepbufr_CONFIG_REGRID.ps 
    point2grid/CONUS_2500_1500_56_-56_-101360_128240.nc 
ERROR: folder MET-develop/test_output missing 2 files
    stat_analysis_WMO_1.5DEG_FILTER_CDF_dump.stat 
    stat_analysis_WMO_1.5DEG_MPR_to_CTC_out.stat

The stat_analysis output files were moved into the climatology sub-directory.
The CONUS_2500* temp files was moved into the point2grid sub-directory.
And the new plot_point_obs output file was added by PR #1628.
Updating the reference revision to get past these changes.

  • Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

    None really. Just confirm that organizing the unit test output area makes sense.

  • Do these changes include sufficient documentation and testing updates? [Yes]
    This PR serves as the documentation for updating the nightly build reference revision.

  • Will this PR result in changes to the test suite? [Yes]

    If yes, describe the new output and/or changes to the existing output:

    As described above.

Pull Request Checklist

See the METplus Workflow for details.

  • Complete the PR definition above.
  • Ensure the PR title matches the feature or bugfix branch name.
  • Define the PR metadata, as permissions allow.
    Select: Reviewer(s), Project(s), and Milestone
  • After submitting the PR, select Linked Issues with the original issue number.
  • After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
  • Close the linked issue and delete your feature or bugfix branch from GitHub.

JohnHalleyGotway and others added 16 commits January 11, 2021 09:34
…ces of the NULL pointer with the nul character (\0) instead.
* #1408 Added get_var_id

* #1408 Check variable name in the configuration to use the variable name instewad of grib code

* #1408 Added point2grid_ascii2nc_surfrad_DW_PSP_by_name
* #1580 Added get_grid_from_lat_lon_vars

* #1580 Added get_grid_from_lat_lon_vars and support 2D time variable

* #1580 Support int type variable without scale_factor and add_offset attributes

* #1580 Support 2D time variable. Implemented filtering by valid_time

* #1580 Bug fix: read time with dimension 0

* #1580 Support time variable with no dimension

* #1580 Initial release

* #1580 Added point2grid_2D_time

* #1580 Check project attribute for GOES

* #1580 Changed NULL to 0 to avoid co,pilation warning

* #1580 Added point2grid_2D_time

* #1580 Added "point2grid configuration file" section

* #1580 Changed to_grid for point2grid_NCCF_UK & point2grid_2D_time

Co-authored-by: Howard Soh <hsoh@kiowa.rap.ucar.edu>
Co-authored-by: John Halley Gotway <johnhg@kiowa.rap.ucar.edu>
* #1580 Correct the precision at _apply_scale_factor

* #1580 Added unit test plot_data_plane_NCCF_time

* #1580 Changed argument type to double at _apply_scale_factor(double)
Co-authored-by: Howard Soh <hsoh@kiowa.rap.ucar.edu>
…date the tool to do the requested regridding. Still need to update the docs.
* Per #1624, define OBS_COMMAND.

* Per #1624, unset the test-specific environment variables after completing the run.

* Per #1624, after PR #1625 merged these changes into develop, they caused 2 unexpected diffs in the NB output. These were caused by enviornment variables being unset after each test. Updating unit_netcdf.xml and unit_point2grid.xml to define more test-specific environment variables to reproduce previous NB output.
…iate directories rather than having them at the top-level directory.
…nt2grid subdirectory instead of the top-level test output directory.
Split the definition of H_RATE and POD
@JohnHalleyGotway JohnHalleyGotway added this to the MET 10.0.0 milestone Jan 22, 2021
@JohnHalleyGotway JohnHalleyGotway changed the title Update develop-ref after Update develop-ref after #1628 Jan 22, 2021
Copy link
Collaborator

@jprestop jprestop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The organizing the unit test output area makes sense to me. I reviewed the files and everything looked good.

@JohnHalleyGotway JohnHalleyGotway removed the request for review from georgemccabe January 22, 2021 18:50
@JohnHalleyGotway JohnHalleyGotway merged commit 3fa2b3a into develop-ref Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants