Testing - Githubissues

osks commented 4 years ago

This integration requires more testing since I don't own all devices.

Use this issue to ask questions or discuss testing for now.

Jpsy commented 4 years ago

I am testing with a Gardena Smart Sileno City 500 mower. So far no issues. Will report if I find any.

dk1305hb commented 4 years ago

I am on Home Assistant 0.108.2 GARDENA smart Sensor GARDENA Watering Sensor

problem in custom_components: git clone https://github.com/osks/hass-gardena-smart-system.git gardena_smart_system

resolves to the folders (from config/custom_components): -gardena_smart_system --custom_components ---gardena_smart_system

Restart Home Assistant, no effect to integration page.

I copied the content of gardena_smart_system to top folder Restart Home Assistant

2. Gardena Integration Name:"gardena_smart_system" Dialog fields for email, pass an id are without names

3. Invalid Config and not named integration

GARDENA smart Sensor added functional The Watering Devices not

LOG 2020-05-23 20:27:43 DEBUG (SyncWorker_13) [custom_components.gardena_smart_system] Using location: My Garden (6b780a9a-70...) 2020-05-23 20:27:46 DEBUG (SyncWorker_13) [custom_components.gardena_smart_system] Starting GardenaSmartSystem websocket 2020-05-23 20:27:47 DEBUG (MainThread) [custom_components.gardena_smart_system] Gardena Smart System component setup finished 2020-05-23 20:27:47 INFO (MainThread) [homeassistant.components.sensor] Setting up sensor.gardena_smart_system 2020-05-23 20:27:47 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform gardena_smart_system.switch: Platform not found (cannot import name 'SwitchEntity' from 'homeassistant.components.switch' (/usr/src/homeassistant/homeassistant/components/switch/init.py)). 2020-05-23 20:27:47 DEBUG (MainThread) [custom_components.gardena_smart_system.sensor] Adding sensor as sensor [<Entity .....names of sensors.... battery level: 99>] ... registering sensors ... 2020-05-23 20:27:48 INFO (MainThread) [homeassistant.setup] Setting up vacuum 2020-05-23 20:27:48 INFO (MainThread) [homeassistant.setup] Setup of domain vacuum took 0.0 seconds. 2020-05-23 20:27:48 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform gardena_smart_system.vacuum: Platform not found (cannot import name 'StateVacuumEntity' from 'homeassistant.components.vacuum' (/usr/src/homeassistant/homeassistant/components/vacuum/init.py)). ... followed by debug logs for connecting to gardena api, retrieving all my devices. seems to be correct. no error logs

thecem commented 4 years ago

Because of the I/O error I changed over to your integration from https://gitlab.com/tobiaskuntzsch/home-assistant-gardena-integration Thanks a lot!

Here my test results:

The Irrigation Control could be activated for 30 min, by flipping the switch. But the switch is not representing the status of the irrigation control and the valve could not be switched off.

If you need some logs let me know.

osks commented 4 years ago

1. problem
   in custom_components:
   git clone https://github.com/osks/hass-gardena-smart-system.git gardena_smart_system
resolves to the folders (from config/custom_components): -gardena_smart_system --custom_components ---gardena_smart_system

Restart Home Assistant, no effect to integration page.

I copied the content of gardena_smart_system to top folder Restart Home Assistant

The instructions for installation as manual custom component was wrong, sorry for that. I have now fixed them in commit 4cf19780a73d482e046abcb5a968e9d551d5135c.

osks commented 4 years ago

2020-05-23 20:27:47 ERROR (MainThread) [homeassistant.setup] Unable to prepare setup for platform gardena_smart_system.switch: Platform not found (cannot import name 'SwitchEntity' from 'homeassistant.components.switch' (/usr/src/homeassistant/homeassistant/components/switch/init.py)).

Apparently SwitchEntity was added in Home Assistant 0.110 (https://developers.home-assistant.io/blog/2020/05/14/entity-class-names). I've added a note in the readme that Home Assistant 0.110 or newer is required.

dk1305hb commented 4 years ago

...Updated to Home Assistant 0.110.2 It works, no I can monitor the watering devices. Thank you

wijnandtop commented 4 years ago

Just checked with my devices:

[X] powersocket - works (including toggling switch)
[x] irrigation controller, "channels" detected, triggering irrigation works status not properly reflected in HASS. Issue #3
[x] water controller - detected, works, state properly reflected. Missing the internal temperature sensor though. Not sure if it is available in official API. My "reversed enginered" implementation did contain a sensor and I'm actually using it. Battery sensor is actually added as a switch entity.
[x] Lawn mower, properly added, Battery sensor is added as a vacuum entity.
[X] Sensor, I own 2 both properly added.

Logs during registration: 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new vacuum.gardena_smart_system entity: vacuum.greet 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new vacuum.gardena_smart_system entity: vacuum.greet_battery_level 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.gras_ambient_temperature 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.gras_soil_temperature 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.gras_soil_humidity 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.gras_light_intensity 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.gras_battery_level 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.border_ambient_temperature 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.border_soil_temperature 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.border_soil_humidity 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.border_light_intensity 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new sensor.gardena_smart_system entity: sensor.border_battery_level 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.broeikas 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.broeikas_battery_level 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.tuinverlichting 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.beregening_voortuin 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.beregening_midden_tuin 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.beregening_speeltuin 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.beregening_druppel_midden 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.beregening_druppel_voortuin 2020-05-25 20:11:57 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new switch.gardena_smart_system entity: switch.beregening_druppel_oprit

osks commented 4 years ago

Thank you for testing it!

Unfortunately the API does not present any temperature information for the water control device. I didn't even realise there was a temperature sensor in it (I haven't tried your integration), but when I think of it I guess it has for the frost detection.

osks commented 4 years ago

@wijnandtop Are all the issues you had fixed now?

wijnandtop commented 4 years ago

Yes, the missing temperature sensor of the water controller is unfortunate, but not something we can change.

osks commented 4 years ago

Then I'm closing this issue. Open new issues instead.

osks / hass-gardena-smart-system

Testing #1