search

StealthChesnut / HA-FoxESS-Modbus

Home Assistant Modbus Integration setup for Fox ESS H1 and AC Inverters
77 stars 37 forks source link

Sensors need Unique ID values #59

Open StealthChesnut opened 1 year ago

StealthChesnut commented 1 year ago

Looking at sensors created by the integration in the settings area of Home assistant shows a message saying they can't be managed from the UI.

Add Unique IDs to prevent this.

canton7 commented 1 year ago

Fixed by #108?

TonyM1958 commented 1 year ago

working on adding unique_ids to my mapping spread sheet along with entity names. Will post this when it's done if that helps?

TonyM1958 commented 1 year ago

I've added a simple formula to work out unique_ids in the attached spreadsheet.

H1 modbus map.xlsx

Once these are fixed, we can copy and paste values to stop them changing if the names are edited.

May need to consider consistency here with @nathanmarlor? Using the H3 config as a template, this currently sets the unique_ids to foxessinv1 in lower case with spaces changed to underscore. However, as far as I can tell, Nathan's code generates a unique id with a prefix of sensor.?

The problem, I think, if there are different unique_ids between these sets of code, anyone moving between the 2 implementations will loose their history?

StealthChesnut commented 1 year ago

I'd like this and Nathan's to be easy to migrate between, and I'm doing some testing to see what doesn't currently line up. Nathan did put some of the unique IDs in, trying to match up what's in this integration, but there may be the odd one here or there that doesn't match for some reason.