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 arduino uno serial support #300

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

Conversation

ygongdev
Copy link

@ygongdev ygongdev commented Nov 4, 2020

I'm a beginner in arduino, but since I had an arduino uno, which had no wifi, I decided to try a serial approach. Related to #52

Changes

  1. Altered the ws2812_controller_esp8266.ino code in arduino_serial.ino to use a buffer to store the serial data instead of udp data.
  2. After experimentation, I noticed that the buffer needs at least 16 bytes to pass the ledstrand test and can hold maximum of 64 bytes.
  3. Added serial support in led.py. Just need to match your serial port and baudrate
  4. Added arduino_serial to CONFIG.DEVICES
  5. I don't know how the FPS counter would work in the serial implementation so I left that out.

Problem
I think the serial communication is way too slow for the visualization to work properly. The strip just seemingly lights up randomly. I'm guessing this is because every udp packet can send a lot more information about more pixels at faster rate than serial?
However if someone can actually make this work, that would be cool. Just wanted to share my investigation @joeybab3 :)

@joeybab3
Copy link
Collaborator

Sorry for the super late response, I'm glad to see a PR for this as I know it's been a long-requested feature!

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