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

[1.1.x] tool change bad behaviours #8496

Merged
merged 1 commit into from Nov 21, 2017
Merged

[1.1.x] tool change bad behaviours #8496

merged 1 commit into from Nov 21, 2017

Conversation

GMagician
Copy link
Contributor

This PR will address two issues reported in #8492. Just one is left because I don't know if it is an issue or not

  1. when no home is done ("no_move" is set) tool change should never move X & Y because... I don't know where I am...only Z should be adjusted to avoid bed collision if other nozzle is lower than current.
  2. Since no move no bed level matrix should be applied
    When SWITCHING_NOZZLE is enabled and system is homed there are two movements to go to destination, but first can move Z to older position and then crash on the bed

@thinkyhead
Copy link
Member

Ok, patched this and the other to ensure XY move before Z lower, or Z raise before XY move. If possible the moves are combined in a single call to do+blocking_move_to.

This PR will address two issues reported in #8492. Just one is left because I don't know if it is an issue or not

1) when no home is done ("no_move" is set) tool change should never move X & Y because... I don't know where I am...only Z should be adjusted to avoid bed collision if other nozzle is lower than current.
2) Since no move no bed level matrix should be applied
    When SWITCHING_NOZZLE is enabled and system is homed there are two movements to go to destination, but first can move Z to older position and then crash on the bed
@thinkyhead thinkyhead merged commit 6ebf8ea into MarlinFirmware:bugfix-1.1.x Nov 21, 2017
@GMagician GMagician deleted the tool-change-fix-1.0 branch November 21, 2017 09:42
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.

None yet

2 participants