Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maze solver #4

Merged
merged 6 commits into from
Mar 13, 2021
Merged

Maze solver #4

merged 6 commits into from
Mar 13, 2021

Conversation

caioaamaral
Copy link
Contributor

@caioaamaral caioaamaral commented Mar 1, 2021

Same as Brazilian-Institute-of-Robotics/doogie_base#17, you will need both these changes in order to properly build the project.

@mateusmenezes95
Copy link
Contributor

I tested both launches files and they're working well. I found a bug in the rviz interface but I will describe it better in an issue. I noted also the robot with a big jerk at the start of movement when it moves straight. Finally, I think there are so many "INFO" when running the move base node. Maybe we can change for "DEBUG" verbosity level (comment valid for New Move Base #17)

Without these notes, the simulation is working well and smoothly. I ran it a few times and I haven't any error. Great job!!!

@caioaamaral
Copy link
Contributor Author

caioaamaral commented Mar 13, 2021

Finally, I think there are so many "INFO" when running the move base node. Maybe we can change for "DEBUG" verbosity level

Good point. I've tried to reduce all ROS_INFO* calls, but would be great changing those to "DEBUG" level. Only problem I see is that we are missing an easy way to enable the debug mode (and display debug msgs), maybe would be better addressing this in a next PR. I'm raising an issue to keep track of this (#6).

@caioaamaral caioaamaral merged commit a78031f into master Mar 13, 2021
@caioaamaral caioaamaral deleted the maze_solver branch March 13, 2021 11:51
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