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

Updates and a couple of fixes for add servo support to magnetic probe #3

Open
wants to merge 2 commits into
base: add-servo-support-to-magnetic-probe
Choose a base branch
from

Conversation

gjdodd
Copy link

@gjdodd gjdodd commented Jan 10, 2025

Description

  • Fix issue with failing probe stow as position after z homing does not fire until after stowing has occured, should just continue to stow the probe as it has just finished homing
  • Add pre deploy and stow steps to allow for position of nozzle to be in a safe place after ABL etc
  • Include sanity check for deactivate servos to include mounted probe servo

Requirements

When moving to stow a probe it does not use the z after homing position before continuing, this leaves the z probe pressed against the bed, if stowing, it should just continue to try and stow the probe.

When checking the configuration and using deactivate_servos_after_move the sanity check needs to now take into account the new config setting.

After completing a sequence which makes use of the probe, such as homing, ABL etc, it should move to a safe position before deploying or stowing.

Benefits

Configurations

Add two configuraitons one for pre deploy of the probe and one for pre stow of the probe, this is to allow the nozzle to be moved to a save position before deploying or stowing and stops crashes into the moved servo arm.

Related Issues

No other issues in this PR

…ot fire until after stowing has occured, should just continue to stow the probe as it has just finished homing

- Add pre deploy and stow steps to allow for position of nozzle to be in a safe place after ABL etc
- Include sanity check for deactivate servos to include mounted probe servo
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

Successfully merging this pull request may close these issues.

1 participant