gody01 / ecovent_v2

Home Assistant Integration for EcoVent VENTO Expert A50/80/100 Fans
GNU General Public License v2.0
37 stars 11 forks source link

Integration crash when Humidity sensor is altered in Blauberg app (Boost trigger enabled/disabled) #17

Open Styx85 opened 1 year ago

Styx85 commented 1 year ago

Hi there, I get an reproducable crash of the integration in HA when I change the hunidity sensor in thie original blauberg app (enable/disable or threshold change) triggering the boost status. Error log as below:

`2023-08-05 21:25:31.424 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities fan.vento_expert_fan are missing or not currently available 2023-08-05 21:25:31.429 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities fan.vento_expert_fan are missing or not currently available 2023-08-05 21:25:31.432 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities fan.vento_expert_fan are missing or not currently available 2023-08-05 21:25:52.586 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Vento Expert Fan 0039003B54435712 for ecovent_v2 Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 388, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/ecovent_v2/init.py", line 37, in async_setup_entry coordinator = VentoFanDataUpdateCoordinator( ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/ecovent_v2/init.py", line 74, in init self._fan.init_device() File "/usr/local/lib/python3.11/site-packages/ecoventv2/init.py", line 225, in init_device return self.update() ^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/ecoventv2/init.py", line 370, in update return self.do_func(self.func['read'], request) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.11/site-packages/ecoventv2/init.py", line 359, in do_func self.parse_response(response) File "/usr/local/lib/python3.11/site-packages/ecoventv2/init.py", line 477, in parse_response setattr ( self, self.params[int(response[:2].hex(),16)][0], response[2:].hex()) File "/usr/local/lib/python3.11/site-packages/ecoventv2/init.py", line 548, in boost_status self._boost_status = self.statuses[val]


KeyError: 2`
gody01 commented 1 year ago

Thank You for report.

Styx85 commented 1 year ago

And sorry I forgot to mention my model. See screenshot. Let me know if I can help with additional testing or information.

Screenshot_2023-08-09-08-33-04-492_io homeassistant companion android

gody01 commented 1 year ago

Should be fixed in 0.9.4 This was undocumented behaviour of Vento Fans, where boost_status can have aditional value of 2 (delayed, after sensor condition is no longer active).

Styx85 commented 1 year ago

Thanks, so far I confirm it as solved. I will test it in several edge cases on the weekend.

When I have a look at the Blauberg Homepage (english one) a generation 3 is launched, therefore there seem to be some additions to the protocol or usage of previously masked functions. But so far I did not have a firmware update.

Am Mi., 9. Aug. 2023 um 16:24 Uhr schrieb Matjaž Godec < @.***>:

Should be fixed in 0.9.4 This was undocumented behaviour of Vento Fans, where boost_status can have aditional value of 2 (delayed, after sensor condition is no longer active).

— Reply to this email directly, view it on GitHub https://github.com/gody01/ecovent_v2/issues/17#issuecomment-1671465886, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHU52EGO2R36TVXG7UNKPWTXUOMTJANCNFSM6AAAAAA3FMS7RI . You are receiving this because you authored the thread.Message ID: @.***>

Styx85 commented 1 year ago

I confirm this issue fixed. No further crashes during long term tests, running stable now for nearly 3 months.