Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

Add continuous sending and synchrone dimming #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sgoudsme
Copy link
Contributor

I moved the send command from the DMXGateway class to a separate thread. Also by changing the "yield from" to "asyncio.ensure_future" it will run parallel and allows changing multiple channels at the same time.
I think it is an answer to issue #5 and issue #6 .
I don't know if it is a good idea to start separate threads in home assistant, but it is running just fine on my raspberry pi.

@jnimmo
Copy link
Owner

jnimmo commented Dec 24, 2018

Thanks very much for that I've incorporated the async fix, and will have a look at the threading soon - it is a hard requirement for my use that it doesn't continuously send packets, so just want to add an option to what you've done to allow disabling that

@maxenced
Copy link

Any news on this one ? I'm especially interested by the continuous sending, which is required to keep some kind of EFX live in software like QLC+

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants