-
Notifications
You must be signed in to change notification settings - Fork 49
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
Moving documentation to ReadTheDocs #438
Moving documentation to ReadTheDocs #438
Conversation
…remove title page.
…CCPP tech doc), fix deprecated feature in conf.py
…atest changes or building with spack-stack; that will come in a subsequent PR
@mkavulich this looks really great! I checked out the branch to add documentation for the single precision build and it was easy to use and worked well. A few comments:
Anyway, these are just small things, overall it worked very well for me! |
@scrasmussen Thanks for the review! I have updated the Makefile, though I am unable to test the latex/pdf rule because I can not install latexmk on my machine (which is part of why this whole conversation of moving to ReadTheDocs started!). I also added a README.md file to assist people who are interested in building the HTML or PDF documentation. With regards to renaming documents/chapters, I kept the names from the original |
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.
Everything builds and 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.
Looks good - just a handful of things I noticed in terms of formatting.
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.
@hertneky Thank you for your thorough review! I believe I have addressed all of your comments.
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.
Looks good now. Thanks Mike
I didn't review in detail, although, in general, it looks great. Although I love LaTeX, this is a better way to format software docs in pretty much every way. Thanks for doing this. |
* upstream/main: Add GP SDFs/NMLs to SCM minor fix/cleanup Add namelist option and sigmab (tracer) for HR3 add SCM_RRFS_v1 SDF, nml, tracer file; update suite_info; update scm_type_defs to recgonize new tracers add script to download and stage climatological aerosol data and fix typo in HR3 namelist merge in changes from NCAR#428 from @hertneky add CITATION.cff file from @scrasmussen from NCAR#458 merge changes from @hertneky in NCAR#446 add HR3 suite and physics namelist from NCAR#445 from @bluefinweiwei changes from @bluefinweiwei for PR#419 Revert submodules Update physics Address reviewers comments update ccpp/physics submodule pointer Moving documentation to ReadTheDocs (NCAR#438) Synced physics update ccpp/physics submodule update ccpp/framework update CCPP_typedefs to work with PR#181 update ccpp/physics and .gitmodules Added step to skip gridpoint in ensemble if error during case generation Added utility script to create input file containing replay points Omission from previous commit Add capability to use input file with longitude and latitude lists update ccpp/physics submodule update .gitmodules for whitespace change removal update ccpp/physics and .gitmodules update ccpp/physics and .gitmodules update ccpp/physics submodule pointer point .gitmodules back to my branch for CI testing update ccpp/physics and .gitmodules update ccpp/physics and .gitmodules update ccpp/physics and .gitmodules Cleanup/Housekeeping. Remove unnecessary interpolation step Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Revert change to CMakeLists Revert change to CMakeLists Revert change to CMakeLists Revert change to CMakeLists Update CI Update CI Add file for Nvidia RTS Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update cmakelists Updated CI Update CI Update CI Update CI Update CI Update CMakeLists for Nvidia support Update CMakeLists for Nvidia support Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Update CI Foundatio for Nvidia CI script Synced physics Small cahnge from physics needed for replay Replay with RUC and NoahMP LSM working Some changes for Man to test If using model initial conditions and forcing from the UFS (e.g. Replay mode) with an active LSM, the SCM performs a cold-start initialization for the LSM. That is, UFS replay mode currently support cases generated from UFS Restart files (e.g spun up (warm) LSM parameters) update ccpp/physics and revert .gitmodules update ccpp/physics submodule and revert .gitmodules changes to work with ufs/dev PR#155 update for ufs/dev PR#94 update ccpp/physics and .gitmodules update ccpp/physics and .gitmodules after testing
This PR removes the old, LaTeX-based users guide in favor of RST files that are buildable with Sphinx. These new docs have been configured to automatically build with each commit to
main
on ReadTheDocs, and can be viewed here: html | PDFFor limitations in the new docs compared to the previous LaTeX/PDF, see this comment
NOTE: The original LaTeX files were auto-converted from
.tex
to.rst
using thepandoc
utility, and then manually updated for format. The conversion was not perfect: in particular, most section/chapter links were broken, and allcode-case text
was omitted from the converted docs. These problems needed to be fixed manually, but it's entirely possible some of these may have slipped into the final product.Resolves #418