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

Allows input from external DMX sources. #36

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

Conversation

Breina
Copy link

@Breina Breina commented Feb 8, 2023

  • Provide a way to update a universe's data (partially).
  • Reverse output correct the DMX data, so that the same uncorrected input values reflect in the same corrected outputs.
  • Add a callback for channels that changed, containing the new values.

I had to make the output correction a class, so that I could give it 2 functions for applying corrections in two ways. Not sure if that's the pythonic way.

This is just a suggestion, let me know if you envision another approach.

Fixes #31

* Provide a way to update a universe's data (partially).
* Reverse output correct the DMX data, so that the same uncorrected input values reflect in the same corrected outputs.
* Add a callback for channels that changed, containing the new values.
@Breina
Copy link
Author

Breina commented Feb 8, 2023

Your GitHub integration is pretty rad btw!

Co-authored-by: spacemanspiff2007 <a.b@c>
@Breina Breina mentioned this pull request Feb 18, 2023
…ess to its values.

In practice, ArtNetController needs these values in order to send them itself.
@spacemanspiff2007
Copy link
Owner

I had to make the output correction a class, so that I could give it 2 functions for applying corrections in two ways. Not sure if that's the pythonic way.

I'm unsure about that. I think we should just accept the raw data as raw dmx input and not reverse correct it.
I'll take a look at it in the next couple of weeks.
Currently I think it's best to separate dmx input and output into different nodes.
However I'll have yet to deep dive into your implementation to see what makes the most sense.

@Breina
Copy link
Author

Breina commented Feb 19, 2023

Alrighty take your time! Better to do it properly than to rush. :)

@Breina Breina mentioned this pull request Jun 5, 2024
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.

DMX Input
2 participants