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

modified probing for better precision #586

Merged
merged 5 commits into from
Jan 17, 2024
Merged

Conversation

Paciente8159
Copy link
Owner

  • modified probing for better precision with option to update/check in realtime
  • modified probing stopping to controlled stop

- modified probing for better precision with option to update/check in realtime
- modified probing stopping to controlled stop
@Paciente8159 Paciente8159 added needs: testing needs: testing core: motion control core: motion control, planning and interpolation enhancement enhancement needs: improvements needs: some feature needs improvements labels Jan 15, 2024
- prevent HOLD command to interrupt probing (midway)
- moved HOLD cleaning to prevent HOLD unlock
- fixed probing loop condition on system failure
- remove cnc unlocking that masks other alarm conditions (like soft reset). Since the stop is now controlled via hold this is no longer necessary.
- updated conditions for probe position update
- probe position updated also on failure
- other probing actions like G39 and Plasma don't affect last probing result (independent)
@Paciente8159 Paciente8159 added tested: OK tested: OK and ready to integrate and removed needs: testing needs: testing needs: improvements needs: some feature needs improvements labels Jan 17, 2024
@Paciente8159 Paciente8159 merged commit 6f89167 into master Jan 17, 2024
38 checks passed
@Paciente8159 Paciente8159 deleted the modified-probe-control branch January 17, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core: motion control core: motion control, planning and interpolation enhancement enhancement tested: OK tested: OK and ready to integrate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant