Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Develop-ref after #1908 and #1914 #1921

Merged
merged 645 commits into from
Nov 7, 2022
Merged

Update Develop-ref after #1908 and #1914 #1921

merged 645 commits into from
Nov 7, 2022

Conversation

georgemccabe
Copy link
Collaborator

PRs #1908 and #1914 added new use cases.

georgemccabe and others added 30 commits March 7, 2022 20:00
…can catch if changes to external repositories caused differences in the output
…ff logic is already running when external repository triggers workflow, which is what we need to test with the diff logic
…al path for all symbolic links for consistency, added usage statement
…sts at all -- previous logic only errors if directory is not empty
…a_init/etc.) to use %H instead of %2H. We discovered that Python 3.6.8 interprets %2H as the same as %H (correctly substituting the hour), however Python 3.8.8 interprets %2H as 2H. The number in between the % and letter is not supported by strftime so it is not valid for these datetime values. The logic to zero-pad using this notation was written in the METplus string substitution logic and only applies to lead and level template tags. ci-run-all-diff
Co-authored-by: Lisa Goodrich <[email protected]>
Co-authored-by: Julie Prestopnik <[email protected]>
jprestop and others added 28 commits October 5, 2022 13:29
* Per #1847, moved release notes to be its own chapter, reformatted component release notes, and added new section for upgrade instructions

* Per #1847, added release-notes internal tag

* Per #1847, fix error and reformat link

* Per #1847, modified release instructions to add a section about upgrade instructions

* Per #1847, modified release instructions to add a section about upgrade instructions

* Per #1847, resolve doc warnings

* Per #1847, updated instructions for clarity.

* Per #1847, modified text and link.

* Per #1847, modified wording and resolve error with note
* Issue #1852 initial work for the Planview plot

* Issue #1852 planview plot for physics tendency

* issue #1852 cleared away unused/unneccessary code

* Issue #1852 update script command

* Issue #1852 first draft, doesn't have correct content

* Issue #1852 updated name of output plot to include short_range in name

* Issue #1852 renamed to include fcstOnly label

* Issue #1852 initial commit

* Issue #1852

* Issue #1852

* Issue #1852 initial commit

* issue #1852 vertical cross section

* Issue 1852 clean up unused lines

* Issue #1852 initial commit

* Issue #1852

* Issue #1852 updates

* Issue #1852 fixed documentation warnings, but some problems remain

* fixing line length for chapter and section headers and one letter capitalization

* cleaning up line spacing and trying to fix a critical error.

* trying to fix critical error again

* Issue #1852 more description for data and dependency on METplotpy with additional instructions

* Issue #1852 update location of input data to single directory

* Removing duplicate files with a misformatted name that are not needed

* Per #1852, making changes to formatting to eliminate errors

* Issue #1852 remove redundant yaml config file

* Issue #1852 update PYTHONPATH information

* Issue #1852 fixed conflicts

* Issue #1852 update path to shapefile

* Issue #1852 fixed copy and paste error

* Issue #1852 add three use case entries: Planview, VerticalProfile, and VerticalCrossSection for the short range Physics tendency use case

* Issue #1852 add latest short range tests to top, move down list when everything passes

* Update use_case_groups.json

move short_range use cases #10-12 to below the short range test for #9

* Issue #1852 clean up conf files by providing variables for the input netCDF files and for the planview plot, allow user to indicate pressure levels

* Issue #1852 Indicate external Python packages

* Issue #1852 added external dependency section

* removed env vars that don't need to be set -- METPLOTPY_BASE is set by the automated test conf (.github/parm/test_settings.conf) and PYTHONPATH doesn't need to be set because METplotpy is installed in Python environment for use case tests

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_Planview.py

Fixed one typo

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_VerticalCrossSection.py

Fixed a typo

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_VerticalProfile.py

Same typo, same fix

* Issue #1852 remove extraneous entry for USER_SCRIPT_COMMAND, a hold-over from the original implementation which doesn't allow the user to easily specify the FV3 history file and grid spec netCDF file

* Update UserScript_fcstFV3_fcstOnly_PhysicsTendency_VerticalProfile.conf

Removed the incorrect USER_SCRIPT_COMMAND in previous commit.  Reinstated the correct version and removed the incorrect command.

* update html file from Planview to VerticalProfile

Co-authored-by: Lisa Goodrich <[email protected]>
Co-authored-by: Julie Prestopnik <[email protected]>
Co-authored-by: George McCabe <[email protected]>
Co-authored-by: Hank Fisher <[email protected]>
@georgemccabe georgemccabe added this to the METplus-5.0.0 milestone Nov 7, 2022
@georgemccabe georgemccabe merged commit b468467 into develop-ref Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

10 participants