-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[FEATURE REQUEST] tmc 2209-5160 coolstep #2491
Comments
Hi @Coffee0297, It did not look like there was a Klipper log file attached to this ticket. The log file has been engineered to answer common questions the Klipper developers have about the software and its environment (software version, hardware type, configuration, event timing, and hundreds of other questions). Unfortunately, too many people have opened tickets without providing the log. That consumes developer time; time that would be better spent enhancing the software. If this ticket references an event that has occurred while running the software then the Klipper log must be attached to this ticket. Otherwise, this ticket will be automatically closed in a few days. For information on obtaining the Klipper log file see: https://github.com/KevinOConnor/klipper/blob/master/docs/Contact.md The log can still be attached to this ticket - just add a comment and attach the log to that comment. Best regards, PS: I'm just an automated script, not a human being. |
This is already possible using SET_TMC_FIELD, just pass the appropriate field names and the values and you'll be able to set up coolstep:
the tmc2660 and tmc5160 also allow you to set these values via the config. Keep in mind that coolstep requires a tuned stallguard value, and that the stallguard tuning is very susceptible to things like ambient temperature, movement speed and stepper motor temperature, so getting one tune for all situations is quite tricky. -Florian |
great then it shuld not be hard to make a setting that says coolstep=true |
CoolStep was removed almost as quickly as it was added to Marlin, so YMMV. |
As Florian pointed out, it needs to be cautiously tuned to your steppers and moving mass before being enabled.
The settings merged to Marlin were tuned to my setup :( I tried to warn MarlinFirmware/Marlin#16790 (comment) but that PR went in quite quickly. |
FYI, I'm not aware of a way to automatically configure "coolstep". To the best of my knowledge it requires external probing hardware - there's info on doing the necessary calibration in the trinamic documentation. Should someone go through those steps, it is possible to configure Klipper with the resulting values. -Kevin |
No one setting will fit every machine, so shipping Klipper with one-size-fits-all setting is unfortunately not possible. It is possible though to create a procedure to automatically gather the actual SG_RESULT for a given machine at given speed ranges, and use that to tune coolstep, and stallguard. No special hardware should be required. It could be similar to the home stepper phase calibration. Cheers, |
Okay, thanks for the info. If a developer is interested in writing such a tuning tool that would be great. Cheers, |
I have to say I'm very tempted :), I just heard about Klipper because of this thread and I'm really impressed by what you've done. All that to say I really get what you are going for, I'll learn a bit more about Klipper and may take on this and other opportunities to make it better :) Cheers, |
I'm going to close this ticket as it looks like the conversation has concluded. If a developer is interested in adding support feel free to open a pull request with the changes. -Kevin |
MarlinFirmware/Marlin#16790
Marlin just got a pull for coolstep, dont look too bad to implement in klipper. it wuld be abselutly perfect for 3d printers, wich are what klipper focus on.
-Tonny
The text was updated successfully, but these errors were encountered: