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

Add component/filter ID to AHRS messages #1441

Merged
merged 7 commits into from
Nov 25, 2015

Conversation

flixr
Copy link
Member

@flixr flixr commented Nov 25, 2015

When running multiple AHRS algos in parallel, each filters can send the same message but with a different ID to distinguish the source.
See also #1145

This adds a comp_id field to the following messages:

  • AHRS_EULER
  • AHRS_EULER_INT
  • AHRS_QUAT_INT
  • AHRS_RMAT_INT
  • AHRS_GYRO_BIAS_INT
  • GEO_MAG

Also register some more message in some of the filters and don't use the value from the state interface to send the attitude in body frame but rather compute it from the actual ahrs output (imu frame).
So it will still be correct if muliple AHRS are running and another one is enabled (sets the attitude in the state interface).

To be able to distinguish from which implementation/component the messages was set (needed if running multiple AHRS)
see paparazzi#1145

Still need possibility to register a message multiple times (from different modules).
float cmpl and mlkf also register AHRS_EULER and AHRS_GYRO_BIAS_INT
when multiple AHRS are running, the body attitude in state interface doesn't have to be the one from the current AHRS
@flixr flixr added this to the v5.8 milestone Nov 25, 2015
@gautierhattenberger
Copy link
Member

I guess this one is okay now

gautierhattenberger added a commit that referenced this pull request Nov 25, 2015
Add component/filter ID to AHRS messages
@gautierhattenberger gautierhattenberger merged commit a470c92 into paparazzi:master Nov 25, 2015
@flixr flixr deleted the ahrs_comp_id branch November 25, 2015 16:07
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.

None yet

2 participants