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

Final updates #62

Merged
merged 15 commits into from
Jun 29, 2022
Merged

Final updates #62

merged 15 commits into from
Jun 29, 2022

Conversation

mkavulich
Copy link
Collaborator

@mkavulich mkavulich commented Jun 28, 2022

A (hopefully) final set of updates prior to the v6 release:

  • Add section on new variable tracking tool
  • Update links for v6 release (SCM UG link is still broken, but should be live soon)
  • Remove mentions of MRW/Global for this release
  • Some formatting and spelling fixes
  • Update Acronyms and Glossary, clean up links to Glossary
  • Remove mentions of optional arguments

You can view the latest version of this branch's proposed changes in ReadTheDocs here: https://mike-k-tech-doc.readthedocs.io/en/latest/index.html

I am still in the process of cleaning up the Glossary references, but I shouldn't make any more substantial changes to this PR.

Copy link
Collaborator

@ligiabernardet ligiabernardet left a comment

Choose a reason for hiding this comment

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

Thank you another round of informative updates.

CCPPtechnical/source/ConfigBuildOptions.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/ParamSpecificOutput.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/CCPPPreBuild.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/CCPPPreBuild.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/CCPPPreBuild.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/CCPPPreBuild.rst Show resolved Hide resolved
CCPPtechnical/source/Glossary.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/AutoGenPhysCaps.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/AutoGenPhysCaps.rst Outdated Show resolved Hide resolved
CCPPtechnical/source/CCPPDebug.rst Outdated Show resolved Hide resolved
@mkavulich mkavulich mentioned this pull request Jun 28, 2022
16 tasks
@mkavulich
Copy link
Collaborator Author

@ligiabernardet Thank you for your thorough review and all the comments and corrections! I believe I have addressed all of your comments, let me know if you have anything else.

@ligiabernardet
Copy link
Collaborator

I believe the ReadTheDocs has been updated. In that case, there is a broken link from https://mike-k-tech-doc.readthedocs.io/en/latest/ConfigBuildOptions.html?highlight=ufs%20wm#ccpp-configuration-and-build-options to the UFS WM UG. This is related to EPIC changing the branching/tagging/releasing naming convention.

@ligiabernardet
Copy link
Collaborator

ligiabernardet commented Jun 29, 2022

Same broken link from https://mike-k-tech-doc.readthedocs.io/en/latest/ConfigBuildOptions.html?highlight=guide#ccpp-configuration-and-build-options to https://ufs-weather-model.readthedocs.io/en/ufs-v3.0.0/BuildingAndRunning.html#building-the-weather-model

Copy link
Collaborator

@ligiabernardet ligiabernardet left a comment

Choose a reason for hiding this comment

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

Almost there. Just a couple of broken links.

@ligiabernardet
Copy link
Collaborator

Refreshed on my end and links work now.

@mkavulich mkavulich merged commit f73fc3a into NCAR:main Jun 29, 2022
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.

None yet

2 participants