search

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

feat: add control of "month" in writeComps() #90

Closed iantaylor-NOAA closed 1 year ago

iantaylor-NOAA commented 1 year ago

While working with Petrale data I realized that the status-quo writeComps() has month = 1 hard-wired for all comp data, even when there are multiple seasons. This change allows the user to choose a separate month for each season in seasonal models, as well as giving control of the month for annual models (with default to 7 instead of 1 since that seems to be the most common choice). I tested this with an input that had 2 seasons and another with only 1.

Please modify as you wish.

iantaylor-NOAA commented 1 year ago

Note that the github action call-r-cmd-check is failing on ubuntu, but the failure is related to setting up R before it even loads this package, so I think we can ignore it. I'm trying that same github actuion in a different repo here https://github.com/r4ss/r4ss/actions/workflows/call-r-cmd-check.yml to see if it works.