-
-
Notifications
You must be signed in to change notification settings - Fork 833
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
Klipper outputs error in regards to G17 after update #22
Comments
You can delete the dummy G17 macro since the latest Klipper has supported this command. -SoftFever |
Unfortunately this does not fix it. I now get an error when the gcode has G2/G3 arcs: "G2/G3 requires IJ, IK or JK parameters" I can't pinpoint exactly why it works for some files (a coin worked without issues) but a larger print with text did not. |
This very latest Klipper update breaks the ARC support: Klipper3d/klipper@0c5c87d#diff-4f1c41066bdb7bfe33f5c832f650b43f1bc088ce7a35edd116d2e5a50bcb39a7 I will see what needs to be done to support that. As a workaround, for now, you can either roll back the Klipper to the previous version. -SoftFever |
@Fl0ken I can't reproduce the issue. -SoftFever |
Here's a .3mf and gcode file which produced the error. Happened on the first layer already. ARC_project.zip |
After investigation, I believe it's a newly introduced bug in the latest Klipper. -SoftFever |
With #5939 merged this should be resolved, please confirm. |
That seems to have resolved it. Thanks! |
Describe the bug
Added the G2/G3 arcs and the dummy command to printer.cfg and used the firmware and arcs without any issues. Updated Klipper to v0.11.0-13-g201f715b and now there is an error that G17 is already registered. Commenting the G17 command out solves this. As it's recommended to have this, any solution to fix this? use "rename_existing"?
To Reproduce
Steps to reproduce the behavior:
Expected behavior
No issues when updating klipper.
Screenshots
The text was updated successfully, but these errors were encountered: