Skip to content

Update global time series plotting docs#308

Merged
forsyth2 merged 1 commit intomainfrom
glb-plot-docs-update
Oct 25, 2022
Merged

Update global time series plotting docs#308
forsyth2 merged 1 commit intomainfrom
glb-plot-docs-update

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-pnnl
Copy link

evasinha-pnnl 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