.args in summarise draws incompatible with summary functions without ellipses
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Bắt đầu tại điểm vào summarise_draws và tái hiện ví dụ với example_draws(), sd, quantile2 và mcse_quantile, xác nhận cách .args đến từng hàm tóm tắt. Được xem là hoàn tất khi probs được các hàm sử dụng nó chấp nhận mà không bị truyền đến stats::sd hoặc stats::var, đồng thời có coverage cho lỗi đã báo cáo.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The .args argument in summarise draws is passed to all summary functions as additional arguments. This fails if the summary function does not handle ... or the keyword arguments specified in .args. Common functions that don't handle ... include stats::sd and stats::var.
I'd like to be able to do something like the following:
x <- example_draws()
summarise_draws(x, mean, sd, quantile2, mcse_quantile) #q5 and q95 by default
#> [Output as expected]
# change the quantiles of interest to q1 and q99
additional_args <- list(probs = c(0.01, 0.99))
summarise_draws(x, mean, sd, quantile2, mcse_quantile, .args = additional_args)
#> Error in stats::sd(x, ...) : unused argument (probs = c(0.01, 0.99))
One option would be to make aliases of these common summary functions that handle ..., but perhaps there is a more sensible way to only pass the keywords to functions that handle them?
- Ngôn ngữ chính
- R
- Star
- 172
- Fork
- 27
- Merge trung bình
- 5 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc 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/posterior
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
-
Use testthat 3eCó thể đã có người làm @mcol đã nhận 7 ngày trước. Đang mởgood first issue test
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
Tất cả issue của stan-dev/posterior
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 68/100
cran-task-views/Spatial#127 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Add two cautions to the `simulate_dyad_responses()` helpCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởdocumentation priority: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Pascal-Kueng/dyadMLM#179 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Consider changesĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
OHDSI/OmopSketch#618 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
GSK-Biostatistics/tfrmt#854 ·