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

AP_OpticalFlow: mavlink driver should consume OPTICAL_FLOW message's flow_rate_x, flow_rate_y #29131

Open
rmackay9 opened this issue Jan 24, 2025 · 0 comments

Comments

@rmackay9
Copy link
Contributor

rmackay9 commented Jan 24, 2025

The OpticalFlow_MAV driver consumes the OPTICAL_FLOW message's "flow_x" and "flow_y" fields which are int16_t. Instead we should consume the "flow_rate_x" and "flow_rate_y" extension fields which are floats and will thus give better accuracy.

We should also consume the "ground_distance" field

We should also consume the OPTICAL_FLOW_RAD. I'm not sure why both messages exist but perhaps this one was created specifically for flow sensors to send their readings to the autopilot while OPTICAL_FLOW was meant for autopilots to send their flow data to the GCS? I'm not sure why we'd need to separate the uses though

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

No branches or pull requests

1 participant