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] Enders 3 V2, Select Auto Bed Leveling on the controller will freeze the machine #18733

Closed
fleek opened this issue Jul 21, 2020 · 4 comments

Comments

@fleek
Copy link
Contributor

fleek commented Jul 21, 2020

Bug Description

Selecting the bed leveling option on the controller will freeze the machine if Autohome was not selected prior.

My Configurations

Marlin-bugfix-2.0.x-Ender3V2-BLTouchOn5Pin-18Jul2020.zip

Required: Please include a ZIP file containing your Configuration.h and Configuration_adv.h files.

Steps to Reproduce

Additional Information

G29 expects a prior G28. However, if the menu option for G29 was not chosen in sequence after a G28 (Autohome), it will freeze the machine.

To check if Autohome was being done and if not issue G28 command prior to G29.

@fleek
Copy link
Contributor Author

fleek commented Jul 24, 2020

I traced the function to dwin.cpp

`/* Leveling */

void HMI_Leveling(void) {
Popup_Window_Leveling();
DWIN_UpdateLCD();
queue.inject_P(PSTR("G29"));
}`

Currently it is only injecting G29, however, if there is no prior G28 being done, it will result in a hang of the machine.
Is there a way to check if G28 was performed before? Otherwise could a forceful G28 be injected prior to G29?

Sorry, I am not quite familiar with the marlin code, so I need to ask before I do something silly.

@CRCinAU
Copy link
Contributor

CRCinAU commented Jul 26, 2020

@fleek - This is one of the patches I finalised last week but didn't get a chance to submit...

I suggest: G28 O\nG29\nM420 S1\n - otherwise the machine doesn't have a way to enable to grid to be used.

@fleek
Copy link
Contributor Author

fleek commented Jul 26, 2020

M420 S1 is to enable the use of the grid after a G28 because G28 disables the grid usage. However, when a G29 is issued, the grid usage is automatically enabled. G28 O\nG29\nM500 would be more appropriate as it will save the probed grid to EEPROM.

@fleek fleek closed this as completed Jul 26, 2020
@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 26, 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

2 participants