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

icp201: increase startup delay (FMUv6X) #21765

Merged
merged 1 commit into from
Jun 26, 2023
Merged

icp201: increase startup delay (FMUv6X) #21765

merged 1 commit into from
Jun 26, 2023

Conversation

mrpollo
Copy link
Contributor

@mrpollo mrpollo commented Jun 26, 2023

Fixes wrong altitude reporting trough barometer on CUAV FMUv6X hardware with the ICP201XX sensor by InvenSense. The issue was first reported here #19819 (comment)

Log with issue
Screenshot 2023-06-26 at 10 52 18 AM

https://logs.px4.io/plot_app?log=a3e68174-e2e7-4b81-b47f-87e03cbc2d69

Log with fix
Screenshot 2023-06-26 at 10 52 36 AM
https://logs.px4.io/plot_app?log=921154ae-a088-4334-b899-debdc04075d5

fixes wrong altitute reporting
@mrpollo
Copy link
Contributor Author

mrpollo commented Jun 26, 2023

Hey @beniaminopozzan would you mind helping me test real quick?

@dagar
Copy link
Member

dagar commented Jun 26, 2023

A bit surprising it slips through this way, but this should be fine. The timing between configure completing and starting to take readings only needs to happen once, so it can easily be much longer.

@dagar dagar merged commit 98d8935 into main Jun 26, 2023
2 of 84 checks passed
@dagar dagar deleted the cuav_v6x_fix_icp201 branch June 26, 2023 18:16
@beniaminopozzan
Copy link
Member

@mrpollo thanks Ramon!
It works now, just tested it!

harrisondragoon pushed a commit to harrisondragoon/PX4-Autopilot that referenced this pull request Jun 30, 2023
mrpollo added a commit that referenced this pull request Jul 11, 2023
MaEtUgR pushed a commit that referenced this pull request Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants