check `dat` argument (and others) at the start of save_all_plots()
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Encuentra save_all_plots() e inspecciona cómo utiliza dat y la estructura de salida de convert_output(). Añade una validación temprana de la estructura esperada y comprueba que una entrada no válida falle rápidamente con un mensaje informativo, en lugar de producir muchos mensajes ... failed to run..
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- R
- Estrellas
- 20
- Forks
- 10
- Merge medio
- 3 d 9 h
- PR fusionados (30 d)
- 21
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nmfs-ost/stockplotr
-
[bug] plot_spawning_potential_ratio() example throwing errorPosiblemente ocupada @JZahner1 la tomó hace 1 día. Abierto
nmfs-ost/stockplotr#444 · 4 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Release Checklist - Monthly UpdatePosiblemente ocupada @Schiano-NOAA la tomó hace 1 día. Abierto
nmfs-ost/stockplotr#442 · 2 asignados ·
Los mantenedores suelen responder en 1 día
-
update SS3 testsPosiblemente ocupada @Schiano-NOAA la tomó hace 10 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
nmfs-ost/stockplotr#441 · 4 comentarios ·
Los mantenedores suelen responder en 1 día
-
Brainstorm ways to automatically update `save_all_plots()`Posiblemente ocupada @sbreitbart-NOAA la tomó hace 9 días. Abierto
nmfs-ost/stockplotr#432 · 4 comentarios · 2 asignados ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
nmfs-ost/stockplotr#422 ·
Los mantenedores suelen responder en 1 día
Todos los issues de nmfs-ost/stockplotr
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 71/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
OHDSI/PatientLevelPrediction#688 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
rfordatascience/tidytuesday#1098 ·