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

Send support #25

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

Send support #25

wants to merge 9 commits into from

Conversation

netmindz
Copy link

Working on porting the send support written for the original non-async version to this library. Not sure what is needed to make it async

@netmindz
Copy link
Author

netmindz commented Apr 5, 2022

Does it look like I'm doing the right bits so far @forkineye ?

@cameronaffleck
Copy link

@forkineye this would very much be appreciated.. Just tried your code commits @netmindz and it wouldn't compile for any ESP32 board

@netmindz
Copy link
Author

Just tried locally with ESP8266 and ESP32 and builds ok. Just also updated the CI to test too

@cameronaffleck
Copy link

Just tried locally with ESP8266 and ESP32 and builds ok. Just also updated the CI to test too

What ESP32 board are you using, out of interest? I'll try again now just in case I was being thick...

@cameronaffleck
Copy link

cameronaffleck commented Nov 23, 2022

Just tried locally with ESP8266 and ESP32 and builds ok. Just also updated the CI to test too

What ESP32 board are you using, out of interest? I'll try again now just in case I was being thick...

So it now successfully compiles, but when looking at sACN data via sACNView, it shows that nothing is being transmitted - anything to look out for? Using WireShark, I can see that my ESP32 is transmitting to the broadcast address of 255.255.255.255 instead of the expected 239.255.0.1 (for my universe 1)

@netmindz
Copy link
Author

It's by no means complete, was checking I was heading in the right direction before I wrote too much

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