Open iantaylor-NOAA opened 3 years ago
I will see what I can do, I think @chantelwetzel-noaa used an asterisk when the full catch couldn't be taken.
I'd be happy with an asterisk.
I had the issues where full removals could not be taken from the final years of the projection in the low state of nature. In my decision table on page xx I highlighted these values by italics and provided a description in the caption. However, in hindsight I think a better approach would be via an asterisk which then could be added as a footnote to the table (or in the table caption).
Used max catch for each year, if it doesn't equal for a given catch stream then the SSB and depletion are italicized. I didn't have an example though to make sure it works. Just shooting off the hip.
We can change to an asterisk later with footnote, but wanted to get something by today.
Also, removed a lot of the coloration code b/c I had to add it to the conditional formatting that italicizes the column. This makes the colors work for no matter how many models are provided.
Done in e1df08c
The new table looked great but was creating duplicate rows for reasons that I couldn't figure out so I went back to the old version and set the catch column to match the middle (base) model to produce something for #166. The improvements will be useful but can wait until later.
@kellijohnson-NOAA, the models for decision table are on Google Drive (sens numbers starting with 600). Table function can be tested by downloaded those files for north or south and running these lines (or modifications of them): https://github.com/iantaylor-NOAA/Lingcod_2021/blob/main/models/model_decision_tables.R#L87-L115
Thanks @iantaylor-NOAA for giving me the model numbers, super helpful. I was able to figure out what I was doing wrong and have committed changes to table_decision with 7969544
Changes by @kellijohnson-NOAA back in August (507cf37), combined with minor edit to caption in commit 969a5c9 resolve this issue.
table_decision()
to italicize only the state of nature for which catch is limiting, rather than all states of nature.As a short-term fix for the final 2021 document, we will edit the .tex files, but a long-term fix can be made to the function itself in the future. Adding to nextassessment
benchmark.
The
table_decision()
function worked really well for quickly combining output from multiple models during the lingcod STAR.However, there are a few changes that I think would be useful:
I'm not sure how best to make these changes elegantly within the existing function. Also, it would be nice to not break the flexibility in the number of columns (models) within each row.
Also
Added 1 Dec 2021: