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

Initial position estimate #30

Open
figuernd opened this issue Jul 12, 2024 · 2 comments
Open

Initial position estimate #30

figuernd opened this issue Jul 12, 2024 · 2 comments

Comments

@figuernd
Copy link

Hi Clyde,

Been using your codebase for a ROS2 educational workshop, thanks for making this public.

Is this part of this README still accurate? Not seeing where the vision position estimate or default pose is sent to ArduSub.

The [BaseController](src/base_controller.cpp) node sends `VISION_POSITION_ESTIMATE` messages to ArduSub at 5Hz.

Thanks in advance.

@clydemcqueen
Copy link
Owner

Been using your codebase for a ROS2 educational workshop, thanks for making this public.

Cool! I hope it is going well.

Not seeing where the vision position estimate or default pose is sent to ArduSub.

orca_base publishes the vision position estimate to the /mavros/vision_pose/pose topic. Mavros subscribes to this topic and publishes VISION_POSITION_ESTIMATE on MAVLink.

ext_nav_pub_ = create_publisher<geometry_msgs::msg::PoseStamped>(

ext_nav_pub_->publish(ext_nav_msg);

https://github.com/mavlink/mavros/blob/b0da849a06eb1a215b9205b92b8cf39c6d7cf88f/mavros_extras/src/plugins/vision_pose_estimate.cpp#L115

It took me a while to trace through the mavros code to see how ROS topics turn into MAVLink messages, and vice versa. It would be nice to document some of this in orca4, I suppose.

ArduPilot's DDS support is getting better, it might be possible to remove mavros entirely one day. :-)

@figuernd
Copy link
Author

@clydemcqueen thanks! That helps. My issue has been waiting for an EKF pose back from ArduSub so that go_auv() can arm. I thought maybe the EKF wasn't initializing without an initial pose but looks like that wouldn't be it.

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

No branches or pull requests

2 participants