Platform ideenergy does not generate unique IDs.

Question

Platform ideenergy does not generate unique IDs.

feserdel opened this issue 2 years ago · comments

Good morning, after installing ideenergy and creating several contracts, by adding integration again, only the first contract show values. Second and third contracts do not show values for accumulated consumption.

On HA log the following lines appear:

2023-02-02 13:56:19.348 ERROR (MainThread) [homeassistant.components.sensor] Platform ideenergy does not generate unique IDs. ID accumulated-consumption already exists - ignoring sensor.es0021*******_accumulated_consumption
2023-02-02 13:57:49.128 ERROR (MainThread) [homeassistant.components.sensor] Platform ideenergy does not generate unique IDs. ID accumulated-consumption already exists - ignoring sensor.es0021*******_accumulated_consumption
2023-02-02 14:01:28.681 ERROR (MainThread) [homeassistant.components.sensor] Platform ideenergy does not generate unique IDs. ID accumulated-consumption already exists - ignoring sensor.es0021*******_accumulated_consumption
2023-02-02 14:01:28.683 ERROR (MainThread) [homeassistant.components.sensor] Platform ideenergy does not generate unique IDs. ID historical-consumption already exists - ignoring sensor.es0021*******_historical_consumption

Luis López · Answer 1 · Fri Feb 03 2023 20:30:43 GMT+0800 (China Standard Time)

The multi-contract stuff is not tested at all… I don't have access to any contract with multiple contacts.

But I can see some code that can be conflictive: https://github.com/ldotlopez/ha-ideenergy/blob/main/custom_components/ideenergy/entity.py#L134

Changing unique_id generation method (and therefore changing it for existing installations) may have side effects.

Let me run some tests.

Luis López · Answer 2 · Sat Feb 04 2023 01:37:03 GMT+0800 (China Standard Time)

Hi.

(Please read the warnings at the bottom.)

I made some modifications to the 'main' branch to resolve the issue with unique_id generation. This should resolve part of the problem, potentially entirely. If you are using HACS, switch to version 'main'.

Let me known how it goes

Warnings:
The changes include registry migrations, so be sure to backup your data.

Felix Serrano · Answer 3 · Mon Feb 06 2023 20:25:56 GMT+0800 (China Standard Time)

Good morning. After switching to "main", and delete contracts (migration did not work, error received), adding integrations again, now I have all three contracts working. Lets wait a few days until confirm that sensor measurements are working. Thanks!

Luis López · Answer 4 · Tue Feb 07 2023 02:59:43 GMT+0800 (China Standard Time)

Hi @feserdel

Thanks for the feedback 👍

i-de is a bit fussy with multiple contracts, some user have reported bans from i-de.es.