tests/test-as_draws.R empty draws conversion errors, lines 291->308
还没有人认领这个 Issue。
评估
调研方向
从 tests/test-as_draws.R 第 291-308 行的 'empty draws objects can be converted' 代码块开始。as_draws_list 和 as_draws_df 那两行的注释似乎对调了，并且该代码块通过 posterior::: 调用内部辅助函数。运行该测试文件，确认当前的 25 个期望全部通过。完成的标准是：测试读起来清晰，不匹配的注释已修正，且文件在相同期望下仍然通过。
由索引模型根据 Issue 内容生成。
描述
test-as_draws.R "empty draws conversion" uses hidden helpers and wants to be posterior:::hidden_helper_x, order of testing under seq_along is not the same as empty_j list creation. local_edition(3)
Perhaps:
291 test_that("empty draws objects can be converted", {
empty_draws <- list(
posterior:::empty_draws_matrix(),
posterior:::empty_draws_array(),
posterior:::empty_draws_list(),
posterior:::empty_draws_df(),
posterior:::empty_draws_rvars()
)
for (j in seq_along(empty_draws)) {
# basically just check they don't error and preserve 0 draws
empty_j <- empty_draws[[j]]
expect_equal(ndraws(as_draws_matrix(empty_j)), 0)
expect_equal(ndraws(as_draws_array(empty_j)), 0)
expect_equal(ndraws(as_draws_list(empty_j)), 0) # this is currently as_draws_df(
expect_equal(ndraws(as_draws_df(empty_j)), 0) # this is currently as_draws_list(
expect_equal(ndraws(as_draws_rvars(empty_j)), 0)
}
}) # line 308
Tests passed with 25 successes 🌈 is preferable.
- 主要语言
- R
- 星标
- 172
- 派生
- 27
- 平均合并
- 5 天 12 小时
- 30 天内合并 PR
- 5
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
stan-dev/posterior 的其他 Issue
-
难度 5/5 一周以上 新手友好度 25/100
-
Use testthat 3e可能已有人在做 @mcol 于 7 天前认领。 未关闭good first issue test
难度 3/5 1-2 天 新手友好度 65/100
-
难度 5/5 一周以上 新手友好度 35/100
-
难度 5/5 一周以上 新手友好度 25/100
-
stop using r_eff in ps_tail_length可能已有人在做 @florence-bockting 于 4 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 50/100
查看 stan-dev/posterior 的全部 Issue
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 68/100
cran-task-views/Spatial#127 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
Add two cautions to the `simulate_dyad_responses()` help可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭documentation priority: low
难度 2/5 1-3 小时 新手友好度 70/100
Pascal-Kueng/dyadMLM#179 ·
维护者通常 1 天内回复
-
Consider changes未关闭
难度 2/5 1-3 小时 新手友好度 68/100
OHDSI/OmopSketch#618 ·
-
难度 2/5 1-3 小时 新手友好度 68/100
GSK-Biostatistics/tfrmt#854 ·