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] Filament Change "Please wait for prompt" does not go away DWIN_CREALITY_LCD_JYERSUI #23700

Closed
severandynasty opened this issue Feb 8, 2022 · 10 comments

Comments

@severandynasty
Copy link

severandynasty commented Feb 8, 2022

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

When using the Filament Change menu to load filament, the printer will not allow you to purge more filament using the "Continue" prompt. The UI becomes unresponsive and the printer must be power cycled to restore function. Bug is present in the 2.0.9.3 release and current bugfix branch. Filament load works correctly when DWIN set is changed to the MarlinUI.

User @doubleO00 also reported this issue.

Bug Timeline

Appears to exist for some time, issue was also reported under issue 22772

Expected behavior

Filament should load to initial load length as defined in FILAMENT_CHANGE_FAST_LOAD_LENGTH then prompt user to purge more if needed or exit. If purge more (Continue) is selected the printer should purge an additional amount each time the user presses Continue until the process is exited.

Actual behavior

Filament load runs the FILAMENT_CHANGE_FAST_LOAD_LENGTH then prompts user to purge more or exit. If purge more is selected the menu freezes at Filament Change "Please wait for prompt". If users exits several other menu items like Disable Steppers no longer work. Printer must be power cycled to restore operation.

Steps to Reproduce

  1. Power on printer
  2. Select Prepare -> Change Filament -> Load Filament
  3. Select appropriate temperature from Preheat menu when prompted then press Continue. Printer heats to required temperature.
  4. Printer loads filament and prompts user to Purge More (Continue) or Exit (Cancel)
  5. User selects Continue, printer freezes at "Filament Load, Please wait for prompt

Version of Marlin Firmware

2.0.9.3

Printer model

Creality Ender 3 v2

Electronics

Stock 4.2.2

Add-ons

BLtouch 3.1, Filament Runout Sensor (microswitch)

Bed Leveling

ABL Bilinear mesh

Your Slicer

Cura

Host Software

OctoPrint

Additional information & file uploads

Configuration.zip

m111 s247 Output:

Send: M111 S247
Recv: echo:DEBUG:ECHO,INFO,ERRORS,COMMUNICATION
Recv: ok
[...]
Recv: echo:busy: processing
Printer seems to support the busy protocol, will adjust timeouts and set busy interval accordingly
Send: M113 S2
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
{{ Additional "processing" messages ommited for brevity }}
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: processing
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
[...]
Recv: echo:busy: paused for user
{{ Printer is locked up here }}

@severandynasty
Copy link
Author

Thanks @tititopher68-dev ! I'll give this a try. Can/should I try to compile with your config files with the 2.0.9.3 Marlin stable release or do I have to stay in the bugfix branch? I'm largely trying to avoid issues that pop up from pre-release code.

@tome9111991
Copy link
Contributor

@severandynasty modify dwin.cpp in /src/lcd/e3v2/jyersui/, line 5001 by :

case PAUSE_MESSAGE_OPTION: 
        pause_menu_response = PAUSE_RESPONSE_WAIT_FOR;
        CrealityDWIN.Popup_Handler(PurgeMore);
        break;

or use this version 2.0.2i here : Jyers#1694 (comment)

Maybe Anyone feels free to create a PR with this fix?

@ellensp
Copy link
Contributor

ellensp commented Feb 11, 2022

@tome9111991 Has this been confirmed to work?

@tome9111991
Copy link
Contributor

tome9111991 commented Feb 11, 2022

@tome9111991 Has this been confirmed to work?

I have only tried @tititopher68-dev version .2.0.2i
I think fix is included and there some other fixes/updates it works nice.

I dont know maybe the can add all the fixes here?

Sebazzz added a commit to Sebazzz/Marlin that referenced this issue Feb 17, 2022
@Dids
Copy link
Contributor

Dids commented Mar 30, 2022

I can confirm that the fix works. Tried it on my own fork and it no longer gets stuck on the prompt, allowing user input (confirm/cancel).

@htro
Copy link

htro commented Apr 29, 2022

@severandynasty modify dwin.cpp in /src/lcd/e3v2/jyersui/, line 5001 by :

case PAUSE_MESSAGE_OPTION: 
        pause_menu_response = PAUSE_RESPONSE_WAIT_FOR;
        CrealityDWIN.Popup_Handler(PurgeMore);
        break;

or use this version 2.0.2i here : Jyers#1694 (comment)

This seems to be only partial fix, it hangs the firmware if purge more functionality is used during filament runout situation. IIRC it also hanged the firmware if the purge more is used multiple times during normal filament change.

@github-actions
Copy link

This issue has had no activity in the last 60 days. Please add a reply if you want to keep this issue active, otherwise it will be automatically closed within 10 days.

@tiagofreire-pt
Copy link

@severandynasty modify dwin.cpp in /src/lcd/e3v2/jyersui/, line 5001 by :

case PAUSE_MESSAGE_OPTION: 
        pause_menu_response = PAUSE_RESPONSE_WAIT_FOR;
        CrealityDWIN.Popup_Handler(PurgeMore);
        break;

or use this version 2.0.2i here : Jyers#1694 (comment)

This seems to be only partial fix, it hangs the firmware if purge more functionality is used during filament runout situation. IIRC it also hanged the firmware if the purge more is used multiple times during normal filament change.

Unfortunately, this is only a partial fix, specially not working while asking for extruding even more.

I have many clients with this bug claiming for a solution, as it is mainly relevant on prints with multiple colors. Blocking those prints mid print.

This is particularly damaging for those with recently launched mainboards (SKR E3 Mini V3), due to the lack of support on many previous Marlin firmware versions.

@ellensp
Copy link
Contributor

ellensp commented Jul 7, 2022

JyersUI has been removed from Marlin. See #24459 for details.

@ellensp ellensp closed this as completed Jul 7, 2022
LCh-77 added a commit to LCh-77/Marlin that referenced this issue Jul 8, 2022
LCh-77 added a commit to LCh-77/Marlin that referenced this issue Jul 18, 2022
@github-actions
Copy link

github-actions bot commented Sep 5, 2022

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 5, 2022
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

7 participants
@ellensp @Dids @htro @tiagofreire-pt @severandynasty @tome9111991 and others