davidrapan / ha-solarman

⚡ Solarman Stick Logger integration for 🏠 Home Assistant
MIT License
114 stars 25 forks source link

Several errors since last update #215

Closed psykokwak-com closed 1 month ago

psykokwak-com commented 1 month ago

Description

Hi all, Since the last update, i have the following errors:

Enregistreur: homeassistant.components.sensor Source: helpers/entity_platform.py:369 intégration: Capteur (documentation, problèmes) S'est produit pour la première fois: 09:11:09 (1 occurrences) Dernier enregistrement: 09:11:09

Error while setting up solarman platform for sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 369, in _async_setup_platform await asyncio.gather(*pending) File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 628, in async_add_entities for entity in new_entities: File "/config/custom_components/solarman/sensor.py", line 52, in async_add_entities(create_entity(lambda x: _create_entity(coordinator, x, battery_nominal_voltage, battery_life_cycle_rating), d) for d in descriptions if (is_platform(d, _PLATFORM) and not "configurable" in d)) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/solarman/entity.py", line 21, in create_entity entity = creator(description) ^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/solarman/sensor.py", line 52, in async_add_entities(create_entity(lambda x: _create_entity(coordinator, x, battery_nominal_voltage, battery_life_cycle_rating), d) for d in descriptions if (is_platform(d, _PLATFORM) and not "configurable" in d)) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/solarman/sensor.py", line 37, in _create_entity return SolarmanSensor(coordinator, description, battery_nominal_voltage, battery_life_cycle_rating) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/solarman/sensor.py", line 86, in init if "name" in sensor and sensor["name"] == "Battery" and battery_nominal_voltage > 0 and battery_life_cycle_rating > 0: ^^^^^^^^^^^^^^^^^^^^^^^^^^^ TypeError: '>' not supported between instances of 'NoneType' and 'int'

Cette erreur provient d'une intégration personnalisée

Enregistreur: custom_components.solarman.entity Source: custom_components/solarman/entity.py:27 intégration: Solarman S'est produit pour la première fois: 09:11:09 (1 occurrences) Dernier enregistrement: 09:11:09

Configuring {'name': 'Battery', 'class': 'battery', 'state_class': 'measurement', 'uom': '%', 'rule': 1, 'registers': [184]} failed. [TypeError: '>' not supported between instances of 'NoneType' and 'int']

Reproduction Steps

Just restart HA. Default deye config.

Log

See

Version

24.10.14

davidrapan commented 1 month ago

You added your device quite some time ago, am I right?

Anyway, thanks for the report!

fix: Add default values to some configurables

psykokwak-com commented 1 month ago

I added it before switching on your plugin. I used the other one before. The migration was ok but since the update it does not works. I rollbacked on the previous update for the moment.

davidrapan commented 1 month ago

If you remove the device and re-add it back again it will also fix the errors.

psykokwak-com commented 1 month ago

Does it changes the names of the entities?

davidrapan commented 1 month ago

Names of the entities depends on the profile used.