Reduce runtime of `convert_output` tests
I maintainer di solito rispondono entro 1 giorno
@JZahner1 ci sta già lavorando.
Dal 9/10/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
The current convert_output test loops over all SS3 fixture models (tests/testthat/test-convert_output.R with fixtures/ss3_models/models/*). At 18 total models, this takes substantial time. Reducing the number of models to be tested should greatly increase efficiency, particularly given the size of some SS3 Report.sso files.
I propose selecting 3-5 models that span a range of complexity and SS3 modeling options to create a representative subset of models to test on. Initial thoughts are that:
- Hake_2018 or Hake_2019_semi-parametric_selex
- Empirical_Wtatage_Age_Selex
- platoons
- three_area_nomove
And one of: BigSkate2019, vermillion_snapper, Simple_*
would make for a nice subset in terms of features and complexity.
- Lingua principale
- R
- Stelle
- 20
- Fork
- 8
- Merge medio
- 20h 52m
- PR unite (30g)
- 10
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di nmfs-ost/stockplotr
-
Handful of Variable Naming Convention ThingsForse già presa @Schiano-NOAA l’ha presa 3 giorni fa. Aperta
nmfs-ost/stockplotr#407 · 1 reazione · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 18/100
nmfs-ost/stockplotr#405 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
[feat]: plot_age_to_lengthAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
nmfs-ost/stockplotr#403 ·
I maintainer di solito rispondono entro 1 giorno
-
[feat] Begin deprecation steps for report.sso file in convert_output()Forse già presa @Schiano-NOAA l’ha presa 4 giorni fa. Aperta
nmfs-ost/stockplotr#402 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
fix: RCeattle `convert_output`Forse già presa @Schiano-NOAA l’ha presa 4 giorni fa. Aperta
nmfs-ost/stockplotr#400 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nmfs-ost/stockplotr
Issue simili
-
Write an exception for delayed NYC data onlyForse già presa @kaitejohnson l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
New `filter_out()` requires dplyr >= 1.2.0Forse già presa @jameslairdsmith l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
-
Add smvr alongside semverAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100