`crps.numeric` does not pass `permutations` correctly to `crps.matrix`
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 75/100
調査の方向性
問題は R/crps.R の 97 行目にあり、permutations が正しく渡されていません。まず、crps.numeric 関数とその crps.matrix への呼び出しを調べてください。permutations 引数が明示的に渡されていることを確認してください。また、同じパターンがないか scrps.numeric も確認してください。提供された再現可能な例を実行して修正を検証してください。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- R
- スター
- 157
- フォーク
- 38
- 平均マージ
- 4日 16時間
- マージ済み PR（30日）
- 2
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
stan-dev/loo のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
-
Clean up issues オープン
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
robjhyndman/forecast#1220 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
JamesHWade/deputy#192 ·
-
bug triage_needed
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
pharmaverse/rtables#1123 · コメント 1 件 · リアクション 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100