Tech debt: consolidate test data from #187 and move signifDigitsEpsilon to veupathUtils
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 35/100
- Issue 类型
- 重构
- 描述清晰度
- 需要澄清
- 活跃度
- 停滞
- 技术栈
- r
- 领域
- testing-qa, tooling
调研方向
Start with the discussion and review on pull request #187, then inspect the existing test object around line 25, including testDF and the tac data. Trace signifDigitEpsilon and the veupathUtils integration, noting that the function is not currently exported from plot.data. Done means consolidating the test data and relocating the utility without losing the behavior described in the referenced review.
由索引模型根据 Issue 内容生成。
描述
@asizemore suggested here
is there a reasonable way we can modify the existing test object (like in line 25) to fail in the same way? Ideally we've been trying to keep the test data to a minimum set that can be used or manipulated for nearly all the tests. Or if we could pick out 500 values of the tac data and add them as a new column contE or something like that in testDF that would similarly prevent test data object sprawl, if possible.
also see this regarding the function
(note that the signifDigitEpsilon is not currently exported from plot.data)
- 主要语言
- R
- 星标
- 1
- 派生
- 0
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
VEuPathDB/plot.data 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 25/100
-
难度 3/5 1-2 天 新手友好度 35/100
-
难度 3/5 1-2 天 新手友好度 35/100
-
难度 5/5 一周以上 新手友好度 25/100
-
empty two by two 未关闭
难度 4/5 3-5 天 新手友好度 35/100
查看 VEuPathDB/plot.data 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
robjhyndman/forecast#1220 ·
-
难度 2/5 1-3 小时 新手友好度 65/100
JamesHWade/deputy#192 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
-
bug triage_needed
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 72/100
pharmaverse/rtables#1123 · 1 条评论 · 1 个 reaction ·