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

[FR] SENSORLESS HOMING - Only set CURRENT_HOME to actively homed/probed axis #26634

Closed
sargonphin opened this issue Jan 6, 2024 · 2 comments · Fixed by #26714
Closed

[FR] SENSORLESS HOMING - Only set CURRENT_HOME to actively homed/probed axis #26634

sargonphin opened this issue Jan 6, 2024 · 2 comments · Fixed by #26714
Labels
T: Feature Request Features requested by users.

Comments

@sargonphin
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Hello,

I have noticed that the stepper CURRENT_HOME is applied every time any form of homing and probing is running when sensorless homing is enabled and working. I have perfected sensorless homing to the best settings but I run into speed problems for X and Y during "travel" moves.

I have noticed for example that, after homing XY, the steppers still retain their CURRENT_HOME settings, which is faaaaar lower than running current. They also retain their spreadCycle mode instead of stealthChop which makes then less reliable at low currents.

Are you looking for hardware support?

--

Describe the feature you want

When an axis is not being actively probed, set currents & stealthChop configuration to their running configuration to allow for faster speeds when homing/probing

Additional context

This feature right now mainly applies for sensorless homing because it uses extremely small currents compared to regular moves. A printer with endstop switches wouldn't benefit much from such feature because it uses homing currents closer to running currents, if different at all

@sargonphin sargonphin added the T: Feature Request Features requested by users. label Jan 6, 2024
@sargonphin sargonphin changed the title [FR] SENSORLESS HOMING - Only set CURRENT_HOME to actively homed axis [FR] SENSORLESS HOMING - Only set CURRENT_HOME to actively homed/probed axis Jan 6, 2024
@Mjankor
Copy link

Mjankor commented Jan 6, 2024

Also related: #26369

@thisiskeithb thisiskeithb linked a pull request May 16, 2024 that will close this issue
15 tasks
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T: Feature Request Features requested by users.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants