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

Adding missing UI settings for ACN protocol #59

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

Conversation

gleeds
Copy link

@gleeds gleeds commented Jan 7, 2019

I tried to test @ahodges9 's ACN (E1.31) addition, but it wasn't supported in the Board Manager UI and didn't have much luck configuring it manually in config.py. I was able to successfully use Systematic LED with a ESP8266 board and the ACN protocol after adding this code. I also added some documentation to the README.

@ahodges9
Copy link
Contributor

ahodges9 commented Jan 7, 2019

Awesome, thanks for the update. I have actually been working on a completely new implementation that has full support for ACN. You should check it out https://github.com/ahodges9/LedFx

@gleeds
Copy link
Author

gleeds commented Jan 7, 2019

@ahodges9 , I'll check it out.

I've only had it setup for a day or so, but the ESPixelStick firmware has been rock solid on my ESP8266, where as the firmware from this repo froze randomly all of the time and nothing I tried to debug it would help (different ESP, different LED strip, listening for crash dumps, adding debugging log messages). It's so much more of a pain to install on the ESP, but is worth it for the stability and support for multiple pieces of software.

@ahodges9
Copy link
Contributor

ahodges9 commented Jan 7, 2019

Yeah I use ESPixelStick for all my devices and have 10+ nodes running flawlessly for to past few month. Has integration with HomeAssistant as well so I can easily have them function as standard lights as well. For LedFx the recommendation is to use ESPixelStick or WLED and I don't include any FW for this reason

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