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

[Bug]: SPR/YPR profile and global_MSY not using correct biology vectors #625

Open
Rick-Methot-NOAA opened this issue Oct 4, 2024 · 1 comment
Assignees
Labels
benchmark - MSY Epic: Equilibrium calcs Groups together issues/user stories releated to equilibrium calculations in progress This is being worked on in a branch
Milestone

Comments

@Rick-Methot-NOAA
Copy link
Collaborator

Describe the bug

These calculations were using biology from the last year of forecast, not the biology associated with Benchmark specifications. This is all being worked on in the issue #191 that worked on correcting use of biology vectors for benchmark calculations.

To Reproduce

set up model with time-varying biology and set benchmark years to not be styr. Output provided by SS3.20.22 was not sufficient to see how time-varying biology was being used for these SPR/YPR

Expected behavior

correct results

Screenshots

No response

Which OS are you seeing the problem on?

No response

Which version of SS3 are you seeing the problem on?

No response

Additional Context

No response

@Rick-Methot-NOAA Rick-Methot-NOAA added benchmark - MSY in progress This is being worked on in a branch Epic: Equilibrium calcs Groups together issues/user stories releated to equilibrium calculations labels Oct 4, 2024
@Rick-Methot-NOAA Rick-Methot-NOAA added this to the 3.30.24 milestone Oct 4, 2024
@Rick-Methot-NOAA Rick-Methot-NOAA self-assigned this Oct 4, 2024
@Rick-Methot-NOAA
Copy link
Collaborator Author

Rick-Methot-NOAA commented Oct 4, 2024

Benchmark quantities stored in y = styr-3

  • Ave_Size
  • NatM (surv1 is stored by season only, but is only used by Pope's F method)
  • Wt_Age_t, which includes (-2 for fecundity, -1 for beg of year, 0 for mid year wt.)
  • save_sel_num (used for output only)
  • sel_a
  • sel_l
  • sel_l_r
  • migrrate
  • retain
  • retain_a
  • discmort
  • discmort2_a
  • discmort2
  • discmort_a

Seasonal values that yet derived from above in Make_FishSelex()
sel_dead_num and 5 other sel vectors

vector created in get_mat_fec() for spawn season only
fec and then gets stored in Wt_Age_t(t, -2, g)

Quantities used in equil_calcs, so all need to be available before calls to equil_calc:
t_base = styr + (eq_yr - styr) * nseas - 1;
recr_dist(y
natM(t_base +
natM(t,
natM(t_base + 1
Hermaphro_val(GP4(g)
migrrate(bio_yr
sel_dead_bio(s and 5 similar sel_ vectors
pred_M2(f1, t) using t=t_base+s
Wt_Age_beg(s
fec(


add temp output of bio_yr and t_base to check what equil_calc is using. Find that during forecast the bio_yr was set to endyr, not to the current forecast year.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmark - MSY Epic: Equilibrium calcs Groups together issues/user stories releated to equilibrium calculations in progress This is being worked on in a branch
Projects
Status: No status
Status: No status
Development

Successfully merging a pull request may close this issue.

1 participant