search

ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
10.99k stars 17.53k forks source link

BLH_Poles with multiple motor types #9982

Open Naterater opened 5 years ago

Naterater commented 5 years ago

Feature request

Although the BLH_Poles works well for configurations with exactly the same motors, some setups may use different motors with different poll counts. As such, I request adding to add the flexibility to assign different poll counts to different motors. Especially useful in plane where forward propulsion may be different motor than vertical lift.

Describe the solution you'd like SERVO_BLH_POLE1 = First BLHeli Motor Pole count SERVO_BLH_POLEn = n'th BLHeli Motor Pole count SERVO_BLH_POLE16 = 16th BLHeli Motor Pole count

Maybe these parameters could be "disabled" or hidden until the SERVO_BLH_MASK has been enabled for a specific number of motors.

Describe alternatives you've considered Doing the math for each motor that has a different number of poles.

Platform [ ] All [ ] AntennaTracker [X] Copter [X] Plane [X] Rover [ ] Submarine

vierfuffzig commented 5 years ago

as mentioned by @WickedShell in respective PR: https://github.com/ArduPilot/ardupilot/pull/9590 i'm still not sure if per-instance settings would really integrate well with other BLH functions.

basti