Skip to content

Developing risk table based on AFSC#14

Closed
sbreitbart-NOAA wants to merge 3 commits intomainfrom
afsc-safe-tbl
Closed

Developing risk table based on AFSC#14
sbreitbart-NOAA wants to merge 3 commits intomainfrom
afsc-safe-tbl

Conversation

@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator

  • Added @BenWilliams-NOAA 's comma function from the AFSC safe package
  • Started making risk table based on AFSC's SAFE main table. Copy/pasted Ben's code, then started adding code to extract specific variables, like total biomass, from model output file
  • Stopped so that we could examine example AFSC model output and decide how to proceed

@BenWilliams-NOAA
Copy link
Copy Markdown

@sbreitbart-NOAA for clarification, this is the 'main table' which is pretty much the first thing our council, ssc, and ap look at. At the afsc a risk table is a qualitative indicator used to determine whether there should be a reduction in catch below maxABC .
The main table is created by pulling from the prior assessment table (if it exists) and incorporated into the parent document for consistency in reported values.

@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator Author

@sbreitbart-NOAA for clarification, this is the 'main table' which is pretty much the first thing our council, ssc, and ap look at. At the afsc a risk table is a qualitative indicator used to determine whether there should be a reduction in catch below maxABC . The main table is created by pulling from the prior assessment table (if it exists) and incorporated into the parent document for consistency in reported values.

Hi @BenWilliams-NOAA - thanks for letting me know!

@Schiano-NOAA Schiano-NOAA changed the base branch from master to main February 24, 2025 22:06
@Schiano-NOAA
Copy link
Copy Markdown
Collaborator

@sbreitbart-NOAA sorry this is so late but is this function adjusted for the standard output dataframe or based on the one from Ben? If we just want to merge in so the table is a part of stockplotr then we need to change the definition of dat in this file rather than inheriting parameters.

@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator Author

@sbreitbart-NOAA sorry this is so late but is this function adjusted for the standard output dataframe or based on the one from Ben? If we just want to merge in so the table is a part of stockplotr then we need to change the definition of dat in this file rather than inheriting parameters.

This function was started using the standard output df. We stopped developing it in the fall because we needed AFSC sample data and didn't have it yet. So this function is still very much in dev.

@Schiano-NOAA Schiano-NOAA marked this pull request as draft April 16, 2025 13:55
@sbreitbart-NOAA sbreitbart-NOAA linked an issue May 6, 2025 that may be closed by this pull request
@Schiano-NOAA
Copy link
Copy Markdown
Collaborator

@sbreitbart-NOAA do you want to close this PR or leave it open and revamped it at another time? The branch is pretty outdated at this point, so might be worth it to move the foundation to something else, but up to you!

@sbreitbart-NOAA
Copy link
Copy Markdown
Collaborator Author

@sbreitbart-NOAA do you want to close this PR or leave it open and revamped it at another time? The branch is pretty outdated at this point, so might be worth it to move the foundation to something else, but up to you!

I'm fine with closing it. It is pretty outdated!

@Schiano-NOAA Schiano-NOAA deleted the afsc-safe-tbl branch April 9, 2026 20:42
@Schiano-NOAA Schiano-NOAA restored the afsc-safe-tbl branch April 9, 2026 20:42
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.

Add AFSC Risk Table

3 participants