Use the shared rotation and transformation helpers
Los mantenedores suelen responder en 9 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- numpy, python
- Área
- computer-vision, robotics
Línea de trabajo
Read src/components/common/matrix_lib.py and src/components/array/xy_array.py, then search src/components for hand-written rotations using the -sin pattern. Replace those rotations with the shared helper, simplify the translation expression, and add a docstring for rot_mat_22; done means behavior stays unchanged. Pull request #156 is already open for this issue.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
src/components/common/matrix_lib.py provides rot_mat_22 and hom_mat_33, but rotation matrices are also written by hand in other places, for example XYArray.homogeneous_transformation() in src/components/array/xy_array.py. XYArray also adds the translation with np.ones(shape) * [[x], [y]], where plain broadcasting (+ np.array([[x], [y]])) is enough. rot_mat_22 has no docstring.
Expected fix
Use rot_mat_22 in XYArray and in other hand-written rotations (find them with grep -rn "\-sin" src/components), simplify the translation, and add the missing docstring. Behavior must not change.
- Lenguaje dominante
- Python
- Estrellas
- 1.9k
- Forks
- 313
- Merge medio
- 9 d 10 h
- PR fusionados (30 d)
- 7
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin 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 ShisatoYano/AutonomousVehicleControlBeginnersGuide
-
good first issue refactoring
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ShisatoYano/AutonomousVehicleControlBeginnersGuide#129 · 1 comentario ·
Los mantenedores suelen responder en 9 días
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ShisatoYano/AutonomousVehicleControlBeginnersGuide#99 ·
Los mantenedores suelen responder en 9 días
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ShisatoYano/AutonomousVehicleControlBeginnersGuide#98 ·
Los mantenedores suelen responder en 9 días
-
documentation enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ShisatoYano/AutonomousVehicleControlBeginnersGuide#88 ·
Los mantenedores suelen responder en 9 días
-
long term
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
ShisatoYano/AutonomousVehicleControlBeginnersGuide#131 ·
Los mantenedores suelen responder en 9 días
Todos los issues de ShisatoYano/AutonomousVehicleControlBeginnersGuide
Issues similares
-
Admin member of a research group cannot see the HX2 allocation until they've been added to itAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
developmentseed/titiler-cmr#278 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 2 días