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

tridonic: Fix bus traffic dumping #118

Merged
merged 1 commit into from
Apr 23, 2023

Conversation

jktjkt
Copy link
Contributor

@jktjkt jktjkt commented Apr 23, 2023

I have more than one control device, which is why the examples/async-dalitest.py hits a framing error when checking if there are any control devices connected. When the bus_watch feature is enabled, this hit a Python error due to some incomplete rename/refactoring where this single occurrence was not changed by mistake.

Fixes: c72374c ("Update the async Tridonic DALI-USB driver to work with 24-bit frames")

I have more than one control device, which is why the
`examples/async-dalitest.py` hits a framing error when checking if there
are any control devices connected. When the bus_watch feature is
enabled, this hit a Python error due to some incomplete
rename/refactoring where this single occurrence was not changed by
mistake.

Fixes: c72374c ("Update the async Tridonic DALI-USB driver to work with 24-bit frames")
@sde1000
Copy link
Owner

sde1000 commented Apr 23, 2023

Good catch. Thank you!

@sde1000 sde1000 merged commit 6d45aad into sde1000:master Apr 23, 2023
5 checks passed
@jktjkt jktjkt deleted the fix-lunatone-bus-watch branch April 30, 2023 18:22
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.

None yet

2 participants