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

Continue when hotend temp above target when probing #256

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stablestud
Copy link

This commit prevents the probing procedure to wait for the hotend temperature to settle if its above the target temperature. This is desired, if for example, the first layer failed and the print must be restarted.

We would be forced to wait until the temperature has fallen, depending on the chamber environment and hotend,
this could take several minutes.

Instead we ignore if its above and continue with probing, this saves time, especially when prototyping. It shouldn't be a problem anyways.

@stablestud
Copy link
Author

This can also be made an configurable behavior with an variable, if desired

@KaratsuFr
Copy link

please make it optionnal with a variable
because in case of probe to high could have filament leak that affect the hotend probe

This commit prevents the probing procedure to wait for the hotend
temperature to settle if its above the target temperature.
This is desired, if for example, the first layer failed and the
print must be restarted.

We would be forced to wait until the temperature has fallen,
depending on the chamber environment and hotend,
this could take several minutes.

Instead we ignore if its above and continue with probing, this saves
time, especially when prototyping. It shouldn't be a problem anyways.
@stablestud stablestud force-pushed the ignore-too-hot-hotend-when-probing branch from 20dc71c to 1eeab41 Compare August 23, 2024 14:06
@stablestud
Copy link
Author

I updated the PR.
Added an option which toggles the behavior.
Also it is disabled by default

@KaratsuFr
Copy link

@jschuh any chance to merge that PR?

@zsilet
Copy link

zsilet commented Oct 29, 2024

FWIW I came across and have been running this change for the past month and love the time savings on my Ender3 V2 setup.

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.

3 participants