Move `INTEGER`/`REAL`/other accessors out of loops
Mantenedores costumam responder em até 1 dia
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
- performance
Direção de pesquisa
Start by locating the loop implementations that use INTEGER, REAL, and other accessors, then determine which accessor expressions do not depend on the current loop iteration. Review the related discussion in pull request 7598. Done means the independent accessors are moved out of their loops without changing behavior; the issue does not name specific files or tests.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
We are always so picky about moving
INTEGERout of loops, but apparently here we did not care yet.
Originally posted by @ben-schwen in https://github.com/Rdatatable/data.table/pull/7598#discussion_r2702792455
It might be possible to write a linter that detects some cases where the expression in accessor(expression) does not depend on the current loop iteration.
- Linguagem predominante
- R
- Estrelas
- 3.9k
- Forks
- 1.1k
- Merge médio
- 2d 6h
- PRs com merge (30d)
- 12
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
- Tem um modelo de pull request
- Ler o 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 Rdatatable/data.table
-
as.data.table() recurses without end on a survival::Surv object (or any data.frame carrying one)Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Rdatatable/data.table#7887 ·
Mantenedores costumam responder em até 1 dia
-
test() doesn't distinguish plain NA_real_, NaNTalvez já em andamento @MichaelChirico assumiu há 87 dias. Abertaconsistency tests
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Rdatatable/data.table#7853 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
-
HAVE_LONG_DOUBLE is conditioned on but never setTalvez já em andamento @venom1204 assumiu há 531 dias. Abertainternals
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Rdatatable/data.table#6938 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
encoding fread
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Rdatatable/data.table#5179 · 8 comentários ·
Mantenedores costumam responder em até 1 dia
-
documentation programming
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Rdatatable/data.table#3199 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de Rdatatable/data.table
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Bioconductor/BiocCheck#264 ·
-
Length 1 columns are written as scalarsTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Abertabug triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
easystats/parameters#1290 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
corybrunson/ordr#91 ·