wokwi / wokwi-docs

Documentation for the Wokwi Simulator
https://docs.wokwi.com
MIT License
136 stars 394 forks source link

LED Stripes missing? #57

Open ModischFabrications opened 3 years ago

ModischFabrications commented 3 years ago

https://docs.wokwi.com/diagram-format doesn't refer to the different LED stripes available.

wokwi-neopixel-canvas works, copied from an example, are others supported as well?

urish commented 3 years ago

That's a good point. The reason I haven't written up the docs yet is that the names are a little bit messy, and I wanted to clean it up a bit before writing the docs.

Right now we have:

ModischFabrications commented 3 years ago

Thanks for the extensive list, I can work with that :)

Are there plans to map other strips as well? My code actually uses a SK9822, at the moment I'm manually patching the config to the supported stripe (WS2811?). It's not necessary, just asking for convenience sake.

urish commented 3 years ago

APA102 is probably coming soon, I've just started a thread on the discord server to come up with a more consistent naming scheme for the smart LED parts. It might be something along the lines to wokwi-<type>-<geometry>, e.g. wokwi-ws2812-ring, wokwi-apa102-matrix, but we're still discussing and hopefully somebody comes up with some alternative idea so we can have something to compare against.

Specifically for the SK9822, are they using the same protocol as APA102? Or would that require a different implementation?

ModischFabrications commented 3 years ago

The new naming scheme seems intuitive to me, I don't see any immediate problems with it. A more general hindrance for me is the "wokwi" prefix everywhere. A prefix for all might as well be for none, but I'm sure there is a reason for it.

FastLED has a specific Chipset for the SK, which works for me (https://github.com/ModischFabrications/turn_indicator_code/blob/f89fb6c4f026df3b65143919f2e83ba02c0c54aa/src/lights.h#L186), but it think they are more or less equal. If I recall correctly choosing SK is backwards compatible to APA, but you might want to check with them directly.