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
Extruder: Allow gaps in extruder numbering. #6302
Conversation
When an extruder is not available, IE, it's broken or otherwise, the strict incremental numbering makes it cumbersome to reconfigure the printer. This allows gaps in the extruder numbering. The only downside I can see is slight extra work on printer startup. That should amount to <1ms of real life latency. Signed-off-by: Viesturs Zariņš <viesturz@gmail.com>
93a59a6
to
6163e0f
Compare
This change might break front ends and/or applications connected through the webhooks API. As things stand now it wouldn't be unreasonable for them to make UI decisions based on the assumption that the primary tool will always be named |
True. Fluidd so far seems totally fine with this.
But more testing would be useful indeed.
This still forces to have a primary extruder named extruder. Only the rest
can be arbitrary ordered.
This does not break any existing configurations, just allows new
configurations that might be incompatible.
Do you think this can be incremental process of Klipper relaxing the
restriction followed by frontends updating if/when this is an issue?
…On Sat, Jul 29, 2023, 23:58 Eric Callahan ***@***.***> wrote:
This change might break front ends and/or applications connected through
the webhooks API. As things stand now it wouldn't be unreasonable for them
to make UI decisions based on the assumption that the primary tool will
always be named extruder, tool 1 extruder1, etc. I cant say for certain
that this is the case, but it is something that likely needs thorough
testing.
—
Reply to this email directly, view it on GitHub
<#6302 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMQT4YMLCXBNABWQTIAKGDXSWBOXANCNFSM6AAAAAA24NKSBQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Thank you for your contribution to Klipper. Unfortunately, a reviewer has not assigned themselves to this GitHub Pull Request. All Pull Requests are reviewed before merging, and a reviewer will need to volunteer. Further information is available at: https://www.klipper3d.org/CONTRIBUTING.html There are some steps that you can take now:
Unfortunately, if a reviewer does not assign themselves to this GitHub Pull Request then it will be automatically closed. If this happens, then it is a good idea to move further discussion to the Klipper Discourse server. Reviewers can reach out on that forum to let you know if they are interested and when they are available. Best regards, PS: I'm just an automated script, not a human being. |
Thanks. As high-level feedback, I'd be leery of making this change for fear of it introducing a regression somewhere. I think it's reasonable to ask users to number their extruders in ascending order. -Kevin |
Unfortunately a reviewer has not assigned themselves to this GitHub Pull Request and it is therefore being closed. It is a good idea to move further discussion to the Klipper Discourse server. Reviewers can reach out on that forum to let you know if they are interested and when they are available. Best regards, PS: I'm just an automated script, not a human being. |
When an extruder is not available, IE, it's broken or otherwise, the strict incremental numbering makes it cumbersome to reconfigure the printer.
This allows gaps in the extruder numbering.
The only downside I can see is slight extra work on printer startup. That should amount to <1ms of real life latency.