Skip to content

Comments

change multiyr average approach#589

Merged
Rick-Methot-NOAA merged 2 commits intomainfrom
586-feature-10+-multiyear-average-f_std_scaling
Apr 25, 2024
Merged

change multiyr average approach#589
Rick-Methot-NOAA merged 2 commits intomainfrom
586-feature-10+-multiyear-average-f_std_scaling

Conversation

@Rick-Methot-NOAA
Copy link
Collaborator

@Rick-Methot-NOAA Rick-Methot-NOAA commented Apr 19, 2024

Concisely describe what has been changed/addressed in the pull request.

Allows for multiyear averaging of F/Fmsy for >10 years. Per request from @HaikunXu

What tests have been done?

example model "simple" was used as test case to demonstrate fidelity of the change.

What tests/review still need to be done?

None

Is there an input change for users to Stock Synthesis?

in starter.ss there are needed changes if multiyr or log(ratio) is invoked

Additional information (optional).

@Rick-Methot-NOAA Rick-Methot-NOAA marked this pull request as ready for review April 19, 2024 16:57
@Rick-Methot-NOAA
Copy link
Collaborator Author

@chantelwetzel-noaa I like your suggestion of putting an example directly in starter.ss_new.

@e-perl-NOAA e-perl-NOAA force-pushed the 586-feature-10+-multiyear-average-f_std_scaling branch from 7c13940 to 4520662 Compare April 25, 2024 19:52
@Rick-Methot-NOAA Rick-Methot-NOAA merged commit 43a8886 into main Apr 25, 2024
@Rick-Methot-NOAA Rick-Methot-NOAA deleted the 586-feature-10+-multiyear-average-f_std_scaling branch April 25, 2024 23:14
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.

[Revise]: 10+ multiyear average F_std_scaling; with change to log(ratio) approach

2 participants