`crps.numeric` does not pass `permutations` correctly to `crps.matrix`
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
Línea de trabajo
El problema está en R/crps.R línea 97, donde permutations no se pasa correctamente. Comience examinando la función crps.numeric y su llamada a crps.matrix. Asegúrese de que el argumento permutations se pase explícitamente. También verifique scrps.numeric para el mismo patrón. Ejecute el ejemplo reproducible proporcionado para verificar la corrección.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Minimal reproducible example:
paste("loo version:", packageVersion("loo"), "\n") |> cat()
N = 1000000
cat("crps.numeric:\n")
system.time(
loo:::crps.numeric(1:3, 4:6, 5, permutations=N)
)
cat("crps.matrix:\n")
system.time(
loo:::crps.matrix(as.matrix(1:3), as.matrix(4:6), 5, permutations=N)
)
Output on my machine:
loo version: 2.10.1
crps.numeric:
user system elapsed
0.007 0.002 0.009
crps.matrix:
user system elapsed
6.954 0.033 6.989
I believe the issue is with this line:
https://github.com/stan-dev/loo/blob/df27ade95093db66cebf19d55530439386ac1a97/R/crps.R#L97
The permutations argument gets absorbed into the ...; I believe it needs to be passed as a named argument.
Also applies to scrps.numeric.
- Lenguaje dominante
- R
- Estrellas
- 157
- Forks
- 38
- Merge medio
- 4 d 16 h
- PR fusionados (30 d)
- 2
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 stan-dev/loo
-
Extend loo_plot_difference Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Clean up issues Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Todos los issues de stan-dev/loo
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
robjhyndman/forecast#1220 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
JamesHWade/deputy#192 ·
-
bug triage_needed
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
pharmaverse/rtables#1123 · 1 comentario · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100