-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
PFW-1561 Fix MMU unload issue #4773
base: MK3
Are you sure you want to change the base?
Conversation
All values in bytes. Δ Delta to base
|
@@ -5216,7 +5216,7 @@ static void lcd_main_menu() | |||
MENU_ITEM_FUNCTION_P(_T(MSG_FILAMENTCHANGE), lcd_colorprint_change);//8 | |||
|
|||
if (!printer_recovering()) { | |||
if ( moves_planned() || printer_active()) { | |||
if ( moves_planned() || printer_active() || fan_check_error == EFCE_REPORTED) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this needed? 🤔 printer_active()
should return true when the print is paused. If the printer is idle (not printing) then disabling fan check should be available in Settings menu.
} | ||
SERIAL_ECHOLNRPGM(MSG_HOST_ACTION_CANCEL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this change needed? Its not really relevant to MMU unload or fan check.
…an check if needed. Send //action:cancel when LCD Stop print is selected.
60d2bfd
to
65a26e8
Compare
The Tune menu is now also available during a fan error, so the user can decide to en/disable the fan check.
Send
//action:cancel
to host when Stop print is executed.Steps to reproduce:
Results:
The preferred state of printer + MMU is to have no filament loaded after a print finished or has been stopped.
Advantages:
Replaces #4771
ToDo