ufs-community / ufs-srweather-app

UFS Short-Range Weather Application
Other
53 stars 114 forks source link

[production/AQM.v7] Merging all the changes to address EE2 compliance issues to support AQMv7.0 implementation #1072

Closed JianpingHuang-NOAA closed 2 months ago

JianpingHuang-NOAA commented 3 months ago

DESCRIPTION OF CHANGES:

1) addressed the issue related to aqmv7 grib2 product files and ensure aqmv7 product file be consistent with production (aqmv6) files

Type of change

TESTS CONDUCTED:

CHECKLIST

LABELS (optional):

A Code Manager needs to add the following labels to this PR:

CONTRIBUTORS (optional):

BrianCurtis-NOAA commented 3 months ago

@JianpingHuang-NOAA Looks like there are merge conflicts you need to address in the files listed in this PR.

MichaelLueken commented 2 months ago

@BrianCurtis-NOAA and @JianpingHuang-NOAA -

Also, to silence the inability to build the RTD for the production/AQM.v7 branch, if you go into the .readthedocs.yaml and change the two instances of docs/UsersGuide/* to doc/UsersGuide/*, the test should no longer fail. It is constantly failing because the expected location, docs/UsersGuide/*, doesn't exist in the branch.

BrianCurtis-NOAA commented 2 months ago

Closing for #1075