Skip to content

Feature #584 proofread - #588

Draft
JohnHalleyGotway wants to merge 8 commits into
developfrom
feature_584_proofread
Draft

JohnHalleyGotway wants to merge 8 commits into
developfrom
feature_584_proofread

Conversation

@JohnHalleyGotway

@JohnHalleyGotway JohnHalleyGotway commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

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 develop, so it can be reviewed in parallel with the link check PR, #587. It has 8 commits and changes 36 files. The two PRs don't touch the same lines, so they merge cleanly in either order.

Changes

Proofreading

  • Spelling and grammar: for example "dependancies", "specifed", "usally", "Documenation", "latititude", doubled words, subject-verb agreement, and "Replace X with".
  • Names: GitHub and GitHub Actions capitalized consistently, Madden-Julian, the Cooperative Institute for Research in the Atmosphere (CIRA), Naval Research Laboratory, the JTTI expansion, METdatadb → METdataio.
  • Copied text: removed "line plot" and "performance diagram" text that had been copied onto the bar, box, spread-skill, and wind rose pages; a repeated sentence and phrases; stray text ("is66TAW"); "Cost/Lost" → "Cost/Loss".

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: eclv_defaults.yaml, ./eclv.data.
  • Box and line: minimal_box.yaml, custom_line.yaml, minimal_line.yaml (the files used in the commands).
  • Scatter: edit $WORKING_DIR/custom_scatter.yaml (where the page copies it), the data file reformatted_data_for_scatter.data, and the output setting in custom_scatter.yaml.
  • Taylor diagram: the directory $METPLOTPY_BASE/test/taylor_diagram, the data file plot_dlwr_sample.data (all 5 references), and minimal_taylor_diagram.yaml.
  • Performance diagram: minimal_performance_diagram.yaml, and the .points1 file.
  • Wind rose: the test/wind_rose directory, wind_rose_defaults.yaml, wind_rose.py, minimal_wind_rose.yaml, and wind_rose_custom.yaml (the page said custom_performance_diagram.yaml).
  • Reliability diagram: reliability_defaults.yaml and minimal_reliability.yaml (the page said minimal_box.yaml).
  • Histograms: rank_hist.yaml, prob_hist.yaml, and rel_hist.yaml (the page listed rank_hist.yaml three times), and dump_points_2.
  • Stratosphere plots: stratosphere_plots.py (was .py.py).

TCMPR Setup

  • The log levels are listed from most to least verbose as DEBUG, INFO, WARNING, ERROR (INFO and DEBUG were swapped).
  • The number of colors, not columns, must match the number of series.
  • Fixed the /path/to/METplotpy_source_code path, and the METcalcpy step now sets METCALCPY_BASE to the METcalcpy source (it set METPLOTPY_BASE to METplotpy), with valid csh setenv syntax.

Descriptions, Links, and References

  • stratosphere_plots.rst: obar is 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.py is 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.py reads dump_points_2).

  • Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

    Review the diff against develop, 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.

  • Add any new Python packages to the METplus Components Python Requirements table.
  • Review the source issue metadata (required labels, projects, and milestone).
  • Complete the PR definition above.
  • Ensure the PR title matches the feature or bugfix branch name.
  • Define the PR metadata, as permissions allow.
    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
  • After submitting the PR, select the ⚙️ icon in the Development section of the right hand sidebar. Search for the issue that this PR will close and select it, if it is not already selected.
  • After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
  • Close the linked issue and delete your feature or bugfix branch from GitHub.

🤖 Generated with Claude Code

JohnHalleyGotway and others added 8 commits October 5, 2026 12:31
…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

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

Labels

None yet

Projects

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

1 participant