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

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

Open
Styx85 opened this issue Aug 5, 2023 · 5 comments

Comments

@Styx85
Copy link
Contributor

Styx85 commented Aug 5, 2023

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

@Styx85 Styx85 changed the title Integration crash when Humidity senser is altered in Blauberg app Integration crash when Humidity sensor is altered in Blauberg app (Boost trigger enabled/disabled) Aug 5, 2023
@gody01
Copy link
Owner

gody01 commented Aug 9, 2023

Thank You for report.

@Styx85
Copy link
Contributor Author

Styx85 commented Aug 9, 2023

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
Copy link
Owner

gody01 commented Aug 9, 2023

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
Copy link
Contributor Author

Styx85 commented Aug 10, 2023 via email

@Styx85
Copy link
Contributor Author

Styx85 commented Sep 25, 2023

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

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

2 participants