Kelli Johnson

call-update-

1

Incorrect name of stock synthesis is being saved when updating the version. Should be "ss3.exe" for Windows. Check the names for other operating systems as well.

Use `@inheritParams` instead of [template folder](https://github.com/ss3sim/ss3sim/tree/f5e133dc897154c1bccc56380858a70264f8faff/man/roxygen/templates) for roxygen of arguments that are shared across parameters. This approach is documented in * https://r-pkgs.org/man.html#inheriting-arguments and * https://cran.r-project.org/web/packages/roxygen2/vignettes/reuse.html

Pull request to update ss3 in {ss3sim} did not run R-CMD checks

1

[update-ss3-version.yml](https://github.com/nmfs-stock-synthesis/workflows/tree/main/.github/workflows/update-ss3-version.yml) led to correct changes in all of the necessary files (see [5c169f9](https://github.com/ss3sim/ss3sim/pull/396/commits/5c169f92f165757e1b9451aef1fce63accfd0838)) but the resulting pull request did not run R-CMD check like normal pull requests do. I am...

call-update-ss3-version.yml does not update NEWS.md

1

The GitHub action needs to open NEWS.md and update the file to indicate the new version number of {ss3sim} and the version number of SS3 being used.

use nmfs-fish-tools pkgdown template

6

Thanks to @eeholmes and @ChristineStawitz-NOAA for the NMFS pkgdown template. I would like to use this for ss3sim documentation. Just trying to figure out what files I need to add...

Another idea @latreesedenson-NOAA and I thought of yesterday related to this one is separating the OM modifications, sampling, and EM running in `ss3sim_base` into separate functions so that they can...

Update vignettes

1

## Problem The vignettes hadn't been looked at in a while and a large number of issues with the vignette label were accumulating. Also, [pkgdown](https://pkgdown.r-lib.org/) is pretty great and will...

combine output from scalar and time series to a single file

3

## Current problem The original results for ss3sim were split into two files because the size of the file for the time series was what we thought was **very** large....

Refactor `get_results_all()` to be more efficient

4

## Problem There are multiple issues that pertain to the results because the function is large and slow, this issues summarizes them. ## Proposal - [ ] #321 - [...

deprecate convert_to_wide

3

@k-doering-NOAA do you think that we can remove https://github.com/ss3sim/ss3sim/blob/eeb0b33a7d30eca93c586a1724fa774e3f21fc44/R/get-results.r#L613 Seems like the long results have been around long enough that we might not need to provide support for the "old"...