Standardize indicator names and adjust figure sizing - #67
Open
stephanie-owen wants to merge 13 commits into
Open
Standardize indicator names and adjust figure sizing#67stephanie-owen wants to merge 13 commits into
stephanie-owen wants to merge 13 commits into
Conversation
…print metadata on plots
…for NE only plots
… '.._both_reports/R'
…put in save_plot() functions in MAB/NE plots/slides scripts
…ecodata source info
Member
|
Thanks for these updates! I'm in the process of versioning the repo and archiving the 2026 docs, after that we can start incorporating changes for 2027 into the dev branch. I'll review and merge this pr in a week or two. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In the figure_captions_summary spreadsheet, the first column
chunkNamehad been previously changed to match the respective ecodata object for each plot. In this PR, I matched thechunkNamecolumn in the spreadsheet to theindicator =value passed in eachsave_plotfunction.All indicator names have been matched in the following scripts:
create_plots_both_reportscreate_plots_ne_onlycreate_plots_mab_necreate_plots_slides_both_reportscreate_plots_slides_ne_onlycreate_plots_slides_mab_neAll figures have been re-ran so the figure names now match the chunkNames. I then edited the figure paths in the spreadsheet to source to the correct figures, and removed figures with old naming conventions.
Finally, I increased the plot height on all figures in the scripts above by 0.25 to account for the added ecodata source information text at the bottom of the plots