fix: correcciones para ejecutar simulación en Jazzy#65
Open
Jcrex-Robotnik wants to merge 2 commits intoRobotnikAutomation:jazzy-develfrom
Open
fix: correcciones para ejecutar simulación en Jazzy#65Jcrex-Robotnik wants to merge 2 commits intoRobotnikAutomation:jazzy-develfrom
Jcrex-Robotnik wants to merge 2 commits intoRobotnikAutomation:jazzy-develfrom
Conversation
|
Author
|
I have analyzed the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
En estos cambios he modificado los paquetes
robotnik_simulation_bringupyrobotnik_localization, donde antes por descuido al lanzar el spawn_robot del bringup no pasaba el argumentorobotyrobot_modelsino 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 elrbkairos_plusnunca lo encontraba porque al no tener el argumento "robot" entonces nunca ubicaba esa version.Por otro lado, en el paquete
robotnik_localizationel launchlocalization.launch.pytenia 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.pyhe puesto el argumento de autostart en true, y el topic delmap_pathen bes de/robo_mapa/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/mapcon la localización funcionaba y con/robot_mapno, asi que es un arreglo temporal hasta que se encuentre el origen del fallo de los topics