Skip to content

Updates to RTD documentation based on full review#1287

Merged
KateFriedman-NOAA merged 33 commits into
NOAA-EMC:developfrom
KateFriedman-NOAA:feature/update-rtd
Jan 31, 2023
Merged

Updates to RTD documentation based on full review#1287
KateFriedman-NOAA merged 33 commits into
NOAA-EMC:developfrom
KateFriedman-NOAA:feature/update-rtd

Conversation

@KateFriedman-NOAA
Copy link
Copy Markdown
Contributor

@KateFriedman-NOAA KateFriedman-NOAA commented Jan 27, 2023

Description

This PR updates the new RTD documentation that was moved from the global-workflow wiki. The wiki (and now RTD) needed to be reviewed for updates related to recent GFS/global-workflow system updates, as well as fix some spelling/syntax issues. This is the first documentation update since the new RTD documentation was added to the repo.

As part of the updates the following changes occurred:

  • A few spelling and/or Sphinx syntax corrections.
  • Introduce use of note and warning boxes.
  • Review and update of content to match GFS system updates.
  • Reorganize/rename some sections.

Preview of incoming updated documentation can be viewed here on RZDM:

https://www.emc.ncep.noaa.gov/gmb/emc.glopara/read-the-docs/global-workflow/issue1272/index.html

Note: Pages will look a bit different (e.g. missing bullets), likely due to no Sphinx installation on RZDM (?). Expect other similar subtle differences compared to live RTD documentation page. I'm open to suggestions to make this preview look more like the live pages (e.g. install Sphinx?).

Type of change

Maintenance and regular updates to documentation.

How Has This Been Tested?

Viewed feature branch Git Action artifact generated html pages in browser.

Resolves #1272

KateFriedman-NOAA and others added 27 commits January 23, 2023 13:38
- Reviewed documentation to correct spelling mistakes
and update contents for current state of system.
- Introduced a couple tables where needed.
- More updates to come.

Refs NOAA-EMC#1272
- Replace "Note" comments with ".. note::" blocks.

Refs NOAA-EMC#1272
- Change table syntax to sphinx syntax.
- Remove extra line in index.rst.

Refs NOAA-EMC#1272
- Update to use header row in tables.
- Update configuration table contents.
- Update example output from setup commands.
- Update warning text for crontab command.

Refs NOAA-EMC#1272
- Update setup script name in configure.rst.
- Update python missing error text in errors_faq.rst.

Refs NOAA-EMC#1272
@KateFriedman-NOAA KateFriedman-NOAA added the maintenance Regular updates and maintenance work label Jan 27, 2023
@KateFriedman-NOAA KateFriedman-NOAA self-assigned this Jan 27, 2023
Comment thread docs/source/clone.rst
Comment thread docs/source/configure.rst Outdated
Comment thread docs/source/monitor_rocoto.rst
KateFriedman-NOAA and others added 5 commits January 31, 2023 10:15
- Move ssh method above https and mention that ssh method
is highly preferred.

Refs NOAA-EMC#1272
- Since DO_WAVE is controlled by the APP switch don't need
to mention DO_WAVE in the switch configuration table.

Refs NOAA-EMC#1272
- Note 3 states that TERM=xterm is the only supported TERM.
- Include bash example of shell function to automatically
switch if user is in different terminal.

Refs NOAA-EMC#1272
@KateFriedman-NOAA
Copy link
Copy Markdown
Contributor Author

@KateFriedman-NOAA KateFriedman-NOAA merged commit 3cbca7d into NOAA-EMC:develop Jan 31, 2023
@KateFriedman-NOAA KateFriedman-NOAA deleted the feature/update-rtd branch January 31, 2023 20:27
bbakernoaa pushed a commit to bbakernoaa/global-workflow that referenced this pull request Mar 19, 2026
… test (was NOAA-EMC#1336) (NOAA-EMC#1287)

* * Update CDEPS
* Update the following tests:
  datm_cdeps_bulk_gefs
  datm_cdeps_control_gefs
  datm_cdeps_control_gefs
  datm_cdeps_iau_gefs
  datm_cdeps_mx025_gefs
  datm_cdeps_stochy_gefs

* Add coupled c48-5deg ocean+ice tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Regular updates and maintenance work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review and update new RTD documentation

2 participants