Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

made changes for ws2812b #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

made changes for ws2812b #10

wants to merge 1 commit into from

Conversation

drvkmr
Copy link

@drvkmr drvkmr commented Feb 4, 2019

So the neopixel strip I had was ws2812b, and your code only worked after making these few changes. Also, I am not very familiar with github's way of contributing, so here is an attempt.

Also, I used your code to build https://github.com/drvkmr/LED_projector.

@bigjosh
Copy link
Owner

bigjosh commented Feb 5, 2019

Your pull request is perfect, and your project looks very cool!

I am not going to merge this request, but not because there is anything wrong with your code.

There are lots of different WS2812B-style pixels available, and different ones have different timing requirements. the timing numbers in my code reflect the values that I've found work best with the original World Semi WS2812B chips.

You should keep the timing values that work for your LEDs, and I will leave this PR open so other people can see your numbers and try them in case my numbers do not work with their chip.

Thanks!

@thoughtfulhedon
Copy link

I ran into the same problem with WS2813, these are the numbers that worked for me, based on the data sheet. I haven't tried tweaking them to test the outer bounds.

T1H 800
T1L 200
T0H 400
T0L 200
RES 400000

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants