andrewmcgr / klipper_tmc_autotune

TMC stepper driver autotuning Klipper python extra
GNU General Public License v3.0
626 stars 137 forks source link

Is `hold_current` required for auto tune? #192

Open spikeygg opened 2 months ago

spikeygg commented 2 months ago

I've been looking at tuning the drivers and according to the klipper docs, hold_current should no longer be used. However, the instructions in this GitHub repo's readme say:

  • Currents (run current, hold current, homing current if using a Klipper version that supports the latter)

I also found this post from Sineos on the klipper discourse server from 2021 which cites that there are negative impacts from using both run_current and hold_current.

I installed the klipper_tmc_autotune and ran it without adding the hold_current to my configuration and everything seems to be working... I just don't want to be missing something important here.

robustini commented 2 months ago

hold_current is not recommended any longer. It has been found that the switch between run_current and hold_current has a negative impact on precision. So this script should not use it at all.