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

Kuara H3, and some KHs and H3s doesn't support new registers in 1.13.1 #756

Closed
Badlee2020 opened this issue Feb 1, 2025 · 14 comments · May be fixed by #757
Closed

Kuara H3, and some KHs and H3s doesn't support new registers in 1.13.1 #756

Badlee2020 opened this issue Feb 1, 2025 · 14 comments · May be fixed by #757

Comments

@Badlee2020
Copy link

With version 1.13.2b1 I get the following error:

Error reading registers. Type: RegisterType.HOLDING; start: 31000; count: 91; slave: 196 from tcp://192.168.2.235:502: Exception Response(131, 3, IllegalAddress)

With version 1.13.0 every was ok

Manager 1.62
Master 1.74
Slave 1.03

I have a KOPP-KUARA 12.0-3H

@canton7
Copy link
Collaborator

canton7 commented Feb 1, 2025

Right, I see. It looks like #703 added support for battery SoH, but this isn't supported by the Kuara. I'll fix.

@canton7
Copy link
Collaborator

canton7 commented Feb 1, 2025

Can you just check quickly whether you can read holding registers 31102, 31103, 31123? Developer tools -> Actions -> FoxESS - Modbus: Read Registers.

canton7 added a commit to canton7/foxess_modbus that referenced this issue Feb 1, 2025
canton7 added a commit to canton7/foxess_modbus that referenced this issue Feb 1, 2025
@canton7
Copy link
Collaborator

canton7 commented Feb 1, 2025

And do you know whether the Kuara supports Peak Shaving work mode?

@canton7 canton7 changed the title Issues with registers Kuara H3 doesn't support new registers in 1.13.1 Feb 1, 2025
@Badlee2020
Copy link
Author

Badlee2020 commented Feb 2, 2025

31102, 31103, 31123 these registers produce errors. Peek shaving is not a work mode I have seen in the documentation. Peek shaving is a work mode on the device though, so i presume it has it. More than willing to test it though:) the documentation with Kopp is quite 😔

@jbarrett-fmad
Copy link

FYI I'm getting the same error on KH10.5 Manager version 1.21. Have requested firmware update from Fox to see if that resolves it.

@hooktheboss
Copy link

H3-6.0-E - AUX no connection after Update to v1.13.1

Error connecting to inverter (friendly name: "FoxESSInverter"). Error reading registers. Type: RegisterType.HOLDING; start: 31000; count: 91; slave: 247 from tcp://192.168.178.17:512: Exception Response(131, 3, IllegalAddress)

Manager 1.56
Master 1.67
Slave 1.03

Reverted to v1.13.0 - working OK.

canton7 added a commit to canton7/foxess_modbus that referenced this issue Feb 2, 2025
This reverts nathanmarlor#703 and nathanmarlor#704, due to complaints that this breaks some H3s,
Huara H3s, and KHs.

See: nathanmarlor#756
@AndaleR
Copy link

AndaleR commented Feb 2, 2025

Same here.

Worst case: Will all data be disappeared? Or is there a solution and it just needs to be updated and all the energy-data will be back in HomeAssistant.

canton7 added a commit to canton7/foxess_modbus that referenced this issue Feb 2, 2025
This reverts nathanmarlor#703 and nathanmarlor#704, due to complaints that this breaks some H3s,
Kuara H3s, and KHs.

See: nathanmarlor#756
@canton7
Copy link
Collaborator

canton7 commented Feb 2, 2025

Right, I've released v1.13.2 which reverts those changes. I'll get them back into the next beta, and we can work out what firmware versions actually support them.

@canton7 canton7 changed the title Kuara H3 doesn't support new registers in 1.13.1 Kuara H3, and some KHs and H3s doesn't support new registers in 1.13.1 Feb 2, 2025
@iancg
Copy link

iancg commented Feb 2, 2025

Right, I've released v1.13.2 which reverts those changes. I'll get them back into the next beta, and we can work out what firmware versions actually support them.

Would another approach be to try reading those registers as a test read after startup, and if success then add them in to the map, otherwise log and skip them?

That may give a way to handle multiple versions without actually hard coding exactly which registers are available in which version.

It may need some validation to handle cases where you don't get illegal address, but just no data. Might not work if the same address is used for different things in different versions (but I don't think I've seen any evidence of that so far?)

@AndaleR
Copy link

AndaleR commented Feb 2, 2025

Thanks. I updated to 1.13.2 - but my energy-data are gone.

But actually it’s collecting data from the inverter.

@WyndStryke
Copy link

. I'll get them back into the next beta, and we can work out what firmware versions actually support them.

Presumably at least the KH1.33+ can read them, at the minimum. I'm not in a position to test the earlier firmware.

Can the integration trap failures and disable individual sensors if they can't be read? Or is it a hard failure?

@canton7
Copy link
Collaborator

canton7 commented Feb 2, 2025

@iancg I've got a branch somewhere which attempts to do that. It's not particularly straightforward:

  • We read large blocks of registers, but the inverter only tells us if one of the block was invalid, so we need logic to work out which particular register was bad
  • We can't avoid creating the entity: we'll just have to leave it in an "Unknown" state
  • It means we won't necessarily find out if we've got the register map incorrect, as people won't tell us

All solvable, but it just needs time to get it over the line. Feel free to pick it up!

@canton7
Copy link
Collaborator

canton7 commented Feb 2, 2025

@AndaleR There's no reason HA just decide to delete all of your energy data just because an integration wasn't working for a bit. You won't have been collecting new data obviously, but that shouldn't affect old data. If your HA instance does delete old energy data when you disable an integration, you should raise that as a bug with HA.

@canton7
Copy link
Collaborator

canton7 commented Feb 2, 2025

Right, closing this off, as v1.13.2 is out. Discussion on re-enabling these registers for the appropriate inverters is in #760.

@canton7 canton7 closed this as completed Feb 2, 2025
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

Successfully merging a pull request may close this issue.

7 participants