Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Add support for lightactions with fireplaces #16

Open
iceman23e opened this issue Feb 27, 2020 · 8 comments
Open

Add support for lightactions with fireplaces #16

iceman23e opened this issue Feb 27, 2020 · 8 comments

Comments

@iceman23e
Copy link

With the most recent update, since it appears the code is now broken into two device classes, I've lost the ability to control the lights in my fireplace. That entity has now become flame control. Is it possible to add lightactions to fireplace devices so that there is separate light and flame control?

@niemyjski
Copy link
Collaborator

niemyjski commented Feb 27, 2020 via email

@iceman23e
Copy link
Author

Right, the flame is treated as a dimmable light. I'm definitely an amateur, but the light.py code appears to map only flame functions to the fireplace "light". It looks like the "supportsLightActions" part of the ceiling fan device type needs to be added to the fireplace and a new device class added? Otherwise, the light entity will only control flame functions. So we'd need fireplace_light, fireplace_flame light entities created to control each.

Before this update, and I assume before the device types were separated, the light entity controlled the lights in the fireplace perfectly.

@niemyjski
Copy link
Collaborator

I recommend making an api call to your local bond api to get a list of all devices and posting it here. We can't update the code appropriately unless we can see what devices are registered (of which we do not own).

@iceman23e
Copy link
Author

This is the fireplace device:

{"name":"Fireplace","type":"FP","location":"Living Room","actions":["TurnLightOff","TurnOff","IncreaseFlame","DecreaseFlame","SetFlame","SetFpFan","TurnFpFanOff","TurnOn","TurnLightOn","Stop","TogglePower","ToggleLight","TurnFpFanOn"],"_":"500358a6","commands":{"_":"12550e9b"},"state":{"_":"80a443b4"},"properties":{"_":"cf4e287f"}}

Fireplace status:

{"power":1,"light":1,"flame":100,"fpfan_power":0,"fpfan_speed":1,"_":"8d2c7267"}

Power controls flame on/off, light on/off, flame controls flame height (which on mine has five settings), fan power and my fan has three settings.

Need anything else?

@niemyjski
Copy link
Collaborator

That looks good, not sure what to do there, maybe turn the light on when the fireplace comes on, or move it into two switches?

@iceman23e
Copy link
Author

Two separate lights for sure. Again, I don't really know what I'm doing, but I managed to get the light entity back for the actual light and not the flame control when copying and pasting the light code from the fan into the fireplace device, but it replaced the flame control light. I just don't know how to get both.

Would this be basically the same fix to get fan controls by modifying the fan.py?

@nguyer
Copy link
Owner

nguyer commented Apr 21, 2020

@iceman23e So just to clarify, to make sure I understand...

You have a device that both has a light, and a flame? But because we now treat the flame as a light, you can't switch the actual light on and off?

@iceman23e
Copy link
Author

@nguyer correct.

My fireplace can modulate 5 flame levels, has 3 light brightness levels (though it appears the device info from bond only allows this light on/off?) and has 3 fan speeds. You are correct that the current code is not able to control the actual light, only the flame.

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

No branches or pull requests

3 participants