Skip to content

klipper: Improve performance#70

Merged
suchmememanyskill merged 2 commits intoOpenCentauri:mainfrom
timiv:klipper-improve-performance
Apr 4, 2026
Merged

klipper: Improve performance#70
suchmememanyskill merged 2 commits intoOpenCentauri:mainfrom
timiv:klipper-improve-performance

Conversation

@timiv
Copy link
Copy Markdown
Contributor

@timiv timiv commented Apr 3, 2026

This slightly improves the performance of klippy by disabling assertions and debug statements in python.

This slightly improves the performance of klippy by disabling assertions and debug statements in python.
@timiv timiv requested a deployment to approval-needed April 3, 2026 17:54 — with GitHub Actions Waiting
@suchmememanyskill
Copy link
Copy Markdown
Contributor

@timiv There's already another pr open (#55 ) that might be more interesting. Also, wouldn't it make sense to do -OO?

@timiv
Copy link
Copy Markdown
Contributor Author

timiv commented Apr 3, 2026

@suchmememanyskill It seems like #55 only tackles RAM usage not CPU load. I had issues with some tight arcs (G3) recently where single core performance was insufficient for the gcode_arcs: 0.1 we use. Together with #41 I was finally able to successfully print the model.

This change does not lead to significant improvements, though.

As far as I understand -OO only disables docstrings, in addition to -O and does not affect performance in a meaningful way.

@timiv timiv temporarily deployed to approval-needed April 3, 2026 18:33 — with GitHub Actions Inactive
@timiv
Copy link
Copy Markdown
Contributor Author

timiv commented Apr 3, 2026

After reviewing #55, I decided to add -OO . We either need to integrate this with #55 or merge it before. #55 only compiles python modules with -OO but does not seem to call python with proper flags.

@suchmememanyskill
Copy link
Copy Markdown
Contributor

I haven't tested this, but i don't see how this would break tbh. Merging

@suchmememanyskill suchmememanyskill merged commit 5c14bc7 into OpenCentauri:main Apr 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants