Skip to content

Conversation

@peterbarker
Copy link
Contributor

avoids double promotion when the argument is not double

@peterbarker
Copy link
Contributor Author

Board                    AP_Periph  antennatracker  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
CubeOrange-periph-heavy  *                                 *                                                   
Durandal                            -8              -56    *           -440    -576              -424   -240   -360
Hitec-Airspeed           *                                 *                                                   
KakuteH7-bdshot                     -8              -56    *           -432    -568              -416   -232   -352
MatekF405                           *               *      *           *       *                 *      *      *
Pixhawk1-1M-bdshot                  *               *                  *       *                 *      *      *
SITL_x86_64_linux_gnu               56              48                 -4048   56                48     56     56
f103-QiotekPeriph        *                                 *                                                   
f303-MatekGPS            *                                 *                                                   
f303-Universal           *                                 *                                                   
iomcu                                                                                *                         
mindpx-v2                           *               *      *           *       *                 *      *      *
revo-mini                           *               *      *           *       *                 *      *      *
skyviper-v2450                                                         *                                       
speedybeef4                         *               *      *           *       *                 *      *      *

@peterbarker peterbarker moved this to ReadyForDevCall in Peter's ArduPilot 4.7 Queue Jan 21, 2026
@peterbarker peterbarker force-pushed the pr/separate-wrap_PI-double-float branch from a44fdfe to ff35693 Compare January 25, 2026 21:44
}

template float wrap_PI<float>(const float radian);
#if AP_MATH_ALLOW_DOUBLE_FUNCTIONS
Copy link
Contributor

Choose a reason for hiding this comment

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

it doesn't make sense to use this define in AP_Math.cpp, its not a project level define, its per cpp file

@peterbarker peterbarker force-pushed the pr/separate-wrap_PI-double-float branch from ff35693 to 38bf47d Compare January 27, 2026 00:22
@peterbarker
Copy link
Contributor Author

Board                    AP_Periph  antennatracker  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
CubeOrange-periph-heavy  *                                 *
Durandal                            -8              -48    *           -432    -568              -416   -240   -352
Hitec-Airspeed           *                                 *
KakuteH7-bdshot                     0               -48    *           -432    -568              -416   -232   -352
MatekF405                           *               *      *           *       *                 *      *      *
Pixhawk1-1M-bdshot                  *               *                  *       *                 *      *      *
SITL_x86_64_linux_gnu               48              56                 48      48                -4048  48     48
f103-QiotekPeriph        *                                 *
f303-MatekGPS            *                                 *
f303-Universal           *                                 *
iomcu                                                                                *
mindpx-v2                           *               *      *           *       *                 *      *      *
revo-mini                           *               *      *           *       *                 *      *      *
skyviper-v2450                                                         *
speedybeef4                         *               *      *           *       *                 *      *      *

After removing defines....

@tridge tridge merged commit cbcb3ba into ArduPilot:master Jan 28, 2026
107 checks passed
@github-project-automation github-project-automation bot moved this from ReadyForDevCall to Done in Peter's ArduPilot 4.7 Queue Jan 28, 2026
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.

2 participants