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

Klipper outputs error in regards to G17 after update #22

Closed
Fl0ken opened this issue Dec 12, 2022 · 8 comments
Closed

Klipper outputs error in regards to G17 after update #22

Fl0ken opened this issue Dec 12, 2022 · 8 comments
Labels
bug Something isn't working

Comments

@Fl0ken
Copy link

Fl0ken commented Dec 12, 2022

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:

  1. Update to Klipper v0.11.0-13-g201f715b
  2. Get error from klipper

Expected behavior
No issues when updating klipper.

Screenshots

@Fl0ken Fl0ken added the bug Something isn't working label Dec 12, 2022
@SoftFever
Copy link
Owner

SoftFever commented Dec 13, 2022

You can delete the dummy G17 macro since the latest Klipper has supported this command.
I will update the readme later.

-SoftFever

@Fl0ken
Copy link
Author

Fl0ken commented Dec 13, 2022

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.

@SoftFever
Copy link
Owner

This very latest Klipper update breaks the ARC support:
Screenshot 2022-12-14 at 12 23 54 AM

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.
Or turn off the arc support in the slicer. (I'm not sure if the arc fitting or the spiral lift caused this issue, I suspect the latter)

-SoftFever

@SoftFever
Copy link
Owner

@Fl0ken I can't reproduce the issue.
Can you share the project?

-SoftFever

@Fl0ken
Copy link
Author

Fl0ken commented Dec 14, 2022

Here's a .3mf and gcode file which produced the error. Happened on the first layer already. ARC_project.zip

@SoftFever
Copy link
Owner

After investigation, I believe it's a newly introduced bug in the latest Klipper.
I saw someone else has raised an issue and PR already: Klipper3d/klipper#5937

-SoftFever

@tehmaze
Copy link

tehmaze commented Dec 14, 2022

With #5939 merged this should be resolved, please confirm.

@Fl0ken
Copy link
Author

Fl0ken commented Dec 15, 2022

That seems to have resolved it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants