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 matrix support #20

Open
ajfisher opened this issue May 24, 2015 · 6 comments
Open

Add matrix support #20

ajfisher opened this issue May 24, 2015 · 6 comments

Comments

@ajfisher
Copy link
Owner

Add the ability to have a rectangular matrix of pixels and support this natively.

@ajfisher ajfisher added this to the Shape capability milestone Dec 3, 2015
@ajfisher
Copy link
Owner Author

ajfisher commented Dec 3, 2015

In multipin multiple strips will be available as well as single strip

Matrix support need to support (at minimum):

  • RC, CR order
  • 3 dimensional matrices
  • Take single strip and split into a matrix NxM (eg 1x25 single strip can become 5x5 square)

@VictorioBerra
Copy link

Take single strip and split into a matrix NxM (eg 1x25 single strip can become 5x5 square)

This is exactly what I came here to find out if this library can do. I know its been a few years now, do we foresee this ever actually happening?

@ajfisher
Copy link
Owner Author

ajfisher commented Aug 15, 2018 via email

@jakepearse
Copy link

What is rc, cr order?

@ajfisher
Copy link
Owner Author

ajfisher commented Dec 11, 2018 via email

@jakepearse
Copy link

My son and I have been playing with this requirement.
Rather than messing about with a bunch of arrays, we used functions to re-map the addresses of pixels in a serpentine array.
Because node-pixel uses functions for addressing rather than presenting strips as 'array-like'.
It wouldn't take much to add cr/rc functionality, I guess.
Although it would perhaps add the need for an argument/options style object, using minor/major axis rather than column/row.
Which would probably better than a bunch of unnamed positional arguments the way we did it.
Anyway its all in a nice-and-small PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants