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

Modify unit tests #75

Open
rklasky opened this issue Dec 5, 2023 · 0 comments
Open

Modify unit tests #75

rklasky opened this issue Dec 5, 2023 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@rklasky
Copy link
Collaborator

rklasky commented Dec 5, 2023

  1. Some unit tests show a 0 output biomass

Modifying the ForcedMigrate parameter with either random or stepped noise causes the output biomass to be 0 when running an RK4 simulation. This also happens with ForcedCatch for Seals but with an AB simulation. Use noise variations of no more than 10% around mean of existing forced parameter (i.e., catch). Add noise of only +-10% of mean around forced parameter in question.

  1. Save current-baseline differences as output data files in addition to the .pdf containing all of the plots.

  2. Allow user to choose between a) showing current and baseline plots superimposed on the same plot or b) showing only the difference of current-baseline plots on each plot.

  3. Add Unit Test Vignette

  4. Add JuvRoundfish1 and AduRoundfish1 species groups.

@rklasky rklasky self-assigned this Dec 12, 2023
@rklasky rklasky added the bug Something isn't working label Dec 13, 2023
@rklasky rklasky changed the title Some unit tests show a 0 output biomass Modify unit tests Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant