-
Notifications
You must be signed in to change notification settings - Fork 119
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
[develop]: Add sample verification case w/instructions #500
[develop]: Add sample verification case w/instructions #500
Conversation
This reverts commit b839658.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gspetro-NOAA Excellent work! I have one minor comment regarding the section under Load the Regional Workflow
, but otherwise, everything looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gspetro-NOAA Thanks! Now approving this PR.
@EdwardSnyder-NOAA Just wanted to check base and make sure that you are happy with the changes that @gspetro-NOAA has made in this PR. Please let @gspetro-NOAA know if there are any other changes that you would like for her to make. Thanks! |
LGTM! @MichaelLueken @gspetro-NOAA |
DESCRIPTION OF CHANGES:
This PR includes a new chapter in the documentation for VX sample cases. Users can run through the case(s) to (1) learn to use the basic verification features and (2) compare any forecast improvements/developments with a baseline SRW forecast to evaluate the extent to which the contribution improves forecasting capabilities. Only one sample case is included in this PR, but others will be added subsequently.
Type of change
TESTS CONDUCTED:
None required. Built the docs for this branch of my fork on RTD: https://srw-ug.readthedocs.io/en/text-vxcases/ .
DEPENDENCIES:
N/A
DOCUMENTATION:
All documentation.
ISSUE:
Issue #499
CHECKLIST
CONTRIBUTORS (optional):
@natalie-perlin @EdwardSnyder-NOAA