-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Mag current/throttle compensation #14457
Conversation
Documentation has started as well: PX4/PX4-user_guide#692 |
This solves the effect of the current generated by the thrust change on the geomagnetic sensor due to the improper layout of the circuit board.Right? 😃 |
d8b41ae
to
f6496ff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is something we can still add to the release. Any objections?
New feature to compensate compass for current/throttle:
Parameters for each axis have to be identified offline using the provided python script. If there is no current reading available, there is the option to compensate for thrust, only advisable if the relationship is linear for the given setup (In my tests I've observed that for some setups the relationship is linear and it works well, for others it is not).
Offline parameter identification:
Log from drone with parameters for current compensation applied:
https://logs.px4.io/plot_app?log=cf3ca047-ea1b-43c0-b8dd-a425bea539d4