Tech debt: consolidate test data from #187 and move signifDigitsEpsilon to veupathUtils
まだ誰も着手していません。
評価
- 難易度
- 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 にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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 1週間以上 初心者へのやさしさ 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 件 ·