Rigging follow-ups from the #365 review: f32 range for scale and manipulation, HashSet in free_joint_name

Abierto
#396 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
55/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
rust

Línea de trabajo

Read the rigging and scene edit paths around edit_node, manipulation, and free_joint_name, along with the known-issues entry and existing index-count tests. First trace how #365 handles f32 range for joint positions and how free joint names are checked. Done means scale and manipulation are covered by a known issue or typed error with a test, and name generation uses a HashSet with a scaling test.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

priority/p3 type/bug urgency/this-quarter

Follow-ups from claude-fleet's full review of #365 (rigging joints) at 7e9f726, both low:

  1. Widen the f32 known-issue. #365 refuses joint positions that leave f32 range: local, relative world and absolute. A transform edit can still push a node's world position out of f32 range in two ways: a scale edit (edit_node) and a manipulation (Move, Rotate, Scale). Every view build then fails until Undo. Extend the known-issues entry to cover scale and manipulation, or refuse those edits with a typed error and a test, as create_joint now does.
  2. free_joint_name scans linearly. It finds a free default joint name by testing each candidate against the scene. Use a HashSet of the existing names, so generating a name in a large scene is not quadratic. Add a scaling test in the style of the existing index-count tests.

Refs #363, #365

Lenguaje dominante
Rust
Estrellas
4
Forks
2
Merge medio
1 d 5 min
PR fusionados (30 d)
213

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de cristim/AztecCraft

Todos los issues de cristim/AztecCraft

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.