delfick / photons

Python3.6+ asyncio framework for interacting with LIFX devices
https://photons.delfick.com
MIT License
74 stars 6 forks source link

Add support for the new SKY firmware effect available on LIFX Ceiling devices #132

Closed Djelibeybi closed 5 months ago

Djelibeybi commented 5 months ago

~Note that the TileEffectSkyType enum is currently missing from the published protocol and needs to be added manually until LIFX add it themselves.~

The TileEffectSkyType enum has been added upstream.

Signed-off-by: Avi Miller me@dje.li

Djelibeybi commented 5 months ago

Upstream have fixed the protocol specification so this is ready for review/merge.

delfick commented 5 months ago

@Djelibeybi are you able to clean up the commit history a little please ? :)

Djelibeybi commented 5 months ago

@Djelibeybi are you able to clean up the commit history a little please ? :)

I haven't squashed any of the commits, but I have cleaned up the commit messages. Hope that's what you meant. ;)

delfick commented 5 months ago

@Djelibeybi as in it feels this PR only really needs three commits

Djelibeybi commented 5 months ago

Done. I also switched the field names to snake case to match everything else and updated the help text just a little bit. :)

Djelibeybi commented 5 months ago

Looks good, thanks, do you want me to make a release too?

Not super urgent, but getting Interactor support would be nifty. I'm not sure if that's automagic these days or not.

delfick commented 5 months ago

I'm not sure if that's automagic these days or not.

not overly. Technically interactor only has a lower bound on photons, but the docker container needs to be rebuilt for home assistant

Anyway I have released new photons core and interactor and the docker builds here https://github.com/delfick/photons/actions/runs/9512926226

@Djelibeybi

Djelibeybi commented 5 months ago

I'll add a reminder to take the container for a spin over the weekend. I've spent enough time staring at a screen today. :)