cibernox / homeassistant-poolstation

HomeAssistant custom component for integrating the Poolstation platform.
MIT License
9 stars 4 forks source link

Lost Entities #12

Closed grulibre closed 1 year ago

grulibre commented 1 year ago

Hello, First i want to thank you this great job, to be able to control my pool with home assistant for me is great. Since last update (AUGUST 2023) I lost a lot of entities like PH, ORP, salt concentration, temperature. I can only see 4 Relays and 4 Digital inputs. You know why?

luigixx73 commented 1 year ago

Same problem for me, Entities such as pH are lost. Reinstalling the integration only creates 9 entities: 4 inputs, 4 outputs and the binary_sensor for water flow

cibernox commented 1 year ago

That's weird. There was a bug in an earlier version of this addon that messed the entity names, but it was fixed within a day. Are you both sure you are on the latest code? I do have all the sensors available.

grulibre commented 1 year ago

This is what it says, since last update:

Logger: homeassistant.loader Source: custom_components/poolstation/sensor.py:45 Integration: Poolstation (documentation, issues) First occurred: 11:38:22 (1 occurrences) Last logged: 11:38:22

Unexpected exception importing platform custom_components.poolstation.sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/loader.py", line 779, in get_platform cache[full_name] = self._import_platform(platform_name) File "/usr/src/homeassistant/homeassistant/loader.py", line 796, in _import_platform return importlib.import_module(f"{self.pkg_path}.{platform_name}") File "/usr/local/lib/python3.10/importlib/init.py", line 126, in import_module return _bootstrap._gcd_import(name[level:], package, level) File "", line 1050, in _gcd_import File "", line 1027, in _find_and_load File "", line 1006, in _find_and_load_unlocked File "", line 688, in _load_unlocked File "", line 883, in exec_module File "", line 241, in _call_with_frames_removed File "/config/custom_components/poolstation/sensor.py", line 45, in device_class=SensorDeviceClass.PH, File "/usr/local/lib/python3.10/enum.py", line 437, in getattr raise AttributeError(name) from None AttributeError: PH


Logger: homeassistant.setup Source: setup.py:344 First occurred: 11:38:22 (2 occurrences) Last logged: 11:38:22

Unable to prepare setup for platform poolstation.sensor: Platform not found (Exception importing custom_components.poolstation.sensor). Unable to prepare setup for platform poolstation.number: Platform not found (Exception importing custom_components.poolstation.number).

cibernox commented 1 year ago

In that case I believe that the reason is that you have an outdated version of home assistant. A new kind of device class for PH sensors has introduced in 2023.08 (I added it to home assistant), and this integration started using a couple weeks after.

luigixx73 commented 1 year ago

I am running 2023.5.3 but don't want to upgrade because other integrations stop working. Is this possible to install prior versions of your integration ? It always offers me to download 1ca2ce3

cibernox commented 1 year ago

Probably, but first met me check if it’s possible to conditionally use this new feature only if present.

luigixx73 commented 1 year ago

It will be great to preserve backward compatibility.

grulibre commented 1 year ago

In that case I believe that the reason is that you have an outdated version of home assistant. A new kind of device class for PH sensors has introduced in 2023.08 (I added it to home assistant), and this integration started using a couple weeks after.

That was right, I updated my home assistant and it works fine.!!! Thanks

cibernox commented 1 year ago

@luigixx73 I released a new version that I think will allow the integration to work in versions older than 2023.08, but since I'm in that version I couldn't verify that it works myself. Give it a try.

luigixx73 commented 1 year ago

Hi @cibernox, it is beautifully working now with 2023.5.3 All entities added successfully. Thanks for the prompt fix!

cibernox commented 1 year ago

Great, i'll close this then.