Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Light switches for ovens #58

Closed
TiddlyWiddly opened this issue Nov 30, 2021 · 7 comments
Closed

Light switches for ovens #58

TiddlyWiddly opened this issue Nov 30, 2021 · 7 comments

Comments

@TiddlyWiddly
Copy link

The SmartHQ app allows for turning on and off of lights for the double oven but they don't seem to be found as entities.

@simbaja
Copy link
Owner

simbaja commented Dec 2, 2021

That would be useful :). Any chance that you can go to the simbaja/gehomesdk repo and create a debug log (where you set the light on/off and let it record)? I think it should be fairly straightforward to add if we see the request/response being sent.

@simbaja
Copy link
Owner

simbaja commented Dec 12, 2021

Not sure if you're comfortable manually installing the component, but the latest dev branch has some logic to control the oven light. I'm not sure it works perfectly (my oven doesn't actually expose some of the properties to make this work), so I'll need some help testing the functionality.

@TiddlyWiddly
Copy link
Author

I can install it manually, I just don't know how. Do you have some instructions I can follow? I'm using HAOS so I've got Supervisor. Or would you rather me still run the debug log?

@simbaja
Copy link
Owner

simbaja commented Dec 15, 2021

You should be able to just copy the ge_home folder from the "dev" branch of this repo into your custom_components directory in the HA config directory. I don't think there's any difference when using HAOS/Supervisor.

@brysons
Copy link

brysons commented Dec 20, 2021

I have a double oven and just gave this a try and ran into some trouble. This is not a high priority feature for me, but maybe sharing what I'm seeing will be useful. First off, the SmartHQ app only shows me a toggle for the upper oven light, which works as expected but it seems strange that only the upper oven is available since the touchscreen on the oven lets me toggle both oven lights together.

I installed the latest dev branch (although I am still running 2021.11.5 and had to revert commit 180b11f in order to get it working). In home assistant, ge_home created an entity select.<my_oven>_lower_oven_light with options "High" and "Off". Note that this is only the "lower_oven" while the SmartHQ app only shows me a light toggle for the "Upper Oven". The select in home assistant has no effect.

In my home assistant log, I see:

2021-12-19 23:15:48 DEBUG (MainThread) [gehomesdk.ge_appliance] Setting ErdCode.UPPER_OVEN_LIGHT to ErdOvenLightLevel.OFF
2021-12-19 23:15:48 DEBUG (MainThread) [gehomesdk.ge_appliance] Setting ErdCode.UPPER_OVEN_LIGHT_AVAILABILITY to ErdOvenLightLevelAvailability(z1=False, z2=True, z3=False, z4=True, z5=False, raw_value='03')

2021-12-19 23:15:48 DEBUG (MainThread) [gehomesdk.ge_appliance] Setting ErdCode.LOWER_OVEN_LIGHT to ErdOvenLightLevel.OFF
2021-12-19 23:15:48 DEBUG (MainThread) [gehomesdk.ge_appliance] Setting ErdCode.LOWER_OVEN_LIGHT_AVAILABILITY to ErdOvenLightLevelAvailability(z1=False, z2=False, z3=False, z4=False, z5=False, raw_value='00')

2021-12-19 23:15:50 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new select.ge_home entity: select.<my_oven>_lower_oven_light
2021-12-19 23:15:50 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event entity_registry_updated[L]: action=create, entity_id=select.<my_oven>_lower_oven_light>
2021-12-19 23:15:50 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=select.<my_oven>_lower_oven_light, old_state=None, new_state=<state select.<my_oven>_lower_oven_light=Off; options=['Off', 'High'], friendly_name=<my_oven> Lower Oven Light, icon=mdi:lightbulb, device_class=select @ 2021-12-19T23:15:50.230822-08:00>>

@stanchat
Copy link

For GE Double ovens it finds the lower light but not the upper light. I was able to fix by adding the following code in devices/oven.py at line 79. It was able to find and now I have 18 entities up from 17.

if upper_light_availability is None or upper_light_availability.is_available: oven_entities.append(GeOvenLightLevelSelect(self, ErdCode.UPPER_OVEN_LIGHT))#SDC added code for upper light 1/14/2023

@simbaja
Copy link
Owner

simbaja commented Apr 23, 2023

I think this is all handled in the newest versions, closing for now.

@simbaja simbaja closed this as completed Apr 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants