loadModel() options break without fileType and ignore flipU/flipV

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#9,302 3 commentaires 0 réactions 1 personne assignée Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

@bababananaa y travaille déjà.

Depuis le 27/10/2026.

  • #9371 par @bababananaa — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
78/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
javascript
Domaine
computer-graphics

Piste de recherche

Le traitement des options pour loadModel(path, options) se trouve dans src/webgl/loading.js, autour des lignes 573-581 ; le ticket indique que fileType est réassigné avant que flipU et flipV ne soient lus, de sorte que l'objet d'options peut atteindre .toLowerCase(). Commence par lire ce bloc, puis reproduis avec loadModel('model.obj', { flipU: true }) et avec fileType inclus. C'est terminé lorsque les deux formes d'appel se chargent sans erreur et que les options de retournement sont appliquées au modèle chargé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Area:WebGL p5.js 2.0+
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:

https://github.com/processing/p5.js/blob/24a68d5c19eb70931c9a08c41b43a658978835f3/src/webgl/loading.js#L573-L581

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.

Langage dominant
JavaScript
Étoiles
24.3k
Forks
3.9k
Merge moyen
3 j 14 h
PR mergées (30 j)
49

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de processing/p5.js

Toutes les issues de processing/p5.js

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.