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

Add Sensorless Homing current #15690

Merged
merged 8 commits into from
Nov 11, 2019
Merged

Add Sensorless Homing current #15690

merged 8 commits into from
Nov 11, 2019

Conversation

danym21
Copy link
Contributor

@danym21 danym21 commented Oct 27, 2019

Requirements

TMC drivers with stallguard enabled

Description

I use TMC5160 stepper drivers, since I use 0.9deg/step instead 1.8deg/step motors stallguard is not triggered correctly with the the current that I use for printing.

as example for printing I use a current from 1100 mA and "STALL_SENSITIVITY 2" the machine is crashing, when I use "STALL_SENSITIVITY 1" Stallguard is trigger often in the middle of the way to the edge. When I use a current from 462 for homing with "STALL_SENSITIVITY 2" all is working fine but on printing it is losing steps.

Benefits

in the homing process set a different current on the steppers so that the sensorless homing is
more flexible.
also spreadCycle can be used as option.

Related Issues

[FR] different driver current for sensorless_homing #15686

@danym21 danym21 changed the title Bugfix 2.0.x different driver current for sensorless_homing Oct 27, 2019
@thisiskeithb
Copy link
Member

Have you tried homing faster? Sensorless homing with StallGuard tends to work better if things are moving quickly.

@danym21
Copy link
Contributor Author

danym21 commented Oct 27, 2019

shure I have tried this, it crash on the machine much harder but will also not help when the head is very near the zero point, then there is not enough time to accelerate.

@thinkyhead thinkyhead changed the title different driver current for sensorless_homing Add a separate current for sensorless_homing Oct 29, 2019
@thinkyhead thinkyhead merged commit 28e1614 into MarlinFirmware:bugfix-2.0.x Nov 11, 2019
@thinkyhead thinkyhead changed the title Add a separate current for sensorless_homing Add Sensorless Homing current Nov 11, 2019
@danym21 danym21 deleted the bugfix-2.0.x branch December 12, 2019 23:00
philippniethammer pushed a commit to philippniethammer/Marlin that referenced this pull request Dec 21, 2019
christran206 pushed a commit to christran206/Marlin2.0-SKR-Mini-E3-1.2 that referenced this pull request Dec 30, 2019
@thinkyhead
Copy link
Member

thinkyhead commented May 1, 2023

I'm working on cleaning up and standardizing all the endstops code, including sensorless with DIAG0 and SPI-based sensorless. I notice that homing current was added to configurations for Z2, etc., but it is not implemented in the code. And later Z homing current was changed to only apply to DELTA. So, these are some issues I hope to work out in PR #25758 before it gets merged. Please add your commentary to that PR if you are still interested in this topic.

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.

3 participants