`crps.numeric` does not pass `permutations` correctly to `crps.matrix`
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 75/100
Hướng nghiên cứu
Vấn đề nằm ở dòng 97 trong R/crps.R, nơi permutations không được truyền đúng cách. Bắt đầu bằng cách kiểm tra hàm crps.numeric và lệnh gọi của nó tới crps.matrix. Đảm bảo đối số permutations được truyền một cách rõ ràng. Cũng kiểm tra scrps.numeric cho cùng một mẫu. Chạy ví dụ có thể tái tạo được cung cấp để xác minh bản sửa lỗi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- R
- Star
- 157
- Fork
- 38
- Merge trung bình
- 4 ngày 16 giờ
- Pull request đã merge (30 ngày)
- 2
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của stan-dev/loo
-
Extend loo_plot_difference Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
-
Clean up issues Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 68/100
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
robjhyndman/forecast#1220 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
JamesHWade/deputy#192 ·
-
bug triage_needed
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
pharmaverse/rtables#1123 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100