Tech debt: consolidate test data from #187 and move signifDigitsEpsilon to veupathUtils
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Refatoração
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Estagnada
- Stack de tecnologia
- r
- Domínio
- testing-qa, tooling
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
@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)
- Linguagem predominante
- R
- Estrelas
- 1
- Forks
- 0
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Sem guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de VEuPathDB/plot.data
-
Need to look into exports of `toJSON`Talvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 25/100
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
-
empty two by twoAberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
Todas as issues de VEuPathDB/plot.data
Issues semelhantes
-
Add smvr alongside semverAberta
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
-
Typos in schema fileAberta
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 65/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 69/100
Merck/metalite.ae#261 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
[R][Parquet] ParquetReaderProperties does not expose page_checksum_verification, so a corrupt page CRC reads silentlyTalvez livre de novo Um pull request para esta issue foi fechado sem ser mesclado. AbertaComponent: R Type: enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
apache/arrow#53138 · 1 comentário ·
Mantenedores costumam responder em até 1 dia