Tech debt: consolidate test data from #187 and move signifDigitsEpsilon to veupathUtils
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- r
- Ambito
- testing-qa, tooling
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
@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)
- Lingua principale
- R
- Stelle
- 1
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di VEuPathDB/plot.data
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 25/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
-
empty two by two Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Tutte le issue di VEuPathDB/plot.data
Issue simili
-
bug triage_needed
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
pharmaverse/rtables#1123 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
-
Trying to reassign a reactiveVal to the name of an existing reactiveVal doesn't overwrite the latter Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100