loadModel() options break without fileType and ignore flipU/flipV
I maintainer di solito rispondono entro 3 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- javascript
- Ambito
- computer-graphics
Direzione di ricerca
L'analisi delle opzioni per loadModel(path, options) si trova in src/webgl/loading.js intorno alle righe 573-581; la issue indica che fileType viene riassegnato prima che vengano letti flipU e flipV, quindi l'oggetto options può arrivare a .toLowerCase(). Inizia leggendo quel blocco, poi riproduci con loadModel('model.obj', { flipU: true }) e con fileType incluso. È completato quando entrambe le forme di chiamata caricano senza errori e le opzioni di flip vengono applicate al modello caricato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Most appropriate sub-area of p5.js?
- Accessibility
- Color
- Core/Environment/Rendering
- Data
- DOM
- Events
- Image
- IO
- Math
- Typography
- Utilities
- WebGL
- WebGPU
- p5.strands
- Build process
- Unit testing
- Internationalization
- Friendly errors
- Other (specify if possible)
p5.js version
2.3.2 / current main
Web browser and version
Chromium 151
Operating system
Fedora Linux
Steps to reproduce this
The documented loadModel(path, options) overload breaks depending on whether fileType is included.
async function setup() {
createCanvas(100, 100, WEBGL);
// Throws: fileType.toLowerCase is not a function
await loadModel('model.obj', { flipU: true });
// Loads, but flipU is ignored
await loadModel('model.obj', {
fileType: '.obj',
flipU: true
});
}
Options like { normalize: true } or {} also throw when fileType is omitted. With fileType present, flipU / flipV are ignored, although calling model.flipU() / model.flipV() afterwards works.
Expected: loadModel(path, options) should work without requiring fileType, and the flip options should be applied.
I traced it to the options parsing here:
fileType is reassigned before flipU / flipV are read, so without fileType the options object eventually reaches .toLowerCase(), while with it the flags are read from a string.
- Lingua principale
- JavaScript
- Stelle
- 24.1k
- Fork
- 3.9k
- Merge medio
- 2g 22h
- PR unite (30g)
- 38
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]: key docs needs updatingForse già presa @MannXo l’ha presa oggi. ApertaHelp Wanted
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 62/100
processing/p5.js#9296 · 5 commenti · 1 reazione ·
I maintainer di solito rispondono entro 3 giorni
-
False positive FES warning for loadModel(path, callback)Forse già presa @xsupremeyx l’ha presa 1 giorno fa. Apertap5.js 2.0+
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
processing/p5.js#9295 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 3 giorni
-
Add unit tests for noiseDetail()Forse già presa @harshiltewari2004 l’ha presa 3 giorni fa. ApertaArea:Math Enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
processing/p5.js#9253 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 3 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 10 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 3 giorni
-
[p5.js 2.0+ Bug Report]: getURLParams() mis-parses valueless parameters and never percent-decodes valuesForse già presa @mayanksingh-7974 l’ha presa 5 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
processing/p5.js#9241 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di processing/p5.js
Issue simili
-
data
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 74/100
gaseous-project/hasheous#637 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
gramps-project/gramps-web#1549 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
change request evaluating
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
sindresorhus/eslint-plugin-unicorn#3907 ·
I maintainer di solito rispondono entro 1 giorno
-
Priority: Low Production Type: Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
GrottoCenter/grottocenter-api#1903 ·
I maintainer di solito rispondono entro 1 giorno