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 MPU6000 to hwdef for ReaperF745 to Support V4 FC #27686

Merged

Conversation

ericjohnson97
Copy link
Contributor

@@ -121,6 +121,7 @@ define HAL_LOGGING_DATAFLASH_ENABLED 1
# spi devices bmi270(v2) icm42688 (v3)
SPIDEV bmi270 SPI3 DEVID1 GYRO_CS MODE3 1*MHZ 4*MHZ
SPIDEV icm42688 SPI3 DEVID1 GYRO_CS MODE3 2*MHZ 8*MHZ
SPIDEV mpu6000 SPI3 DEVID1 GYRO_CS MODE3 1*MHZ 4*MHZ
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4Mhz is pretty slow. I would stick with 8Mhz if that works ok

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry to go dark. I copied that line from another board. I'll try the IMU at 8k. I haven't gotten my build air born yet, but I will report back after I am able to flight test.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ping @ericjohnson97 how'd the flight test go?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've run into a number of issues that I haven't worked through yet. First, I took my new drone out to test at the beginning of august in Texas and after a bit of time in the sun and on the hot concrete the 3d printed frame started melting.
😂

I also was unable to get a GPS lock even though the board reported detecting a ublox gps. I reprinted my drone in ASA and went to a GPS module i've had success with in the past. I took it on the roof of my apartment to see if I could get a GPS lock this weekend and unfortunately it still was not getting a GPS lock even though once again it detected a ublox GPS.

This time I did notice that after powering on the drone the estimated velocity was growing bigger and bigger over time. I re-did all of the calibration, but no luck. I was going to reflash it and start the setup from scratch again, but I've been a little busy.

Let me know if you have any advice for those issues. I'll report beck once I hopefully get my aircraft in the air.

here are some pics of my setup for context
20240802_231510
20240802_231600
20240802_231437
20240731_235845

I'm using OpenHD for video and telemetry.

@andyp1per
Copy link
Collaborator

Without GPS and in heat you will likely get an accel bias which will cause your velocity to increase, we make some attempt to compensate for this but a GPS is your best answer. No idea why you would not get GPS lock.

@peterbarker
Copy link
Contributor

For your GPS, you can:

  • for testing purposes unclip it from its holder and stretch it out as far from the other electronics as you can - does that get lock? If so, you've got interference problems.
  • check the voltage to the GPS unit. They really do not like to be undervolted!
  • add a chunky copper groundplane underneath the GPS (unetched circuit board works well for this)
  • check the accel biases in the dataflash logs (https://ardupilot.org/copter/docs/logmessages.html#xkf2)
  • see if the velocity problems occur when the vehicle is kept at a constant temperature
  • consider doing an IMU temperature calibration (https://ardupilot.org/copter/docs/common-imutempcal.html)

@andyp1per is there any way that we could become happy with this PR without flight? Any amount of ground manipulation we could use instead?

@ericjohnson97
Copy link
Contributor Author

@peterbarker those all good pieces of advice. I'll try those steps

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Sep 5, 2024
@andyp1per
Copy link
Collaborator

I don't think these issues can be anything to do with the IMU, I think we should just merge this @peterbarker

@peterbarker peterbarker force-pushed the update/eric/reaperf745V4update branch from af2ebd9 to 361b78a Compare September 6, 2024 09:52
@peterbarker
Copy link
Contributor

I've fixed the commit message and marked for MergeOnCIPass

Thanks!

@peterbarker peterbarker merged commit a1a4c63 into ArduPilot:master Sep 6, 2024
42 checks passed
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