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

Bosch BMI055 cleanup #12065

Merged
merged 1 commit into from
May 27, 2019
Merged

Bosch BMI055 cleanup #12065

merged 1 commit into from
May 27, 2019

Conversation

dagar
Copy link
Member

@dagar dagar commented May 24, 2019

  • move to PX4Accelerometer and PX4Gyroscope helpers

@dagar dagar force-pushed the pr-bmi055_cleanup branch from 469b12c to 52e01c6 Compare May 24, 2019 19:35
@dagar
Copy link
Member Author

dagar commented May 24, 2019

Saves ~ 3.5 KB of flash.

image

 - move to PX4Accelerometer and PX4Gyroscope helpers
@dagar dagar force-pushed the pr-bmi055_cleanup branch from 52e01c6 to 9beed70 Compare May 26, 2019 17:00
@dagar
Copy link
Member Author

dagar commented May 26, 2019

@PX4/testflights can you give this a quick try on a Pixhawk 4 or Pixhawk 4 mini (flight controllers with a Bosch BMI055 IMU). Please enable "Sensor comparison" via the SDLOG_PROFILE parameter.

@dagar dagar requested a review from a team May 26, 2019 17:02
@dagar dagar marked this pull request as ready for review May 26, 2019 17:02
@dagar dagar requested a review from davids5 May 26, 2019 17:02
@dannyfpv
Copy link

dannyfpv commented May 27, 2019

Tested on Pixhawk 4 v5
Modes Tested:

Position Mode: no issue
Altitude Mode: no issue
Stabilized Mode: no issue

Procedure:
Arm in position mode, check to take off behavior (Vehicle should stay on the ground as long as the throttle stick is at the middle or below, and start to ascend smoothly as long as throttle stick as above the middle.), the land and disarm.
enable "Sensor comparison" via the SDLOG_PROFILE parameter.

Notes:
No issues were noted, good flight in general.

Logs:

https://review.px4.io/plot_app?log=c25c1c19-ad22-4cbf-9b61-eb5a9ead669f

https://review.px4.io/plot_app?log=dbeaea5f-5938-4515-aa37-c0df6ff1cdf3

https://review.px4.io/plot_app?log=f2bbc861-f20c-48c0-bd79-743ebce74634

https://review.px4.io/plot_app?log=75cfa4d7-514c-41a3-ad8e-e37886aa508a

@dagar
Copy link
Member Author

dagar commented May 27, 2019

Notes:
No issues were noted on position and altitude, but while testing stabilized mode, vehicle started to ascend slowly before reaching middle position, good flight in general.

Can you point to where? Log and roughly when.

The BMI055 data checks out, but I'd still like to investigate any side issues.

@dagar dagar merged commit f7ff82c into PX4:master May 27, 2019
@dagar dagar deleted the pr-bmi055_cleanup branch May 27, 2019 22:03
@Junkim3DR
Copy link

Tested on Pixhawk 4mini v5

Flight Card 1

Modes Tested:

  • Position Mode: Good.
  • Altitude Mode: Good.
  • Stabilized Mode: Good.

Procedure:
Arm in position mode, check take off behavior (Vehicle should stay on the ground as long as the throttle stick is at the middle or below, and start to ascend smoothly as long as throttle stick as above the middle.), the land and disarm.
Same procedure was repeated with Altitude and Stabilized mode.

Notes:
No issues were noted, good flight in general.

Flight Card 2

Modes Tested

  • Mission Plan Mode (Automated): Good.
  • RTL (Return To Land): Good.

Procedure
Arm and Take off in mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoint see landing behavior.

Note:
No issues were noted, good flight in general.

Flight Card 3

Modes Tested

  • Position Mode: Good.
  • Mission Plan Mode (Automated): Good.
  • RTL (Return To Land): Good.

Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoint see landing behavior.

Note:
No issues were noted, good flight in general.

Logs:

@dannyfpv
Copy link

@dagar, everything is working as expected.

There was an issue while commenting, comment has been updated.

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.

4 participants