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

Swallow i2c read timeouts #24

Closed
wants to merge 3 commits into from
Closed

Swallow i2c read timeouts #24

wants to merge 3 commits into from

Conversation

thetic
Copy link
Owner

@thetic thetic commented Mar 5, 2025

Alternative approach to !21. I did not experience any read timeouts in my testing, but others did. If this works, I'd probably like to combine them, but I want to get some testing of this alone.

@thetic thetic marked this pull request as ready for review March 5, 2025 05:24
@ket4mon
Copy link

ket4mon commented Mar 5, 2025

I tried to install the PR with "git clone https://github.com/thetic/klipper-sgp40.git -b exceptional"
Klipper now reports "Internal error during connect: attempted relative import beyond top-level package"

It seems like the error is related to line 14, i'm not quite sure if a relative import over a link (link called sgp40 in ~/klipper/klippy/extras/ which leads to ~/klipper-sgp40/src/klipper_sgp40/) is working

I'm not familiar with python, this was just what i was able to find out and may or may not be the case

@thetic
Copy link
Owner Author

thetic commented Mar 6, 2025

no dice

@thetic thetic closed this Mar 6, 2025
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