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 NOZZLE_PARK_Z_RAISE_MIN #17624

Merged
merged 3 commits into from
Apr 25, 2020

Conversation

onovy
Copy link
Contributor

@onovy onovy commented Apr 20, 2020

Description

This change behavior of parking nozzle for example during filament
change or pause.

Before this change, nozzle gets lifted everytime by NOZZLE_PARK_POINT.z
with default value 2cm. User want to have this value realativly high,
because it's not possible to manually clean nozzle when nozzle is too
low - near to bed. But this value is too much when nozzle is already
high enough.

After this change, when nozzle gets parked, it's always lifted at least
by NOZZLE_PARK_Z_RAISE_MIN, with default value 2mm. That's enough to not
crash with nozzle into printed model. But if Z is bellow NOZZLE_PARK_POINT.z
nozzle is raised to NOZZLE_PARK_POINT.z. So always high enough to clean
it from bottom manually, but not useless raised if it's high enough.

Benefits

Faster parking in higher layers.

Related Issues

none

This change behavior of parking nozzle for example during filament
change or pause.

Before this change, nozzle gets lifted everytime by NOZZLE_PARK_POINT.z
with default value 2cm. User want to have this value realativly high,
because it's not possible to manually clean nozzle when nozzle is too
low - near to bed. But this value is too much when nozzle is already
high enough.

After this change, when nozzle gets parked, it's always lifted at least
by NOZZLE_PARK_Z_RAISE_MIN, with default value 2mm. That's enough to not
crash with nozzle into printed model. But if Z is bellow NOZZLE_PARK_POINT.z
nozzle is raised to NOZZLE_PARK_POINT.z. So always high enough to clean
it from bottom manually, but not useless raised if it's high enough.
@studiodyne
Copy link
Contributor

Please , at the same time , can you add the only one axis parking like TOOL_CHANGE_PARK to move only one axis to park the head

Thks

@onovy
Copy link
Contributor Author

onovy commented Apr 24, 2020

TOOLCHANGE_PARK_X_ONLY + TOOLCHANGE_PARK_Y_ONLY? That's imho distinct problem from this PR, please feel free to send different request. :)

@thinkyhead thinkyhead merged commit d2a5d51 into MarlinFirmware:bugfix-2.0.x Apr 25, 2020
jmp0x0000 pushed a commit to jmp0x0000/Marlin that referenced this pull request Aug 7, 2020
njibhu pushed a commit to njibhu/Marlin that referenced this pull request Aug 24, 2020
HairingX pushed a commit to HairingX/Marlin that referenced this pull request Jun 16, 2021
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