Skip the key and node maps on single-row list edits?
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Attiva
- Stack tecnologico
- javascript, typescript
- Ambito
- frontend, performance
Direzione di ricerca
The issue names reconcileArrays, mapArray, and flatten, but gives no file paths or test names. Read those implementations and the existing list reconciliation tests, then check the reported size gate and benchmark setup. Done means maintainers decide which optimizations, if any, merit a PR and resolve the duplicate-key and moved-node behavior questions.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
On 500-row keyed lists, moving one row takes about 100 µs, against about 25 µs to remove one, because reconcileArrays and mapArray each build a map that a two-pointer walk doesn't need. I have a prototype that skips them for end moves, swaps and order-keeping removals, plus a flatten fast path. Is any of it worth a PR?
Solid-only A/B on the Octane fork, 3 rounds on a machine that wasn't idle, so this needs a clean board run:
| Suite | Reconcile only | All three |
|---|---|---|
js-framework-reorder |
−9.3% | −15.6% |
js-framework |
−4.2% | −5.4% (swap −52%) |
uibench |
−2.3% | −7.9% (one-row moves −59%) |
Brotli on js-framework |
+65 B | +240 B |
The existing suites pass, but I haven't written new specs. All three changes fail the size gate, at about +850 B minified on app scenarios.
Questions:
- Which of these would justify a size exception, if any?
stageEndspairs duplicate keys by position, so row-local state can land on a different duplicate. Should it decline that case?- With the reconcile change, a node that user code moved out of the list can stay out where
nextpulls it back. Is that acceptable?
- Lingua principale
- TypeScript
- Stelle
- 36.2k
- Fork
- 1.1k
- Merge medio
- 9h 58m
- PR unite (30g)
- 474
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 solidjs/solid
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
I maintainer di solito rispondono entro 1 giorno
-
1.x
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
I maintainer di solito rispondono entro 1 giorno
-
What gets passed to the `when` prop of a `Show` still gets executed twice on SSRForse già presa @brenelz l’ha presa 6 giorni fa. Aperta1.x
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
solidjs/solid#4447 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di solidjs/solid
Issue simili
-
Chip source filter and denial markers miss Storage rows when a Storage activity source is pickedApertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
omdsh-dev/DSH-better-sidebar#1141 ·
I maintainer di solito rispondono entro 2 giorni
-
desktop linux photos
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Cleanup SetupForse già presa @AlexanderKaran l’ha presa oggi. Apertadocumentation enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
e18e/framework-tracker#769 ·
I maintainer di solito rispondono entro 1 giorno
-
fix(docker): correct stale --db-path comments in fetch/git Dockerfiles and clarify git READMEApertabug v2
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
modelcontextprotocol/servers#5483 ·
I maintainer di solito rispondono entro 1 giorno