nmfs-ost / nmfs-ost/ss3-source-code

[Feature]: Include MSY options selected in Report.sso

Open
#294 2 comments 0 reactions 2 assignees View on GitHub

@iantaylor-NOAA is already working on this.

Since Dec 1, 2025.

misc. output new request question
Dominant language
C++
Stars
46
Forks
19
PR merge metrics
No merged PRs in 30d

Description

Thanks @puntae for the suggestion to provide more info in report.sso regarding run settings (e.g., MSY). It is sometimes challenging for reviewers of stock assessments to know the definition of MSY when just looking at the report file, because the settings are only shown in the forecast.ss input file.

Note that Definitions are often at the top of tables in the report file currently; it might be nice to put in all in one place instead. Not a lot of info available in the starter and forecast files is included in Report.sso currently.

Please feel free to add more suggestions or ideas.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.