[p5.js 2.0+ Bug Report]: randomGaussian() leaks its spare value across p5 instances, breaking seeded determinism
I maintainer di solito rispondono entro 2 giorni
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 38/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- javascript
- Ambito
- frontend
Direzione di ricerca
Inizia in src/math/random.js e ispeziona il test di indipendenza esistente e saltato per la modalità istanza. Riproduci le chiamate intercalate con seed usando due istanze di p5, quindi esegui i test rilevanti sui numeri casuali. Il lavoro è completato quando le istanze producono sequenze deterministiche indipendenti e il test di regressione ha esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Most appropriate sub-area of p5.js?
Math
p5.js version
2.x main (4b096e2)
Actual vs expected behavior
The randomSeed() reference promises that a constant seed "makes these functions produce the same results each time a sketch is run", explicitly naming randomGaussian(). That breaks as soon as two p5 instances exist on a page.
randomGaussian() uses the Marsaglia polar method, which produces values in pairs. The flag that says "a spare value is cached" lives on the instance (this._gaussian_previous), but the spare value itself lives in module scope (let y2 = 0; in src/math/random.js). With two instances, B's spare overwrites A's, and A's next call returns B's value.
Executed repro (both instances seeded, one interleaved call on instance B):
A alone : -0.2050, -0.3410, 0.3986, -1.4309
A interleaved: -0.2050, -0.9404, 0.3986, -1.4309
Same seed, different sequence. The existing test suite even contains a skipped "instance mode / should be independent" block asserting exactly this independence. PR #1681 (2015) made _gaussian_previous per-instance but left y2 module-scoped, which is the remaining half of that fix.
Steps to reproduce
Outputs above are from executed runs against main, two instances sharing the prototype as in instance mode.
Note
I have a fix ready (store the spare as this._gaussian_y2) with a regression test, mutation-tested against main. Filing for approval first per the contributing guide; will open the PR once approved.
- Lingua principale
- JavaScript
- Stelle
- 24.1k
- Fork
- 3.9k
- Merge medio
- 3g 10h
- PR unite (30g)
- 30
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 processing/p5.js
-
[p5.js 2.0+ Bug Report]: SVG importer does not respect preserveAspectRatio="none" for <symbol>/<use>Forse già presa @Danyccsf l’ha presa 3 giorni fa. ApertaArea:Core p5.js 2.0+
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
processing/p5.js#9257 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 2 giorni
-
Add unit tests for noiseDetail()Forse già presa @Pcmhacker-piro l’ha presa 4 giorni fa. ApertaArea:Math Enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
processing/p5.js#9253 ·
I maintainer di solito rispondono entro 2 giorni
-
[p5.js 2.0+ Bug Report]: storeItem() validation branches are missing `return`, so bad input throws a TypeError or is stored anywayForse già presa @iUjjwalRaj l’ha presa 5 giorni fa. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
processing/p5.js#9242 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
processing/p5.js#9241 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
[p5.js 2.0+ Bug Report]: Typo in Spanish reference documentation for ellipseMode()Forse di nuovo libera @cgutierrezval l’ha presa 8 giorni fa e non c’è nessuna pull request aperta. ApertaInternationalization p5.js 2.0+
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
processing/p5.js#9231 · 3 commenti ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di processing/p5.js
Issue simili
-
[dsh-plugin.org | dsh-plugin-hub] plugin distribution incomplete: yjh051108/dsh-routing-suiteAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 71/100
yjh051108/dsh-routing-suite#227 ·
-
needs-triage release-watch
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
dusk-network/exu#17 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
jspreadsheet/ce#1809 ·
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
githubnext/gh-aw-workshop#4458 ·
I maintainer di solito rispondono entro 1 giorno