fix: Stop overloading `d.location` for vertex `shaderLocation`
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Tranquilo
- Stack tecnológico
- typescript
- Área
- computer-graphics
Línea de trabajo
Empieza rastreando cómo fluye d.location a través de arrayOf() y vertexLayout, comparando el manejo de miembros de struct o IO con el manejo de vertex shaderLocation. El trabajo estará terminado cuando los decoradores tengan un solo propósito, vertexLayout siga recibiendo shaderLocation y arrayOf() ya no necesite un caso especial para @location.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
fix: Stop overloading d.location for vertex shaderLocation
In WGSL, decorators like @size or @align can only decorate struct members.
Issue #2754 pointed out that we could pass decorators to arrayOf(), which usually had no effect and was misleading.
PR #2781 partially fixed this by rejecting non-@location decorations on array element types. @location remains allowed as an exception, because it is used with vertexLayout.
Problem
d.location currently means two different things depending on context:
- On a struct / IO member, it maps to the WGSL
@locationattribute. - In a
vertexLayout, decorating an array element withd.locationsets the pipeline’sshaderLocationattribute.
Moreover, due to this exception, d.location can always be passed to arrayOf(), which might be misleading.
Proposal
Make decorators single-purpose and find another way to pass shaderLocation to the vertexLayout, so arrayOf() no longer needs a special case for @location.
- Lenguaje dominante
- TypeScript
- Estrellas
- 3.3k
- Forks
- 140
- Merge medio
- 3 d 6 h
- PR fusionados (30 d)
- 74
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 software-mansion/TypeGPU
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
software-mansion/TypeGPU#3276 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
software-mansion/TypeGPU#3219 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
texture.write(VideoFrame) uses the padded coded size instead of the visible sizePosiblemente ocupada @RaphaelFakhri la tomó hace 7 días. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
software-mansion/TypeGPU#3213 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
impr: Error message in examplesAbiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
software-mansion/TypeGPU#3008 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
software-mansion/TypeGPU#2948 · 2 comentarios ·
Los mantenedores suelen responder en 2 días
Todos los issues de software-mansion/TypeGPU
Issues similares
-
New icon: `filter-clear`Abiertocalcite-ui-icons enhancement impact - p3 - not time sensitive needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Esri/calcite-design-system#15467 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
diegosouzapw/OmniRoute#18390 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 86/100
guilhermerodz/input-otp#157 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
davidjerleke/embla-carousel#1414 ·
Los mantenedores suelen responder en 4 días