-
Notifications
You must be signed in to change notification settings - Fork 18.6k
AHRS: fixed initial attitude for non-zero AHRS_ORIENTATION #29667
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
Conversation
for testing non-level flight controllers
we would update the orientation in the IMUs 1s after startup, leaving a short time when the data was processed incorrectly
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.
LGTM
this is included in 4.6.0-beta6 |
@tridge
Results:
Here is a log of it happening after boot. I have also confirmed that cherry picking this commit back to 4.5.7 also gives the same results. |
I've reviewed @dallinbriggs's log and also reproduced it on my own CubeOrange. I've created an issue to capture the problem here #30064 This is what I see with 4.6.0-beta6 and 4.7.0-dev using Copter: Strangely 4.5.7 has very similar behaviour but it cannot be captured in the logs, perhaps because logging starts later? Dallin's log shows slightly different behaviour with the roll-over to the correct attitude taking much longer. Maybe this is because DCM is the active attitude estimator in Plane? EDIT: yes, the issue appears to be worse on Plane. here is the output from my test with Plane-4.7.0 and it looks like Dallin's |
we need to get the orientation right before any data is read