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

HIGHRES_IMU attribute collision #939

Open
Domattee opened this issue Apr 17, 2024 · 2 comments
Open

HIGHRES_IMU attribute collision #939

Domattee opened this issue Apr 17, 2024 · 2 comments

Comments

@Domattee
Copy link

In MAVLink v2, HIGHRES_IMU (id 105) messages have a field called "id", which collide with the message type class attributes (also "id"). The class attribute intended to store the message id instead stores the ID of the source IMU. This causes issues when using msg.id to determine the type of message received, in my case HIGHRES_IMU messages were being treated as heartbeats. I assume the collision is present in all v20 dialects, since it is a base mavlink message, but definitely in ardupilotmega and cubepilot.
I think pymavlink message composing/parsing methods might also be affected, because I kept getting missing attribute errors trying to use msg.get_msgId() on HIGHRES_IMU messages.

@peterbarker
Copy link
Contributor

Have you tried the get_type method?

@Domattee
Copy link
Author

I used get_msgId, the issue with that was a bug on our side with bad V1 vs V2 communication.

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