epics-modules / motor

APS BCDA synApps module: motor
https://epics-modules.github.io/motor/
20 stars 47 forks source link

Torsten/181203 1217 add accs field #122

Closed tboegi closed 1 year ago

tboegi commented 4 years ago

Branch was removed by mistake - now it is restored

kmpeters commented 1 year ago

@tboegi, there is a bug in accEGUfromVelo. The original calculation of acc used local vel and vbase variables that had units of steps. accEGUfromVelo uses the the values of the motor record's VELO and VBAS fields that are in EGU. This results in a much smaller than desired acceleration.

tboegi commented 1 year ago

@kmpeters : Thanks for the review. I'll have a look.