Skip to content
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

Update global time series plotting docs #308

Merged
merged 1 commit into from
Oct 25, 2022
Merged

Conversation

forsyth2
Copy link
Collaborator

Improve documentation on adding a new plot to Global Time Series.

@forsyth2 forsyth2 added the Documentation Files in `docs` modified label Sep 26, 2022
@forsyth2 forsyth2 self-assigned this Sep 26, 2022
@forsyth2 forsyth2 force-pushed the glb-plot-docs-update branch from faabcda to 11a5574 Compare September 26, 2022 19:05
@forsyth2 forsyth2 changed the title Glb plot docs update Update global time series plotting docs Sep 26, 2022
@forsyth2
Copy link
Collaborator Author

@evasinha Add any comments here on what would be helpful to include in the plotting docs. Once we successfully merge a new plot into zppy, we can merge this doc update. Thanks!

@evasinha
Copy link

evasinha commented Sep 27, 2022

@forsyth2 - Maybe adding something similar to this:

After making edits to your zppy fork, use the following pre-commit commands described earlier before generating plots with your edits:

pre-commit install
pip install .

@forsyth2 forsyth2 force-pushed the glb-plot-docs-update branch from 11a5574 to a6fc217 Compare October 25, 2022 20:47
@forsyth2 forsyth2 force-pushed the glb-plot-docs-update branch from a6fc217 to db37fd6 Compare October 25, 2022 20:48
@forsyth2 forsyth2 marked this pull request as ready for review October 25, 2022 20:48
@forsyth2 forsyth2 merged commit cdb40e1 into main Oct 25, 2022
@forsyth2 forsyth2 deleted the glb-plot-docs-update branch October 25, 2022 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Files in `docs` modified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants