Skip to content

Conversation

@graysonarts
Copy link

Here is the pull request for the generic version from here

@graysonarts graysonarts changed the title Neopixel strip LED Strip (sorry the branch name is still neopixel_strip) Dec 8, 2014
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's necessary per the protocol, but has no meaning for this particular device. For something like analog input or pwm it specifies the resolution in number of bits.

@soundanalogous
Copy link
Member

Please report which Arduino (and Arduino-compatible) devices you have tested this with.

Try to use it with other features such as analog input, pwm and digital outputs and an I2C device to be sure there are no unexpected issues in the case that a user has a LED, a couple of pots, a servo and an accelerometer (or other i2c device) attached simultaneously.

Also, please report approximately how much memory this code consumes (you can find out using the freeMemory function of the ArduinoUnit library by printing the memory pre and post initialization of your library, you can also report ongoing in a loop to see if there are leaks - although I'd assume Adafruit has done a pretty good job ensuring their library is stable).

@ajfisher
Copy link

ajfisher commented Dec 8, 2014

Hey all. We've been having a chat today @rwaldron me & others) about
appropriate place to deal with the silliness that is the "almost spi"
protocol that is the ws2812 as it relates to both J5 and firmata and having
to deal with it on both sides.

I think we've come up with a more sustainable solution that will allow us
to deal with future cases of this exact scenario. This is more J5 specific
but I think it applies to firmata too.

I need to do a proof of concept hopefully over next couple of nights so
I'll report back what I find and feed it into you guys.

On Mon, 8 Dec 2014 14:15 Jeff Hoefs [email protected] wrote:

Please report which Arduino (and Arduino-compatible) devices you have
tested this with.

Try to use it with other features such as analog input, pwm and digital
outputs and an I2C device to be sure there are no unexpected issues in the
case that a user has a LED, a couple of pots, a servo and an accelerometer
(or other i2c device) attached simultaneously.

Also, please report approximately how much memory this code consumes (you
can find out using the freeMemory function of the ArduinoUnit library by
printing the memory pre and post initialization of your library, you can
also report ongoing in a loop to see if there are leaks - although I'd
assume Adafruit has done a pretty good job ensuring their library is
stable).


Reply to this email directly or view it on GitHub
#168 (comment).

@soundanalogous
Copy link
Member

See #132 for an update on how we're thinking to include Firmata features that wrap 3rd party libraries.

@graysonarts
Copy link
Author

I'm going to withdraw this proposal for now. This was original written to add support into Johnny-five, but Rick and a few others came up with a better solution (I2C interface for led strips), so I think this is no longer needed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants