mavlink icon indicating copy to clipboard operation
mavlink copied to clipboard

Add HEXADECAROTOR

Open muramura opened this issue 3 years ago • 9 comments

Add 16 rotor definition for OCTA-X16 COPTER. 18 rotor to be added for VOLOCOPTER.

https://github.com/ArduPilot/ardupilot/issues/21154

HexadecaOcta ( OCTA-X16) Copter Screenshot from 2022-07-14 06-05-45

VOLOCOPTER: Screenshot from 2022-07-11 07-03-03

muramura avatar Jul 09 '22 05:07 muramura

This looks pretty cool. So:

  • MAV_TYPE_HEXADECAROTOR = 16 rotor
  • MAV_TYPE_OCTADECAROTOR - 18 rotor

@rmackay9 @MaEtUgR @meee1 @DonLakeFlyer I am inclined to accept this - any objections?

My assumption is that most GCS don't really need anything more than the number of rotors? In PX4 at least you'd get more detailed geometry information from the component information file.

The reason I ask is that the Octa in the image is coaxial, so you might want to/need to identify it using MAV_TYPE_OCTADECA_COAX_ROTOR. I have no idea what you'd call the MAV_TYPE_HEXADECAROTOR layout in that image!

hamishwillee avatar Jul 13 '22 01:07 hamishwillee

I have no idea what you'd call the MAV_TYPE_HEXADECAROTOR layout in that image!

Ex HEXADECAROTOR layout Screenshot from 2022-07-14 06-05-45

muramura avatar Jul 13 '22 21:07 muramura

Discussed in the dev call. How can this work when neither ArduPilot or PX4 currently support more than 16 outputs? We're not opposed to the names, but if this is not going to make its way into something useful for a GCS to configure itself on, not much point adding at this time.

hamishwillee avatar Jul 21 '22 05:07 hamishwillee

The latest versions of AP do support up to 32 outputs but we don't have support for this vehicle type in AP yet.

rmackay9 avatar Jul 21 '22 06:07 rmackay9

Thanks @rmackay9 I guess it is chicken and egg - you'll need this before it goes in. Any particular ETA?

hamishwillee avatar Jul 21 '22 07:07 hamishwillee