Skip to content

fix: correcciones para ejecutar simulación en Jazzy#65

Open
Jcrex-Robotnik wants to merge 2 commits intoRobotnikAutomation:jazzy-develfrom
Jcrex-Robotnik:jazzy-devel
Open

fix: correcciones para ejecutar simulación en Jazzy#65
Jcrex-Robotnik wants to merge 2 commits intoRobotnikAutomation:jazzy-develfrom
Jcrex-Robotnik:jazzy-devel

Conversation

@Jcrex-Robotnik
Copy link
Copy Markdown

En estos cambios he modificado los paquetes robotnik_simulation_bringup y robotnik_localization, donde antes por descuido al lanzar el spawn_robot del bringup no pasaba el argumento robot y robot_model sino que solamente estaba "robot_model" por lo cual en la mayoría de casos donde los robot solo tenían un modelo no fallaba, pero cuando probaba el rbkairos_plus nunca lo encontraba porque al no tener el argumento "robot" entonces nunca ubicaba esa version.

Por otro lado, en el paquete robotnik_localization el launch localization.launch.py tenia un error que lanzaba tanto el mapping como la localización a la vez, lo cual provocaba conflictos si querías hacer una localización con un mapa ya construido, porque lo sobrescribir, así que modifique para que compruebe si indicaste que quieres hacer mapping y no usar un mapa ya hecho.

Ademas en el launch de mapping_2d.launch.py he puesto el argumento de autostart en true, y el topic del map_path en bes de /robo_map a /map, este ultimo cambio puede que al final no sea necesario y quizas no es lo que busquen pero mientras probaba el programa si usaba el topic /map con la localización funcionaba y con /robot_map no, asi que es un arreglo temporal hasta que se encuentre el origen del fallo de los topics

@asoriano1 asoriano1 self-assigned this Mar 26, 2026
@asoriano1 asoriano1 self-requested a review March 26, 2026 10:58
@asoriano1
Copy link
Copy Markdown

  • check if "has_arm" has any consecuence in the code
  • "autostart" of mapping should be false by default

@Jcrex-Robotnik
Copy link
Copy Markdown
Author

I have analyzed the has_arm argument and found no indication that it is being used in any of the package's files or programs; it appears to be a deprecated or currently unused argument. Based on the limited information available, it seems it was intended to enable or disable the arm controllers, but I haven't found any .xacro, .launch, or other files containing an if condition that evaluates has_arm to control the activation of the arm controller.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants