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

Add support for zigzag layout neopixel matrix #29

Open
punkto opened this issue Jul 15, 2023 · 2 comments
Open

Add support for zigzag layout neopixel matrix #29

punkto opened this issue Jul 15, 2023 · 2 comments

Comments

@punkto
Copy link

punkto commented Jul 15, 2023

Hi,

I have a Neopixel array like these:

I wanted to use your crate to drive the array, but I found that it is not supported (or I have not found the way). I found as reference the NEO_MATRIX_ZIGZAG option in https://learn.adafruit.com/adafruit-neopixel-uberguide/neomatrix-library

I wonder if you would like to add this functionality on your crate. If it is the case, I can send to you a PR with the changes but I would need some guidance.

Regards.

@cat-in-136
Copy link
Owner

NeoPixel can be in any possible wiring order. ws2812_esp32_rmt_driver::lib_embedded_graphics::LedPixelShape trait is provided for the functionality to represent a Point to LED pixel address conversion.

In order to use the ZigZag-layout LED matrix, it is currently necessary to create a new LedPixelShape your own. (unstable option required)

@punkto
Copy link
Author

punkto commented Aug 31, 2023

Thanks for your comments and please excuse the delay in answering.

Following your guide, I have developed a pair of LedPixelShape that cover my needs. If you are interested, they are here .

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

No branches or pull requests

2 participants