Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is hold_current required for auto tune? #192

Open
spikeygg opened this issue Aug 18, 2024 · 4 comments
Open

Is hold_current required for auto tune? #192

spikeygg opened this issue Aug 18, 2024 · 4 comments

Comments

@spikeygg
Copy link

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
Copy link

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.

@Sonnenwerk
Copy link

Had the same question this day. Anyone willing to adapt the docs ?

@andrewmcgr
Copy link
Owner

andrewmcgr commented Jan 2, 2025 via email

@Sonnenwerk
Copy link

Ah ! You are right !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants