Skip to content

Conversation

JohnHalleyGotway
Copy link
Collaborator

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 NB20210114 regression test:
ERROR: folder MET-develop-ref/test_output missing 4 files
    plot_data_plane/plot_data_plane_NCCF_time.ps 
    point2grid/point2grid_2D_time_west_bering_sea.nc 
    point2grid/point2grid_NCCF_UKMet.nc 
    point2grid/surfrad_to_G212_by_name.nc 
ERROR: folder MET-develop/test_output missing 1 files
    point2grid/point2grid_NCCF_TO_G231.nc 

These new/modified outputs are all expected changes from recent PR's.

  • Recommend testing for the reviewer to perform, including the location of input datasets:

    None needed.

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

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

    1 new plot_data_plane and 3 new point2grid outputs.

  • After merging, should the reviewer DELETE the feature branch from GitHub? [No]

    Do NOT delete the develop branch.

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.

JohnHalleyGotway and others added 5 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)
@JohnHalleyGotway JohnHalleyGotway added this to the MET 10.0.0 milestone Jan 14, 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.

As noted in this pull request, there is no testing for the reviewer to perform. This is a straightforward merge of the latest version of develop into develop-ref.

@jprestop jprestop merged commit 5178055 into develop-ref Jan 14, 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.

3 participants