Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

triple_to_grid docstring fix #100

Merged
merged 12 commits into from
Aug 12, 2022
Merged

triple_to_grid docstring fix #100

merged 12 commits into from
Aug 12, 2022

Conversation

anissa111
Copy link
Member

Summary of changes:

  • changes to docstring structure for rtd generation
  • fix of issue in parameter order in docstring example

See reported issue in geocat-comp here: NCAR/geocat-comp#235

Note: this PR only addresses triple_to_grid, more docstrings should undergo similar updates

Copy link
Contributor

@hCraker hCraker left a comment

Choose a reason for hiding this comment

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

I found some typos in the docstring. Otherwise it looks okay to me.

src/geocat/f2py/triple_to_grid_wrapper.py Outdated Show resolved Hide resolved
src/geocat/f2py/triple_to_grid_wrapper.py Outdated Show resolved Hide resolved
src/geocat/f2py/triple_to_grid_wrapper.py Outdated Show resolved Hide resolved
src/geocat/f2py/triple_to_grid_wrapper.py Outdated Show resolved Hide resolved
src/geocat/f2py/triple_to_grid_wrapper.py Outdated Show resolved Hide resolved
anissa111 and others added 5 commits August 12, 2022 09:05
Co-authored-by: Heather Craker <hcraker@ucar.edu>
Co-authored-by: Heather Craker <hcraker@ucar.edu>
Co-authored-by: Heather Craker <hcraker@ucar.edu>
Co-authored-by: Heather Craker <hcraker@ucar.edu>
Co-authored-by: Heather Craker <hcraker@ucar.edu>
@anissa111
Copy link
Member Author

Ah! Great catch(es) Heather, thank you!

@anissa111 anissa111 requested a review from hCraker August 12, 2022 15:06
@anissa111 anissa111 dismissed hCraker’s stale review August 12, 2022 16:51

changes accepted

@anissa111 anissa111 merged commit f873519 into main Aug 12, 2022
@anissa111 anissa111 deleted the triple_to_grid_examples branch August 12, 2022 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants