Check the generated C code of analytic IK against MISRA C
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
- 48/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Área
- embedded-iot, robotics
Línea de trabajo
On the feat/symbolic-ik branch, read src/roboticstoolbox/robot/symbolic_ik.py and symbolic_ik_rr.py; use docs/notebooks/ik-symbolic.ipynb to generate the Puma and Jaco C files. Run a MISRA C checker, record violations by rule, and identify inexpensive generator fixes. Done when cheap fixes are made and remaining deviations are documented.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Analytic inverse kinematics can write its solution as C code that needs no library: robot.analytic_ik().generate("c", "my_ik"). This is on the branch feat/symbolic-ik and is not yet merged. We would like to know how close that C code is to MISRA C, since it is meant for embedded use, and to fix the generator where that is cheap.
What we know. I checked the generated C for the Puma 560, Stanford, UR5 and Jaco:
- no
malloc,calloc,reallocorfree - no input or output, and no
goto,setjmporlongjmp - no library other than
math.h, andstring.hfor the general 6R code - fixed-size working arrays, and the caller provides the array for the solutions
- no recursion and no variable-length arrays
- no warnings with
cc -std=c99 -pedantic -Wall -Wextra -Wvla
What we don't know. The code has not been checked against MISRA C:2012 or any other coding standard, so we say only that it avoids dynamic memory, not that it is compliant. The C++ output returns a std::vector, which allocates, and is out of scope here.
Help wanted:
- run a MISRA checker on the generated Puma and Jaco files (for example cppcheck with its MISRA addon, or a commercial tool)
- list the violations by rule, and say which look cheap to fix in the generator, for example floating-point equality,
NAN, multiple returns, or implicit conversions - fix those, and document the deviations that we keep
The generator is symbolic_ik.py and symbolic_ik_rr.py in src/roboticstoolbox/robot/ on that branch. The notebook docs/notebooks/ik-symbolic.ipynb shows how to generate the code.
🤖 Generated with Claude Code
- Lenguaje dominante
- C++
- Estrellas
- 3.5k
- Forks
- 639
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 113
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 petercorke/robotics-toolbox-python
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
petercorke/robotics-toolbox-python#709 ·
Los mantenedores suelen responder en 1 día
-
enhancement help wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
petercorke/robotics-toolbox-python#825 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
feat(ets): numerical inverse kinematics for planar ETS2Posiblemente ocupada @qwist1233-cpu la tomó hace 4 días. Abiertoenhancement help wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
petercorke/robotics-toolbox-python#824 ·
Los mantenedores suelen responder en 1 día
-
Rigorous testing of operational-space methodsPosiblemente ocupada @kVinsom la tomó hace 2 días. Abiertotech-debt Tests needed
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
petercorke/robotics-toolbox-python#804 ·
Los mantenedores suelen responder en 1 día
-
tech-debt
Dificultad 5/5 Más de una semana Aptitud para principiantes 10/100
petercorke/robotics-toolbox-python#803 ·
Los mantenedores suelen responder en 1 día
Todos los issues de petercorke/robotics-toolbox-python
Issues similares
-
v9.6.0: CMake incorrectly enforces presence of libevdevPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
mltframework/shotcut#1943 ·
Los mantenedores suelen responder en 1 día
-
Rowan (2017) electron heating gives NaN for beta > beta_max; clip() then sets electrons to kel_minAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
parthenon-hpc-lab/kharma#213 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
metasequoiaime/msime#9113 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
tone-3000/tone3000-plugin#257 · 1 comentario ·
Los mantenedores suelen responder en 1 día