-
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 SRW-SD chapter to SRW App docs #1128
[develop]: Add SRW-SD chapter to SRW App docs #1128
Conversation
…her-app into text/rrfs-sd
…s-sd' into text/rrfs-sd-main_aqm-edits
text: update RRFS-SD documentation to align with the use of the main_aqm branch
Co-authored-by: BenKoziol-NOAA <bkoziol@redlineperf.com>
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.
These changes look good to me! I was wondering whether the word sample should be changed to simple (as noted in my review), but that's the only comment I have.
Co-authored-by: BenKoziol-NOAA <bkoziol@redlineperf.com>
Co-authored-by: Michael Lueken <63728921+MichaelLueken@users.noreply.github.com>
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.
Thanks, @gspetro-NOAA!
DESCRIPTION OF CHANGES:
Adds documentation for the RRFS-SD features added to PR #1127 .
Type of change
TESTS CONDUCTED:
None required. Documentation builds on RTD (see check below). Ran the workflow on Hera using the instructions in these docs.
DEPENDENCIES:
PR #1127
DOCUMENTATION:
All documentation.
ISSUE:
Issue #1126
CHECKLIST
CONTRIBUTORS (optional):
@chan-hoo