check `dat` argument (and others) at the start of save_all_plots()
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 68/100
Direzione di ricerca
Trova save_all_plots() e verifica come usa dat e la struttura di output di convert_output(). Aggiungi una validazione anticipata della struttura prevista, quindi verifica che un input non valido fallisca rapidamente con un messaggio informativo, invece di produrre molti messaggi ... failed to run..
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I accidentally passed an object to the dat input of save_all_plots() that wasn't the output from convert_output() and I got 100+ lines of messages about ... failed to run. Adding a check at the start to confirm that the dat input is of the appropriate structure would allow the function to fail faster and more informatively.
- Lingua principale
- R
- Stelle
- 20
- Fork
- 10
- Merge medio
- 3g 9h
- PR unite (30g)
- 21
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
-
[bug] plot_spawning_potential_ratio() example throwing errorForse già presa @JZahner1 l’ha presa 1 giorno fa. Aperta
nmfs-ost/stockplotr#444 · 3 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Release Checklist - Monthly UpdateForse già presa @Schiano-NOAA l’ha presa 1 giorno fa. Aperta
nmfs-ost/stockplotr#442 · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
update SS3 testsForse già presa @Schiano-NOAA l’ha presa 9 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
nmfs-ost/stockplotr#441 · 4 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Brainstorm ways to automatically update `save_all_plots()`Forse già presa @sbreitbart-NOAA l’ha presa 9 giorni fa. Aperta
nmfs-ost/stockplotr#432 · 4 commenti · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
nmfs-ost/stockplotr#422 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nmfs-ost/stockplotr
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
rfordatascience/tidytuesday#1098 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
microsoft/ML-For-Beginners#1056 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
atomashevic/transforEmotion#46 ·
I maintainer di solito rispondono entro 1 giorno
-
`pk.calc.time_above()` with lin-up/log-down silently drops segments that end at a negative valueAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
I maintainer di solito rispondono entro 1 giorno