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

[BUG] Idling after a G28; Probe crashes into bed when performing a G29 #18506

Closed
STaveras opened this issue Jul 2, 2020 · 7 comments
Closed

Comments

@STaveras
Copy link

STaveras commented Jul 2, 2020

Idling after homing can cause the probe to crash into the bed when deployed to build a mesh of the build platform.

  1. Home by sending G28
  2. Idle after 5 minutes
  3. Send a G29 P1 to Rebuild Mesh

Expected: Raise Z, Deploy Probe, Begin Probing
Actual: Deploy Probe, Begin Probing

Configuration.zip

@sjasonsmith
Copy link
Contributor

Please test the bugfix-2.0.x branch to see where it stands. If the problem has been resolved then we can close this issue. If the issue isn't resolved yet, then we should investigate further.

There was an issue related to homing where sometimes Z homing could be skipped and cause crashes. That should be prevented in the latest bugfix-2.0.x. I think that was updated a couple weeks ago?

@lilasiian8
Copy link

I am on the the bugfix-2.0.x branch; updated this on Sunday, The probe crashes into the bed on slow/second probe after I printed something and then the printer sat idle. My work around was to home (G28) and G29 P1 from the LCD; instead of terminal via OctoPrint.

@Dracrius
Copy link

@STaveras Pull Request #18637 is more then likely the solution to your issues.

The problem is made far worse with ADAPTIVE_STEP_SMOOTHING enabled as it is in your configs as ADAPTIVE_STEP_SMOOTHING currently can occupy 100% processor power for the stepper-interrupt alone by stepping at the wrong speed. This is not the main issue but another that makes things worse the pull request fixes the Servo Pin priority which prevents ADAPTIVE_STEP_SMOOTHING misbehavior from causing probing interference. A separate fix for ADAPTIVE_STEP_SMOOTHING is also in the works.

@STaveras
Copy link
Author

STaveras commented Jul 15, 2020

Thanks a lot. I was on commit 7b6629c when I submitted the issue. My apologies for not updating this issue earlier, but I haven't had a problem since 9ee891c as there was fix for TMC drivers and homing. I also removed the "UNKNOWN_Z_NO_RAISE" that was defined by default with the example config for the SKR Mini E3 V2.0 as an additional measure. This latest fix is a godsend, as it gets rid of the massive discrepancies that would show up as huge dips or voids in the mesh. Thanks again.

@thisiskeithb
Copy link
Member

I also removed the "UNKNOWN_Z_NO_RAISE" that was defined by default with the example config for the SKR Mini E3 V2.0

Odd. It's disabled in the ones we host here: https://github.com/MarlinFirmware/Configurations/blob/5224f4944f93694dc0506b13fe38a6a973f9c033/config/examples/Creality/Ender-3/BigTreeTech%20SKR%20Mini%20E3%202.0/Configuration.h#L1101

@thinkyhead
Copy link
Member

Current list of configs with UNKNOWN_Z_NO_RAISE:

  • Creality/CR-10mini/MEEB-3DP
  • Creality/Ender-3/MEEB-3DP
  • Einstart-S
  • EXP3D/Imprimante multifonction
  • FlashForge/Creator 2X
  • FlashForge/CreatorPro
  • Kingroon/KP3
  • Qidi/Qidi 1
  • Wanhao/Duplicator 4S

@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 Sep 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants