-
Notifications
You must be signed in to change notification settings - Fork 7
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
Figure j in exec summary and Figure 39 need forecast time series #1111
Comments
Thanks for adding these @cgrandin. I notice that the pre-forecast relative spawning biomass time series in these plots don't match our relative spawning biomass time series without forecasts (i.e., figures e and 27) - maybe it is only for the 2024 point. Last year, the end year (2023) lined up correctly. |
It's for the 2024 point, because 2024 is replaced with the forecast value for that plot. It is the only way to make the series line up, and it is "more" correct isn't it? In previous years it was the same, the end year in the two plots is not the same. |
I don't think it is correct. Biomass (and relative biomass) at the start of 2024 is not a forecast. Our executive summary says rel biomass at the start of 2024 is 76.4%, but then these forecast figures say it is ~43%. I think that's wrong. Our decision tables all show rel biomass at the start of 2024 is 76.4% as well. This figure in last years document lines up with what is in the executive summary (i.e., both show 2023 rel biomass just above B0), so there must have been a change. Last years decision tables show rel biomass for 2023 at 104% (matching). |
Ok, I can put it back to use the 2024 value from the model output. It will look really weird though as there'll be a huge bump up and then down in one year. |
Updated the 2024 depletion in b4313bc |
The values appear to be correct... This is the chunk of code where the calculations take place: The depl value is Bratio_* / SSB_Initial from the forecasts mcmc values (posteriors.sso). Is that correct? See this line: hake-assessment/R/load-forecasts.R Line 100 in b4313bc
|
Nevermind... I shouldn't be dividing BRatio by SSB_Initial, it should just be BRatio itself. Stand by |
ohh - that looks good! I don't blame you at all. I blame myself for asking you to make the static and dynamic B0 plots and the fished and unfished biomass trajectories plots because then you start dividing by all sorts of non-usual stuff that gets confusing. So I'm guessing you were still thinking on that level..... Close this sucker up Chris. |
Done in 5031a68 |
These two identical figures (one in the main document and one in the executive summary) need the forecast catch scenarios added to the plot.
The text was updated successfully, but these errors were encountered: