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

Only Daily water usage works for Phyn Smart Water Assistant #13

Open
kasik96 opened this issue Oct 23, 2023 · 2 comments
Open

Only Daily water usage works for Phyn Smart Water Assistant #13

kasik96 opened this issue Oct 23, 2023 · 2 comments

Comments

@kasik96
Copy link

kasik96 commented Oct 23, 2023

Hi. I have Phyn Smart Water Assistant configured, and only Daily water usage works.

image

All the others are grey, I think it should show for Hot and Cold separately. Is this even supported?

If you can show me how I can debug this, I would be happy to help with coding :)

`Logger: homeassistant.components.sensor
Source: helpers/entity_platform.py:367
Integration: Sensor (documentation, issues)
First occurred: 12:44:47 PM (2 occurrences)
Last logged: 7:19:52 PM
Error while setting up phyn platform for sensor

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 367, in _async_setup_platform
await asyncio.gather(*pending)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 507, in async_add_entities
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 752, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1057, in add_to_platform_finish
self.async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 779, in async_write_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 879, in _async_write_ha_state
state, attr = self._async_generate_attributes()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 820, in _async_generate_attributes
state = self._stringify_state(available)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 785, in _stringify_state
if (state := self.state) is None:
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 475, in state
value = self.native_value
^^^^^^^^^^^^^^^^^
File "/config/custom_components/phyn/sensor.py", line 89, in native_value
if self._device.current_flow_rate is None:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/phyn/device.py", line 86, in current_flow_rate
return self._device_state["flow"]["mean"]
~~~~~~~~~~~~~~~~~~^^^^^^^^
KeyError: 'flow'`

@kasik96
Copy link
Author

kasik96 commented Oct 23, 2023

Update, the problem is that Phyn Smart Water Assistant have temperature1 and temperature2, then pressure1 and pressure2. I will create a pull request.

sov_status, water_consumption and flow is not availible

@kasik96
Copy link
Author

kasik96 commented Oct 23, 2023

Pull Request: #14

Thanks :)

@MizterB

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

1 participant