[p5.js 2.0+ Bug Report]: TRIANGLES Requires Multiples of 3
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 66/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript
- Área
- frontend
Línea de trabajo
Empieza reproduciendo el caso informado en el sketch del editor de p5.js enlazado: llama a beginShape(TRIANGLES), añade cuatro vértices y llama a endShape(). Compara el resultado con p5.js 1, donde se ignoraba el vértice adicional. El trabajo estará terminado cuando p5.js 2 gestione el vértice adicional sin un error de ejecución y dibuje el triángulo completo.
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
No response
Operating system
No response
Steps to reproduce this
Steps:
Creating a custom shape using the beginShape(TRIANGLES) option in v2.0+ requires the number of vertices to be exactly a multiple of 3 or it will hit a runtime error and not draw anything. The error is not a particularly friendly one: TypeError: can't access property "position", v1 is undefined
In p5 V1, extra vertices that could not be wrangled into a triangle (e.g. a shape with 4 vertices) were simply ignored.
I think the V1 method makes a lot more sense, and we should go back to ignoring extra vertices. This would also maintain some continuity for people porting V1 code to V2.
Snippet:
https://editor.p5js.org/lewi0622/sketches/lnZLG0GhN
beginShape(TRIANGLES);
vertex(random(width), random(height));
vertex(random(width), random(height));
vertex(random(width), random(height));
//uncommment fourth vertex to see issue in v2+, not in an issue in v1
// vertex(random(width), random(height));
endShape();
- Lenguaje dominante
- JavaScript
- Estrellas
- 24.2k
- Forks
- 3.9k
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 35
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
-
Area:WebGL p5.js 2.0+ p5.strands
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
processing/p5.js#9317 · 4 comentarios ·
Los mantenedores suelen responder en 2 días
-
Area:DOM Good First Issue p5.js 2.0+
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
processing/p5.js#9316 · 4 comentarios · 1 reacción ·
Los mantenedores suelen responder en 2 días
-
[p5.js 2.0+ Bug Report]: touches[0].x lags one pointer event behind while draggingQuizá libre de nuevo @titanstride la tomó hace 8 días y no hay ningún pull request abierto. 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 2 días
-
loadModel() options break without fileType and ignore flipU/flipVQuizá libre de nuevo @saniyaa-17 la tomó hace 8 días y no hay ningún pull request abierto. 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 2 días
-
[p5.js 2.0+ Bug Report]: key docs needs updatingQuizá libre de nuevo @MannXo la tomó hace 9 días y no hay ningún pull request abierto. AbiertoHelp Wanted
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
processing/p5.js#9296 · 6 comentarios · 1 reacción ·
Los mantenedores suelen responder en 2 días
Todos los issues de processing/p5.js
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Yomguithereal/mnemonist#247 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
up-for-grabs/up-for-grabs.net#6330 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
TheOdinProject/curriculum#31490 ·
Los mantenedores suelen responder en 1 día
-
bug require-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
expressjs/expressjs.com#2549 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Docs: no path for someone who just wants to try IndiekitPosiblemente ocupada @rmdes la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
getindiekit/indiekit#1028 ·
Los mantenedores suelen responder en 1 día