Feature #584 proofread - #588
Draft
JohnHalleyGotway wants to merge 12 commits into
Draft
JohnHalleyGotway wants to merge 12 commits into
JohnHalleyGotway wants to merge 12 commits into
Conversation
… linkcheck.yml, and update Sphinx to match METplus - Run dtcenter/metplus-action-linkcheck@v1 as a "Check links" step of documentation.yaml after the docs build, with the same inputs linkcheck.yml used, as was done for MET (dtcenter/MET#3457) and METplus (dtcenter/METplus#3359). The weekly schedule is dropped. - Add a workflow_dispatch trigger so the workflow can be run manually. - Update docs/requirements.txt to the versions METplus uses: sphinx 8.2.3, sphinx-design 0.6.1, sphinx-rtd-theme 3.0.1, and sphinx-gallery 0.19.0. - Use Python 3.12 for the Documentation workflow (was 3.10), since Sphinx 8.2 requires Python 3.11 or later. Read the Docs already uses 3.12. - Stop installing unpinned sphinx, sphinx-gallery, and sphinx_rtd_theme before docs/requirements.txt. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The sites.gsl.noaa.gov author page returns 503 Service Unavailable. This section will be removed before the next release, so no replacement link is needed. This matches the same change in MET (dtcenter/MET#3457) and METplus (dtcenter/METplus#3359). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ed-out examples dtcenter/metplus-action-linkcheck checks DOI links with the DOI API, so a DOI whose publisher blocks automated requests still passes, and a mistyped DOI fails. Shorten the comment for bmcnoldy.rsmas.miami.edu, which is still ignored. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Tabs in the .rst files are replaced with spaces, matching the spacing conventions used elsewhere in the docs: - Tabs used for indentation are replaced with the same number of spaces docutils already used for them (tab stops every 8 columns), so the rendered docs are unchanged. - Lines containing only whitespace are now empty, and trailing tabs are removed. - Tabs between words in a sentence, or after a directive name, are replaced with a single space. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…tently Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…structions - "has has" -> "has" - "verbosity of logging than what is set" -> "from what is set" - "Replace <placeholder> to <directory>" -> "with <directory>" - add the missing closing quote after "human-friendly data serialization language" - "i.e" -> "i.e.", "permissions.The" -> "permissions. The" - "is base directory" -> "is the base directory" - "where the author has" -> "where the user has", "the the" -> "the" Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…on individual pages Spelling (dependencies, specified, usually, latitude, acronym, labeled), missing or extra words, punctuation, wrong words left over from other plot pages (e.g. "line plot" on the bar, box, and ensemble spread-skill pages and "performance diagram" on the wind rose page), and organization names (Madden-Julian, Cooperative Institute for Research in the Atmosphere, Naval Research Lab, Joint Technology Transfer Initiative). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ted phrase - "Add User Documenation" -> "Add User Documentation" (title underline lengthened to match) - remove the repeated "for more information" in the pytest sentence Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- "in come cases" -> "in some cases", "were also" -> "was also" - "Predication"/"Predicatio" -> "Prediction" in the acronym list - "Earth System's"/"Earth Systems Research Laboratory" -> "Earth System Research Laboratory" - "METdatadb" -> "METdataio" (the repository's current name), "-System" -> "- System" Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…t instructions The plot pages named config files, data files, scripts, and directories that don't exist in the repository, mostly copied from other plot pages or older names. Use the names of the actual files (for example eclv_defaults.yaml, minimal_box.yaml, custom_line.yaml, minimal_line.yaml, reformatted_data_for_scatter.data, plot_dlwr_sample.data, minimal_taylor_diagram.yaml, minimal_performance_diagram.yaml, wind_rose_defaults.yaml, wind_rose.py, wind_rose_custom.yaml, reliability_defaults.yaml, minimal_reliability.yaml, prob_hist.yaml, rel_hist.yaml), the .points1 file, and the dump_points_2 setting. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- List the log levels from most to least verbose: DEBUG, INFO, WARNING, ERROR. - "number of colors", not columns, must match the number of series. - Fix the METplotpy_source_code path, and set METCALCPY_BASE (not METPLOTPY_BASE) to the METcalcpy source, with valid csh setenv syntax. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ading - stratosphere_plots.rst: fix the script name, and describe obar as the observed mean plotted as contour lines (the text was copied from the weather regime page). - difficulty_index.rst: the link goes to a METplus use case, not the METviewer documentation. - make_maki_enso.rst: link the MJO_ENSO use case, which was missing. - mjo_rmm_omi.rst: compute_mjo_indices.py is in METcalcpy. - references.rst: remove the Hoaglin et al. DOI, which resolves to a 1984 review of the book rather than the book. - index.rst: complete the NCAR sentence, as in the other components. - release-notes.rst: fix "histogramm" to match the title of #558. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to #584
Summary
Proofreads the METplotpy documentation, and corrects file names, config settings, setup steps, links, and descriptions found while reading every page.
This branch is based on
feature_584_linkcheck(#587). Until #587 is merged, this PR also shows its 4 commits (the link check workflow, Sphinx update, and spacing changes). After #587 is merged, only the 8 commits for this PR remain: 36 files changed.Changes
Proofreading
File and Config Names in the Plot Instructions
The instructions named files that don't exist in the repository. They now use the actual names:
eclv_defaults.yaml,./eclv.data.minimal_box.yaml,custom_line.yaml,minimal_line.yaml(the files used in the commands).$WORKING_DIR/custom_scatter.yaml(where the page copies it), the data filereformatted_data_for_scatter.data, and the output setting incustom_scatter.yaml.$METPLOTPY_BASE/test/taylor_diagram, the data fileplot_dlwr_sample.data(all 5 references), andminimal_taylor_diagram.yaml.minimal_performance_diagram.yaml, and the.points1file.test/wind_rosedirectory,wind_rose_defaults.yaml,wind_rose.py,minimal_wind_rose.yaml, andwind_rose_custom.yaml(the page saidcustom_performance_diagram.yaml).reliability_defaults.yamlandminimal_reliability.yaml(the page saidminimal_box.yaml).rank_hist.yaml,prob_hist.yaml, andrel_hist.yaml(the page listedrank_hist.yamlthree times), anddump_points_2.stratosphere_plots.py(was.py.py).TCMPR Setup
/path/to/METplotpy_source_codepath, and the METcalcpy step now setsMETCALCPY_BASEto the METcalcpy source (it setMETPLOTPY_BASEto METplotpy), with valid cshsetenvsyntax.Descriptions, Links, and References
stratosphere_plots.rst:obaris described as the observed mean plotted as contour lines over the bias. The previous text ("frequency of occurrence of each cluster") was copied from the weather regime page.difficulty_index.rst: the link text said "METviewer documentation" but links to a METplus use case.make_maki_enso.rst: the sentence pointing to the MJO_ENSO use case ended with ":" and no link; the use case is now linked.mjo_rmm_omi.rst:compute_mjo_indices.pyis in METcalcpy (metcalcpy/contributed/rmm_omi), not METplotpy.references.rst: removed the Hoaglin et al. (1983) DOI, which resolves to a 1984 review of the book in The Statistician, not the book.index.rst: completed the NCAR sentence ("NCAR is sponsored by NSF"), as in the other components.release-notes.rst: "histogramm" → "histogram", matching the title of Remove plotly: Update bar, box, and histogram plot #558.Pull Request Testing
Describe testing already performed for these changes:
Full HTML builds of the docs (including the sphinx-gallery example) with Sphinx 8.2.3 in nitpicky mode (
-n), before and after these changes, both have no warnings, and the content of exactly the edited pages changed. Each corrected file, directory, config, and script name was checked against the files in the repository and the code (for example,histogram_2d.pyreadsdump_points_2).Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Review the diff against
feature_584_linkcheck(or againstdevelopafter Feature #584 linkcheck #587 is merged), especially the file names in the plot instructions and the TCMPR setup steps.Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes]
Do these changes include sufficient testing updates? [Yes]
Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:
Do these changes introduce new SonarQube findings? [No]
If yes, please describe:
Please complete this pull request review by [Fill in date].
Pull Request Checklist
See the METplus Workflow for details.
Select: Reviewer(s) and Development issue
Select: Milestone as the version that will include these changes
Select: Coordinated METplus-X.Y Support project for bugfix releases or METplotpy-X.Y.Z Development project for official releases
🤖 Generated with Claude Code