rgc99/irrigation_unlimited

Valve entity error log

Closed this issue · 5 comments

I recently add a water timer controller that use two valve entities. And with this configuration:

irrigation_unlimited: granularity: 60 refresh_interval: 30 controllers: - name: 'Rega Horta e Plantas' zones: - name: Rega Horta entity_id: valve.dual_water_timer_wt05_rega_horta_valve_1 show: # <= Add these two lines to the timeline: true # <= configuration for individual zones schedules: - time: "08:00" duration: "00:03:00" month: [apr, may, oct]
I get the following error:

2024-04-27 00:51:25.166 WARNING (MainThread) [homeassistant.components.homeassistant] The service homeassistant.turn_off does not support entities valve.dual_water_timer_wt05_rega_horta_valve_1

Is this supported?

https://www.home-assistant.io/integrations/valve/#valve-control-services

Or I should open a feature request.
Thank you very much.

The documentation does state

These can be any entity from the switch or light platforms or anything that supports the turn_on and turn_off actions.

So in theory that may work. Home assistant introduced valves with the Jan 2024 release only, so that may require an update on the irrigation unlimited switch entity.

I am running everything on KNX so I am waiting for a valve device type specifically on KNX to then adapt my switches to it.

@rgc99 can you maybe check if device type valve is supported? If not it should be relatively easy to integrate as you already support switch and light device type. Thanks!

@evercape I tried in the developer tools to use the homeassistant.turn_off with a valve entity and it fails. The service to be used must be valve.close (and valve.open). I'm not sure if this fits as a bug but maybe as a feature request since as you mentioned the valve entities are quite new in HA.

yes good point. I am sure it's simple to add.

For those on KNX I have requested to add valve type to KNX as well.

The valve domain is not currently supported. A work around is to track the state with an automation similar to what is shown in the documentation for a cover. You might also be able to create a switch type alias for the valve but I am not sure on this.

Best to handle it natively. Please open a feature request.

Feature request opened.