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

change package name to stockplotr #50

Merged
merged 4 commits into from
Feb 24, 2025
Merged

change package name to stockplotr #50

merged 4 commits into from
Feb 24, 2025

Conversation

Schiano-NOAA
Copy link
Collaborator

Changing package name

Schiano-NOAA and others added 4 commits February 24, 2025 14:19
* Added units to plots that can produce relative values (biomass, recruitment, and spawning biomass) as per #44

* Added new arg to add_more_key_quants to allow for values to be scaled; updated fxns with this arg (plot_biomass, recruitment, ssb); moved code that computes min/max B, R, ssb from write_captions to add_more_key_quants; updated add_more_key_quants to label scaled values with appropriate label that's inclusive of scale's order of magnitude

* Added code that adds new key quantities extracted in add_more_key_quants into captions/alt text csv; updated exp_all_figs_tables to include new arguments specifying scaling amounts for biomass, recruitment, and sp biomass plots

* Fix test

* Update documentation for exp_all_figs_tables

* Attempt to fix test failures by fixing path in add_more_key_quants()

* add dat arg to B, R, SSB plotting fxns, and add_more_key_quants, to address failing tests

* Updates per Schiano-NOAA's comments on PR (minor edits to code, documentation)
@Schiano-NOAA Schiano-NOAA merged commit 7361412 into master Feb 24, 2025
3 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants