Rename AP::ahrs().get_roll() to AP::ahrs().get_roll_rad() etc.

[peterbarker]

Costs more bytes on boards with scripting because of the extra bindings

Board                    AP_Periph  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
CubeOrange-periph-heavy  *                 *                                                   
CubeRedPrimary                      416    *           408     416               408    416    416
Durandal                            416    *           408     416               408    416    408
Hitec-Airspeed           *                 *                                                   
KakuteH7-bdshot                     408    *           416     408               416    408    408
MatekF405                           *      *           *       *                 *      *      *
Pixhawk1-1M-bdshot                  *                  *       *                 *      *      *
f103-QiotekPeriph        *                 *                                                   
f303-Universal           *                 *                                                   
iomcu                                                                *                         
revo-mini                           *      *           *       *                 *      *      *
skyviper-v2450                                         *                                       

Changes all the scripts to choose between renaming to use the new binding or use the other new binding introduced, get_roll_deg etc. Most places change to use _deg

Deprecates the old get_roll etc bindings, so at some stage that space can be reclaimed.

If scripting is disabled then this is a no-compiler-output change. To that end, reviews could probably focus on the LUA script changes.

Board                    AP_Periph  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
CubeOrange-periph-heavy  *                 *
CubeRedPrimary                      *      *           *       *                 *      *      *
Durandal                            *      *           *       *                 *      *      *
Hitec-Airspeed           *                 *
KakuteH7-bdshot                     *      *           *       *                 *      *      *
MatekF405                           *      *           *       *                 *      *      *
Pixhawk1-1M-bdshot                  *                  *       *                 *      *      *
f103-QiotekPeriph        *                 *
f303-Universal           *                 *
iomcu                                                                *
revo-mini                           *      *           *       *                 *      *      *
skyviper-journey                                       *
skyviper-v2450                                         *

[IamPete1]

Suggested change

	singleton AP_AHRS method get_roll deprecate Use get_roll_rad
	singleton AP_AHRS method get_roll deprecate Use get_roll_rad or get_roll_deg

[peterbarker]

Done, thanks!

[peterbarker]

I've removed the _deg bindings and marking for merge as discussed at DevCall.

I brought up the fact that you can get degs trivially, and that the flash cost for _deg bindings is real and permanent... so we decided to nix them for now.

@IamPete1 there was the question as to whether we could add custom bindings for get_roll() and instantly deprecate them. This would allow us to not have the bindings in the main code at all. I just wasn't sure we could deprecate custom functions....