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

[FR] NOZZLE_TO_PROBE Z offset value Sanity Check for BLTouch and Notes #23380

Closed
rongoodman303 opened this issue Dec 28, 2021 · 3 comments
Closed
Labels
T: Feature Request Features requested by users.

Comments

@rongoodman303
Copy link

rongoodman303 commented Dec 28, 2021

Is your feature request related to a problem? Please describe.

It is not clear that the NOZZLE_TO_PROBE Z offset value must always be a negative number. Compile is successful with no errors or warnings when a non-negative Z offset is specified, and G29 leveling appears to work. But the result after G29 is a "floating" nozzle, above the bed by the incorrect offset, and inability to successfully get a valid Z offset wizard result.

Are you looking for hardware support?

NA

Describe the feature you want

  1. Sanity check should error when a non-negative Z offset is provided and BLTOUCH is enabled.
  2. In the examples in the NOZZLE_TO_PROBE_OFFSET section, add below the sample grid:
  • The Z offset for BLTouch must always be a negative number because the probe must always be higher than the nozzle.

Additional context

In addition to experiencing this myself, I have seen a number of user posts in various forums about the issue of "floating" nozzles, after G29 ABL, and many of those can probably be attributed to a non-negative Z offset being specified.

@rongoodman303 rongoodman303 added the T: Feature Request Features requested by users. label Dec 28, 2021
@thisiskeithb
Copy link
Member

It is not clear that the NOZZLE_TO_PROBE Z offset value must always be a negative number.

That is not correct: #20344

@rongoodman303
Copy link
Author

rongoodman303 commented Dec 29, 2021

This issue needs to be reopened for further and wider discussion.

The issue you cited was for a very specific printer and highly unusual situation, and it also clearly specifies that NOZZLE_TO_PROBE_OFFSET_Z_POSITIVE must also be enabled as a double check, which it is NOT. Certainly when BLTOUCH is enabled, it cannot be a positive number, as their documentation makes clear. BLTouch and its many clones are the vast majority of ABL devices in use today, so the negative impact of a non-negative value to a larger community of users outweighs the needs of a very small minority, especially when their issue has already been addressed by other means.

At the very minimum, the sanity check I proposed needs to be implemented when BLTOUCH is enabled. Whether or not there needs to be another sanity check for NOZZLE_TO_PROBE_OFFSET_Z_POSITIVE when it is a positive value is another issue, because it obviously does not get verified as described in the issue you cited. And a comment needs to be added to the Configuration.h as I suggested, with addition of "for BLTouch and similar ABL devices", to minimize issues that a non-positive value could have to the vast majority of users.

Enhancement request edited to reflect above. Please reopen.

@rongoodman303 rongoodman303 changed the title [FR] NOZZLE_TO_PROBE Z offset value Sanity Check and Notes [FR] NOZZLE_TO_PROBE Z offset value Sanity Check for BLTouch and Notes Dec 29, 2021
@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T: Feature Request Features requested by users.
Projects
None yet
Development

No branches or pull requests

3 participants
@thisiskeithb @rongoodman303 and others