pfmc-assessments / PacFIN.Utilities

R code to manipulate data from the PacFIN database for assessments
http://pfmc-assessments.github.io/PacFIN.Utilities
Other
7 stars 1 forks source link

Recognize age method 10 #121

Closed okenk closed 8 months ago

okenk commented 11 months ago

Is your feature request related to a problem? Please describe. @brianlangseth-NOAA and I recently learned that the CAPS lab aged canary rockfish for the 2023 cycle using a new method "break and bake." ODFW has coded this method as 10 in PacFIN. However, getAge does not recognize age method 10. While at present, this method only applies to canary rockfish, it will likely be used again with other species going forward.

Describe the solution you'd like Include method 10, "break and bake", under the umbrella of break and burn methods. That is, if a user wants to include break and burn ages, method 10 will be included instead of excluded.

Describe alternatives you've considered Create a new option for keep_age_method in cleanPacFIN, so break and bake is not included within break and burn, but is an option that getAge recognizes and can retain. I do not think this is as good of a solution because future assessors may not recognize break and bake, and thus accidentally end up excluding these perfectly good ages, as we did, whereas I cannot really imagine cases where someone would want only one method but not the other.