False positive FES warning for loadModel(path, callback)
Los mantenedores suelen responder en 3 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 62/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript
- Área
- developer-experience
Línea de trabajo
La advertencia proviene de las comprobaciones de argumentos del Friendly Error System para loadModel(). Empieza ahí y encuentra la rama que espera una cadena como segundo parámetro. Compárala con la corrección del PR #9286, que trataba loadModel(path, true), y haz que la forma con callback pase la misma comprobación. Está terminado cuando loadModel(path, callback) con un callback válido no registra ninguna advertencia de FES y loadModel(path, true) sigue comportándose igual que antes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.4
Web browser and version
Google Chrome 155.0.8059.40 (Official Build) (64-bit)
Operating system
Windows 11 Version 25H2 (Build 26200.9457)
Steps to reproduce this
Steps:
- Open the p5.js Web Editor.
- Upload a valid
teapot.objfile into anassetsfolder in the sketch. - Run the following sketch and check the console.
Snippet:
let teapot;
function setup() {
createCanvas(400, 400, WEBGL);
loadModel("/assets/teapot.obj", function (modelData) {
teapot = modelData;
console.log("Model loaded successfully:", modelData);
});
}
function draw() {
background(220);
orbitControl();
normalMaterial();
if (teapot) {
scale(2);
model(teapot);
}
}
Expected behavior
Calling loadModel(path, callback) with a valid callback should not produce a Friendly Error System (FES) warning.
Actual behavior
The model loads and renders successfully, and the callback executes. However, FES reports:
p5.js says: Expected string at the second parameter in loadModel().
This appears to be a false positive because the callback is a valid second argument for this overload.
Additional context
This is a follow-up to Issue #9176 and it's fix PR #9286 , which addressed the false-positive FES warning for loadModel(path, true).
The callback overload loadModel(path, callback) still produces a similar warning after that fix. I reproduced this behavior in the p5.js Web Editor using a local OBJ file.
I'd like to work on this issue if approved. Could a maintainer please review it and assign it to me if appropriate?
Screenshot of Issue Visually
- Lenguaje dominante
- JavaScript
- Estrellas
- 24.1k
- Forks
- 3.9k
- Merge medio
- 2 d 20 h
- PR fusionados (30 d)
- 37
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de processing/p5.js
-
[p5.js 2.0+ Bug Report]: touches[0].x lags one pointer event behind while draggingPosiblemente ocupada @titanstride la tomó hace 2 días. AbiertoArea:Events p5.js 2.0+
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
processing/p5.js#9305 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
loadModel() options break without fileType and ignore flipU/flipVPosiblemente ocupada @saniyaa-17 la tomó hace 2 días. AbiertoArea:WebGL p5.js 2.0+
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
processing/p5.js#9302 · 2 comentarios ·
Los mantenedores suelen responder en 3 días
-
[p5.js 2.0+ Bug Report]: key docs needs updatingPosiblemente ocupada @MannXo la tomó hace 3 días. AbiertoHelp Wanted
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
processing/p5.js#9296 · 5 comentarios · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
Add unit tests for noiseDetail()Posiblemente ocupada @harshiltewari2004 la tomó hace 5 días. AbiertoArea:Math Enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
processing/p5.js#9253 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 3 días
-
[p5.js 2.0+ Bug Report]: storeItem() validation branches are missing `return`, so bad input throws a TypeError or is stored anywayPosiblemente ocupada @iUjjwalRaj la tomó hace 13 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
processing/p5.js#9242 ·
Los mantenedores suelen responder en 3 días
Todos los issues de processing/p5.js
Issues similares
-
idnits3
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ietf-tools/idnits#363 ·
Los mantenedores suelen responder en 1 día
-
prototype(verifier_token): map disallowed warning to contraindicated before composite precedence checkPosiblemente ocupada @drk-ram la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
agentrust-io/trace-spec#582 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Medio día Aptitud para principiantes 72/100
TheOdinProject/curriculum#31479 ·
Los mantenedores suelen responder en 1 día
-
chore
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
SignalK/n2k-signalk#368 ·
Los mantenedores suelen responder en 1 día