Skip to content

Pr 864 doc updates#1

Merged
gspetro-NOAA merged 5 commits into
gspetro-NOAA:text/ug-updatesfrom
mkavulich:PR_864_doc_updates
Jul 20, 2023
Merged

Pr 864 doc updates#1
gspetro-NOAA merged 5 commits into
gspetro-NOAA:text/ug-updatesfrom
mkavulich:PR_864_doc_updates

Conversation

@mkavulich

Copy link
Copy Markdown

Hi Gillian, since you're working on major updates I thought it was best to make my changes for PR 864 on top of your branch. I hope this isn't too disruptive; let me know if you have any questions or if I can help out with anything!

Updates for ufs-community#864:

  • Remove mentions of turning off task_get_obs_* tasks
  • Re-work the section with the table describing the verification tasks; this includes some clarifications and corrections around which tasks are activated when
  • Documents new behavior for *_OBS_DIR variables and task_get_obs_* tasks
  • Remove mention of non-existent "OBS_DIR" variable

RTD for my branch can be found here: https://ufs-srweather-app-mkavulich.readthedocs.io/en/latest/index.html

@mkavulich mkavulich requested a review from gspetro-NOAA as a code owner July 19, 2023 05:50

@gspetro-NOAA gspetro-NOAA left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mkavulich Looks good to me! Do you have any additional changes you'd like to make, or can I merge it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants