Skip to content

Scale uncertainty intervals if relative = TRUE for B and SB plots#230

Merged
Schiano-NOAA merged 18 commits intodevfrom
fix-rel-b
May 1, 2026
Merged

Scale uncertainty intervals if relative = TRUE for B and SB plots#230
Schiano-NOAA merged 18 commits intodevfrom
fix-rel-b

Conversation

@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator

Address #218

Comment thread R/plot_biomass.R Outdated
Copy link
Copy Markdown
Collaborator

@Schiano-NOAA Schiano-NOAA left a comment

Choose a reason for hiding this comment

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

Putting this on halt until we can talk to the steering committee and also incorporate the calcs for error here

Schiano-NOAA and others added 2 commits April 16, 2026 09:56
* initial commit of new obs v pred plot

* adjust process data so it converts all indexed data to character to function even when input values are numeric per #212

* update plot indices to use new plot_obsvpred function and adjust based on needs

* update documentation for package

* adjust plot_indices where nfleet = 1 and update test to remove new file produced

* add missing dependency
@Schiano-NOAA
Copy link
Copy Markdown
Collaborator

@sbreitbart-NOAA can you review this? 😄

Comment thread R/plot_biomass.R
prepared_data <- filter_data(
dat = dat,
label_name = "^biomass$",
label_name = ifelse(relative, "biomass_biomass_unfished|biomass_ratio", "^biomass$"), # what exactly is biomass_ratio?
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Did you find out what biomass_ratio is? 😄

Comment thread R/plot_fishing_mortality.R
Comment thread R/plot_fishing_mortality.R
@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator Author

Looks good to me! Thanks for updating those csvs 👍

@Schiano-NOAA Schiano-NOAA merged commit b5aa352 into dev May 1, 2026
1 check 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.

[Bug]: plot_biomass not plotting relative error values when relative = TRUE

3 participants