search

anthturner / TPLinkSmartDevices

C# Library for Discovering and Operating TP-Link Smart Devices (HS100/105/110, LB100/110/120/130)
Apache License 2.0
40 stars 18 forks source link

added support for KL400 smart light strip #14

Closed bcwood closed 1 year ago

anthturner commented 1 year ago

Hey, thanks for the contribution!

As I don't have this product, can I safely assume you have tested to make sure it works?

If so I think we can bring this in and release a new NuGet package too.

bcwood commented 1 year ago

Yes I do, and this has worked great! In theory you can address each LED in the light strip individually, and control the color of each separately, so eventually I would like to enhance this to add that functionality. But for the time being, this has worked great to address my immediate use case.

On Fri, Oct 21, 2022 at 10:45 AM Anthony Turner @.***> wrote:

Hey, thanks for the contribution!

As I don't have this product, can I safely assume you have tested to make sure it works?

If so I think we can bring this in and release a new NuGet package too.

— Reply to this email directly, view it on GitHub https://github.com/anthturner/TPLinkSmartDevices/pull/14#issuecomment-1287320413, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABN46FGNSTMFO3TZ43VYFDWELQDVANCNFSM6AAAAAARKVQ6RQ . You are receiving this because you authored the thread.Message ID: @.***>

anthturner commented 1 year ago

Cool! If you have any docs on how to individually address LEDs, I can take a crack at it in my spare time. If I do, I'll keep an active public branch with the changes.

Glad you're able to find this library useful!

anthturner commented 1 year ago

Note for future: I plan to release a new NuGet version shortly, but I want to put a tiny bit of time in to refactoring a couple pieces of this code that I cringe at when I browse this repo.

(Should be within the next week, though)

bcwood commented 1 year ago

I don't have any docs on how to individually address LEDs, I just know that in the app, they have some presets with varying colors across the light strip, so I'm assuming it must be possible.