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

Error on Integration #58

Closed
sigmapillar opened this issue Jan 10, 2025 · 13 comments · Fixed by #62
Closed

Error on Integration #58

sigmapillar opened this issue Jan 10, 2025 · 13 comments · Fixed by #62

Comments

@sigmapillar
Copy link

sigmapillar commented Jan 10, 2025

It appears on a fresh install of HA and this repo through HACS, it's halting on the Indoor Air Quality (IAQ) sensor that the integration is reporting.

Daikin calls it their own IAQ, but it's a Haven device based on the equipment sitting on the wifi.

Log output:

Logger: homeassistant.config_entries
Source: config_entries.py:640
First occurred: 5:15:51 PM (3 occurrences)
Last logged: 5:16:49 PM
Error setting up entry Daikin One for daikinone

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 640, in __async_setup_with_context
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/daikinone/init.py", line 49, in async_setup_entry
await data.update()
File "/config/custom_components/daikinone/init.py", line 28, in update
await self._update(no_throttle=no_throttle) # type: ignore
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/daikinone/init.py", line 37, in _update
await self.daikin.update()
File "/config/custom_components/daikinone/daikinone.py", line 247, in update
await self.__refresh_thermostats()
File "/config/custom_components/daikinone/daikinone.py", line 308, in __refresh_thermostats
self.__thermostats = {device.id: self.__map_thermostat(device) for device in devices}
~~~~~~~~~~~~~~~~~~~~~^^^^^^^^
File "/config/custom_components/daikinone/daikinone.py", line 345, in __map_thermostat
air_quality_indoor=self.__map_air_quality_indoor(payload),
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
File "/config/custom_components/daikinone/daikinone.py", line 369, in __map_air_quality_indoor
particles=payload.data["aqIndoorParticles"],
~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^
KeyError: 'aqIndoorParticles'

@zlangbert
Copy link
Owner

@sigmapillar can you grab the diagnostics from the thermostat device in HA and drop that here?

@sigmapillar
Copy link
Author

@zlangbert
Copy link
Owner

@sigmapillar can you update to 0.9.1 and download the diagnostics again? I had previously only implemented diagnostics for the thermostat devices themselves, but the config entry diagnostics will work on 0.9.1

@sigmapillar
Copy link
Author

@zlangbert
Copy link
Owner

Wait this looks like the diagnostics from HACS not the integration. You need to click here:

image

@sigmapillar
Copy link
Author

It's not there?

image

When checking HA logs page, the logs are as follows:

Logger: homeassistant.config_entries
Source: config_entries.py:640
First occurred: 2:47:46 PM (1 occurrences)
Last logged: 2:47:46 PM

Error setting up entry Daikin One for daikinone
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 640, in __async_setup_with_context
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/daikinone/__init__.py", line 49, in async_setup_entry
    await data.update()
  File "/config/custom_components/daikinone/__init__.py", line 28, in update
    await self._update(no_throttle=no_throttle)  # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/daikinone/__init__.py", line 37, in _update
    await self.daikin.update()
  File "/config/custom_components/daikinone/daikinone.py", line 246, in update
    await self.__refresh_thermostats()
  File "/config/custom_components/daikinone/daikinone.py", line 307, in __refresh_thermostats
    self.__thermostats = {device.id: self.__map_thermostat(device) for device in devices}
                                     ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^
  File "/config/custom_components/daikinone/daikinone.py", line 344, in __map_thermostat
    air_quality_indoor=self.__map_air_quality_indoor(payload),
                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
  File "/config/custom_components/daikinone/daikinone.py", line 368, in __map_air_quality_indoor
    particles=payload.data["aqIndoorParticles"],
              ~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^
KeyError: 'aqIndoorParticles'

@sigmapillar
Copy link
Author

Happy to do a gmeet/zoom if it'd help?

@zlangbert
Copy link
Owner

Ohh I didn't realize you couldn't get diagnostics if the integration failed to set up, though that makes sense.

Okay what I'll do is push a new version that makes these fields optional so it doesn't crash. And then we can get diagnostics and see what fields you actually have

@zlangbert
Copy link
Owner

Also @sigmapillar what equipment do you have? Particularly what thermostat

@sigmapillar
Copy link
Author

do-_autoaway_front_08-21_hr-removebg

Daikin One+ Thermostat (v.3.5.6), Haven IAQ, Daikin DX17VSS outdoor unit, DM97MC indoor unit, and "media filter", but I think that's just generic air filter item. Lastly an EEV coil.

@zlangbert
Copy link
Owner

Okay great, I see the problem. I just typo'd the keys of a couple values.

Can you do me one more favor and screenshot what the app/thermostat shows for indoor air quality measurements? Was hoping I could find someone to get confirmation of the units of these values

@sigmapillar
Copy link
Author

So, it's incredibly useless because how Daikin integrates it - they don't publish values. Just some BS "high/low" random chart mappings.

The outdoor values it pulls from some web source. I wish I actually knew what the values were or what its detecting.

Screenshot_20250112_151911_Daikin One Home
Screenshot_20250112_151920_Daikin One Home
Screenshot_20250112_151927_Daikin One Home
Screenshot_20250112_151936_Daikin One Home

@zlangbert
Copy link
Owner

Ha okay, good job Daikin. Well looking at the Haven documentation I think I probably got it right now.

Just released a new version that should fix your issue. Let me know if not!

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

Successfully merging a pull request may close this issue.

2 participants