Move `INTEGER`/`REAL`/other accessors out of loops
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Refactoring
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- r
- Bereich
- performance
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- R
- Sterne
- 3.9k
- Forks
- 1.1k
- Ø Merge
- 15 Std. 51 Min.
- Gemergte PRs (30 T.)
- 3
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Rdatatable/data.table
-
as.data.table() recurses without end on a survival::Surv object (or any data.frame carrying one)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Rdatatable/data.table#7887 ·
-
test() doesn't distinguish plain NA_real_, NaNEvtl. vergeben @MichaelChirico hat das vor 69 Tagen übernommen. Offenconsistency tests
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Rdatatable/data.table#7853 · 3 Kommentare ·
-
HAVE_LONG_DOUBLE is conditioned on but never setEvtl. vergeben @venom1204 hat das vor 513 Tagen übernommen. Offeninternals
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Rdatatable/data.table#6938 · 1 Kommentar ·
-
encoding fread
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Rdatatable/data.table#5179 · 8 Kommentare ·
-
documentation programming
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Rdatatable/data.table#3199 · 3 Kommentare ·
Alle Issues in Rdatatable/data.table
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
bug code cleaning
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
furrer-lab/abn#272 ·
-
R CMD check red on main: load_* loader tests fail (2026-season release assets not published yet)Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 69/100
sportsdataverse/hoopR#227 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
philchalmers/SimDesign#106 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
hubverse-org/hubData#157 ·