`mergeBufferGeometries` give error for empty array
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 42/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- threejs, typescript
- Área
- computer-graphics
Línea de trabajo
Comienza en src/utils/BufferGeometryUtils.ts, en mergeBufferGeometries, y reproduce la llamada con un array vacío que provoca el TypeError por un índice undefined. Compara la implementación upstream enlazada de three.js, determina el comportamiento previsto para una entrada vacía y asegúrate de que el comportamiento en tiempo de ejecución y la firma de TypeScript comuniquen ese resultado de forma coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
threeversion: 0.160@types/threeversion:three-stdlibversion:
Problem description:
mergeBufferGeometries gives error Uncaught TypeError: Cannot read properties of undefined (reading 'index')
It is from some sloppy list access
which also exists in three.js upstream
Relevant code:
Suggested solution:
I generally would return some reasonable identity object for such cases than raising sloppy errors.
Maybe null can be reasonable.
Or otherwise, it may be useful to denote the types as [BufferGeometry, ...BufferGeometry[]] instead of BufferGeometry[],
where typescript can just warn users that users should pass array with at least one element.
- Lenguaje dominante
- JavaScript
- Estrellas
- 858
- Forks
- 144
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 pmndrs/three-stdlib
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
pmndrs/three-stdlib#423 ·
-
SketchUp (.skp) loaderAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 58/100
pmndrs/three-stdlib#437 ·
-
Automated releases are failingAbiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
pmndrs/three-stdlib#435 ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
pmndrs/three-stdlib#426 · 1 comentario · 1 reacción ·
-
Update MeshOpt v1Abiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
pmndrs/three-stdlib#425 ·
Todos los issues de pmndrs/three-stdlib
Issues similares
-
curriculum documentation quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
githubnext/gh-aw-workshop#3897 ·
Los mantenedores suelen responder en 2 días
-
agent/quality hive/hosted-available-lke648397-260827-5n31 quality testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 91/100
Los mantenedores suelen responder en 1 día
-
Add: RSV Honduras FeedAbiertocheck:failed feeds:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
iptv-org/database#36179 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[Bug] : Main navigation "Upcoming Events" link navigates to /events/ without query parameterAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día