ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

AP_MSP: MSP_RAW_GPS cog should be decidegrees not centidegrees

Open catkira opened this issue 1 year ago • 3 comments

image

source: http://www.multiwii.com/wiki/index.php?title=Multiwii_Serial_Protocol

catkira avatar Sep 12 '24 11:09 catkira

I think we should also make ground_course_dd uint16_t, because the range is 0 - 3600 deci-degrees. I agree that having centi-degrees would be smarter, because it could still fit into a 16-bit number, but just doing it differently here causes incompatibility issues.

catkira avatar Sep 12 '24 11:09 catkira

Looks likely.

Commit message should be AP_MSP:, not fix:. Want to fix that or shall I?

peterbarker avatar Sep 12 '24 12:09 peterbarker

I can fix. Should I also change it to uint16_t? (-> see my last force push)

catkira avatar Sep 12 '24 12:09 catkira

@rmackay9 can you backport this to plane 4.5.6 ?

catkira avatar Sep 13 '24 06:09 catkira