Revise 2027 addressable LEDs documentation for Systemcore#3195
Revise 2027 addressable LEDs documentation for Systemcore#3195Another-Person1 wants to merge 41 commits intowpilibsuite:2027from
Conversation
Updated documentation for addressable LEDs to include additional LED types and clarify connections. Added notes on powering and best practices.
Another-Person1
changed the title
rzblue
left a comment
rzblue
left a comment
There was a problem hiding this comment.
The section after the first paragraph should be cleaned up; 5 admonitions in a row is excessive and some of the information there might be better to collect into its own section
| .. note:: WS281x LEDs (with the exception of the WS2815, see below warning) are designed for **5V**, but Systemcore ports output **3.3V**. This may cause issues with some LED strips, and a logic level shifter is needed, such as the [Adafruit Pixel Shifter](https://www.adafruit.com/product/6066). | ||
|
|
||
| .. seealso:: For detailed information about powering and best practices for addressable LEDs, see the [Adafruit NeoPixel Überguide](https://learn.adafruit.com/adafruit-neopixel-uberguide/powering-neopixels). | ||
| .. note:: The 300-500 ohm data line resistor and large 1000uF capactitor across the power pins are highly recommended. Also, use a VRM to power the LEDs, with the *grounds* tied together. |
There was a problem hiding this comment.
I can't say I've seen teams using a capacitor on their LED strips.
The VRM is a specific (discontinued) product; teams can use any 5v regulator
There was a problem hiding this comment.
the idea was mostly to avoid powering long strips of LEDs off of the systemcore
There was a problem hiding this comment.
| .. note:: The 300-500 ohm data line resistor and large 1000uF capactitor across the power pins are highly recommended. Also, use a VRM to power the LEDs, with the *grounds* tied together. | |
| .. note:: The 300-500 ohm data line resistor and large 1000uF capactitor across the power pins are highly recommended. Also, use an external 5V regulator to power the LEDs, with the *grounds* tied together. |
There was a problem hiding this comment.
I suspect the systemcore has a series resistor for current limiting that would make the data line resistor unnecessary (the roboRIO did)
There was a problem hiding this comment.
does it?
supposedly doubling up doesn't cause problems.
There was a problem hiding this comment.
It could cause signal integrity issues if there's too much series resistance
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
other than the removal of the neopixel uberguide, which was already previously there you can make the changes you mentioned.
Co-authored-by: Dan Katzuv <31829093+katzuv@users.noreply.github.com>
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
Made changes based on others' suggestions. If something is not working please let me know.
I agree, I think a wiring section could have a lot of the content and be more readable. There's also currently a rendering issue with one of them |
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
Made more edits and fixes
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
Removed duplicate lines and hopefully fixed indent issue
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
Fixed more stuff and added a basic diagram
fixed the image hopefully
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
Fixed image, should hopefully fix error
|
|
||
| .. warning:: WS2815 LEDs (and maybe other products as well, **check the label**) are designed for **12V** but have **5V** data (see above note). The **backup line on the first strip** needs to be connected to **ground**, **not** connected to the data line despite what other guides may say. | ||
| Example wiring diagram: | ||
| .. image:: https://github.com/user-attachments/assets/2f1d7f05-f11b-4bed-bb91-267134f04f23) |
There was a problem hiding this comment.
| .. image:: https://github.com/user-attachments/assets/2f1d7f05-f11b-4bed-bb91-267134f04f23) | |
| (insert one here in the future) |
There was a problem hiding this comment.
I'm not familiar with the github web editor, but you'll need to commit the image to the repo, not just have it upload to the github server.
There was a problem hiding this comment.
i'll see if I can do that
Another-Person1
left a comment
Another-Person1
left a comment
There was a problem hiding this comment.
removed wiring diagram, checking to see if it is fixed
Co-authored-by: sciencewhiz <sciencewhiz@users.noreply.github.com>
Co-authored-by: sciencewhiz <sciencewhiz@users.noreply.github.com>
Another-Person1
changed the title
4 participants
Updated 2027 Systemcore documentation for addressable LEDs to include additional LED types and clarify connections. Added notes on powering and best practices.