`crps.numeric` does not pass `permutations` correctly to `crps.matrix`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 75/100
Direzione di ricerca
Il problema si trova in R/crps.R riga 97, dove permutations non viene passato correttamente. Inizia esaminando la funzione crps.numeric e la sua chiamata a crps.matrix. Assicurati che l'argomento permutations venga passato esplicitamente. Controlla anche scrps.numeric per lo stesso schema. Esegui l'esempio riproducibile fornito per verificare la correzione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- R
- Stelle
- 157
- Fork
- 38
- Merge medio
- 4g 16h
- PR unite (30g)
- 2
Guida per i contributori
Apri 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 stan-dev/loo
-
Extend loo_plot_difference Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
-
Clean up issues Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
Tutte le issue di stan-dev/loo
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
robjhyndman/forecast#1220 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
JamesHWade/deputy#192 ·
-
bug triage_needed
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
pharmaverse/rtables#1123 · 1 commento · 1 reazione ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100