Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve documentation content and building #212

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

stanwest
Copy link

This pull request corrects and improves the content of the documentation and resolves several warnings and errors that appear when building the documentation.

The main content changes are as follows:

  • Correct math expressions in docstrings for cross_phase (changing math "extArg" to text "Arg") and for isotropic_cross_spectrum, isotropic_power_spectrum, and isotropize (changing math "extiso" to text "iso").
  • Correct the rendering of the equations under xrft.power_spectrum.
  • Correct the rendering of the list under the "mode" argument of padding.pad.
  • Move parameters of padding.unpad into the "Parameters" section from the "Returns" section.
  • Format keyword arguments more consistently.
  • Correct the rendering of the issue directives in "doc/whats-new.rst".

The resolved messages include:

  • SyntaxWarning: invalid escape sequence '\m'
  • WARNING: Invalid configuration value found: 'language = None'. Update your configuration to a valid language code. Falling back to 'en' (English).
  • WARNING: html_static_path entry '_static' does not exist
  • WARNING: Explicit markup ends without a blank line; unexpected unindent.
  • ERROR: Unexpected indentation.
  • WARNING: Block quote ends without a blank line; unexpected unindent.
  • WARNING: Inline strong start-string without end-string.
  • WARNING: Title underline too short.
  • ERROR: Unknown interpreted text role "issue".

Resolves messages such as "SyntaxWarning: invalid escape sequence '\m'"
that can appear in tracebacks and when building the documentation.

Corrects math expressions in docstrings for `cross_phase` (changing math
"extArg" to text "Arg") and for `isotropic_cross_spectrum`,
`isotropic_power_spectrum`, and `isotropize` (changing math "extiso" to
text "iso").
Also resolves the message "WARNING: Inline strong start-string without
end-string. [docutils]" when building documentation.
Extend a title underline to the correct length, separate a bulleted list
with blank lines, and indent the content of math directives.  Corrects
the rendering of the "mode" argument of `padding.pad` and of the
equations under `xrft.power_spectrum`.  Resolves the following messages
when building the documentation:

doc/whats-new.rst:
    *   WARNING: Title underline too short.

xrft/padding.py:
    *   ERROR: Unexpected indentation.
    *   WARNING: Block quote ends without a blank line; unexpected
        unindent.

xrft/xrft.py:
    *   WARNING: Explicit markup ends without a blank line; unexpected
        unindent.
Corrects the rendering of the "issue" directives in "doc/whats-new.rst"
and resolves the message 'ERROR: Unknown interpreted text role "issue".'
when building the documentation.
Resolves the message "WARNING: Invalid configuration value found:
'language = None'. Update your configuration to a valid language code.
Falling back to 'en' (English)." when building the documentation.
Resolves the message "WARNING: html_static_path entry '_static' does not
exist" when building the documentation.
Format keyword arguments more consistently and correct spelling.
Resolves the message "WARNING: No 'apidoc_module_dir' specified;
skipping API doc generation" that appears when building the
documentation.
@stanwest
Copy link
Author

The most recent commit resolves the last warning message that I encounter:

  • WARNING: No 'apidoc_module_dir' specified; skipping API doc generation

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant