-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Test Navigation integration with robot_localization package (Kalman Filter) #40
Comments
@mhpanah - Do we still want to do this or should I close it? |
We haven't done this yet. It's a good idea to do it at some point. However, low priority at the moment. |
@rotu - I see you have been using the robot_localization package as part of your Rover demo. Are there any changes to Nav2 required or can I close this issue? If it's working, maybe you could give us a quick overview of how you connected the two? Are you using AMCL? |
There are no changes to Nav2 required, but robot_localization has no current official branch working under Dashing, so I'm using an unofficial fork https://github.com/RoverRobotics-forks/robot_localization. @klintan was also working on the port to Dashing but that seems to have stalled. Yes, I'm using it with AMCL. |
See my answer here :) |
I'm still 2 layers deep getting everything in order so we can meaningfully talk about getting R_L shipped out. The state of the ROS2 port of R_L also isn't awesome so I have plans post-ROSCon to probably sit down and re-port it from melodic after talking with Tom. Steps I'm in the middle of
|
@rotu - can you run rqt_graph and show how you have connected the inputs / outputs of the robot_localization package to Nav2? |
@rotu - I'm still interested in knowing how you got this to work, would you be willing to run rqt_graph on your running system and post back here? Even better would be a tutorial document. Do you have any bandwidth to help at all on this? |
And all my config lives here: https://github.com/RoverRobotics/openrover-ros2/blob/master/openrover_demo/launch/presence.launch.py I'm not sure what tutorial would be helpful. It's just figuring out the parameters and topic mappings and such. |
I'm just not clear on what purpose AMCL is serving vs the se_node. It looks like the map->odom /tf and the /odom topic are both being published by the se_node. If so, then is AMCL even needed? |
Without running it, it looks right to me with world_frame set to |
se_node has no absolute positioning info and doesn’t know about the map frame. |
Based on this, I think the intention of this issue, which was to prove someone can use robot_localization with nav2, has been met. I'm closing this issue! |
I’d like to keep this open until there’s an official release of RL for ROS2 that we can validate against. Right now its various folks’ forks which don’t necessarily represent a stable state of RL stable with Nav2 stable |
Fair enough, I just made you the owner of this ticket as it met my original intent already |
Is someone working on this? |
Slowly, there’s alot of long standing background tasks that move at glacial speeds. I just finally got all the ROS2 PRs merged into mainline after possibly the longest code review in history (thanks @klintan for sticking with me). Next steps are to look at Rover’s fork and see if there’s anything worth adding or just different, thoroughly test, and deploy with bloom. If you want to go through Rover’s fork (and anyone else that looks like they did substantial work on top of the ros2 branches) and PR anything that’s worth including back in mainline to RL, that would save me a bunch of time and I can skip right to release. I have a week scheduled for RL related work in 2 weeks time. The biggest time suck will be testing every nook and cranny then reenabling CI |
As an update, my PR has been merged and at a high level tested. Mabel has another PR to add the missing stuff from Melodic over to Dashing (think estimator, etc). Once that is in, we can probably do a release. We are ready, though, to test the EKF and the UKF in Dashing against Navigation2. The basic stuff is there and in mainline for this work to actually take place now. |
Shown to work as part of the ROS2 porting of RL |
We need to verify that our localization can work with the robot_localization package which has recently been ported to ROS2.
The text was updated successfully, but these errors were encountered: