-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[CURA-12622] Warning on unused extruder. #20765
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
base: main
Are you sure you want to change the base?
Conversation
Since the bed-temp (for example) might be unfluenced by unused but (probably accidentally) enabled extruders, we want to be able to warn the user of such. part of CURA-12622
text=catalog.i18nc("@message", "At least one extruder that wasn't disabled, remains unused in this print (preview). " | ||
f"Unused extruders: '<b>{", ".join(extruder_names)}</b>'. This can sometimes become a problem " | ||
"(for example when the bed-temperature is adjusted by the material-profile present in the unused extruder). " | ||
"It therefore might be desirable to disable these unused extruders manually, depending on the situation. "), |
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.
[nitpick] may I suggest slightly altered version? I believe that this is a bit more clear about the potential impact on quality and reliability
text=catalog.i18nc("@message", "At least one extruder that wasn't disabled, remains unused in this print (preview). " | |
f"Unused extruders: '<b>{", ".join(extruder_names)}</b>'. This can sometimes become a problem " | |
"(for example when the bed-temperature is adjusted by the material-profile present in the unused extruder). " | |
"It therefore might be desirable to disable these unused extruders manually, depending on the situation. "), | |
text=catalog.i18nc("@message", ""**Warning**: not all extruders that were left enabled are actually extruding in this printjob." | |
f"Unused extruders: '<b>{", ".join(extruder_names)}</b>'." | |
"This may lead to sub-optimal settings! To enjoy optimal printed part quality and reliability, please consider to disable the unused extruder(s)!"), |
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.
Thanks! I think that might indeed be more clear -- excepting the bit about 'sub-optimal settings', the sort of users that need this warning are just going to ignore that I feel, so a concrete example might help.
What do you think @HellAholic ?
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.
Depending on the material in the unused extruder, this might actually need to be an error, not just a warning, since it can lead to print failures. There are a few scenarios we still need to consider, and the exact wording of the message isn’t final yet.
The main goal is to help users avoid failed prints, so the message should be clear and firm, not just a gentle suggestion. If a user chooses to ignore the warning or error, then they’re accepting the risk themselves.
One candidate message could be:
The following extruder(s) are enabled but not used in this print: {", ".join(extruder_names)}.
Unused extruders will affect the print, as their material profiles can still modify global settings such as bed temperature.
The impact may range from minor inconsistencies to serious print failures,
depending on the materials and configuration involved.
To avoid these issues, it is strongly recommended to disable any extruders not used in this print.
Continuing with them enabled means accepting responsibility for any resulting problems.
Since the bed-temp (for example) might be influenced by unused but (probably accidentally) enabled extruders, we want to be able to warn the user of such.
Note that it isn't necessary to take skirt/brim into account, since unused extruders (in the rest of the print) are already dropped from the those, even when the prime-tower is enabled.
Engine: Ultimaker/CuraEngine#2240