[Bug]: Changed Battery, useless warning

TopoDiFogna commented 3 months ago

Describe the issue

I had to change the battery for a faulty hardware, the inverter now reports battery_1_type = 0 which is NONE as stated in huawei-solar.register_values and battery_2_type=2 whi is LUNA2000 from the same file.

The integration reports on startup that Detected two batteries of a different type. This can lead to unexpected behavior, which is true just because battery 1 became NONE

Describe your Huawei Solar Setup

Inverter Type: SUN2000-6KTL-L1 Inverter Firmware version: V200R001C00SPC130 sDongle present: Yes / No sDongle Type: sDongleA-05 (WiFi / Ethernet) sDongle Connectivitiy: WiFi sDongle Firmware: Power meter present:single phase Optimizers Present: Yes Battery: LUNA2000-SO 6kWh Battery Firmware version: Huawei Solar integration version: 1.4.1

How do you connect to the inverter?

Via the SDongle, wireless connection

Upload your Diagnostics File

config_entry-huawei_solar-a4d413360b55e2872fb2e82e748906f3.json

Upload your relevant debug logs

No relevant debug logs, everything is visible from Diagnostic file

Please confirm the following:

[X] I'm running the latest release of Home Assistant.
[X] I'm running the latest release of this integration.
[X] I did not find an existing issue describing this problem.
[X] I did upload the diagnostics-file that I could retrieve from the 'Devices & Services Page'
[X] I did upload the relevant debug logs (via 'Enable Debug Logging'-feature or by manually configuring HA logging)

Roving-Ronin commented 3 months ago

Have you also reapplied the firmware to the LUNA battery? Common issue is that when a new (or replacement) battery module is added to an existing battery stack, the new module has an older revision of firmware upon it, that they causes issues/bad readings for the overall battery.

TopoDiFogna commented 3 months ago

Yes, the battery was update to the latest firmware revision.

The battery stack that I have Is composed of the header, which was replaced, and two 5kW batteries.

The previous header was updated with the batteries at installation time and then I started using the integration.

The second one was installed, updated then the integration was re-enabled. Everything was updated a second time remotely by the installer again few days later while the integration was running.

Il mar 13 ago 2024, 04:42 Roving Ronin @.***> ha scritto:

Have you also reapplied the firmware to the LUNA battery? Common issue is that when a new (or replacement) battery module is added to an existing battery stack, the new module has an older revision of firmware upon it, that they causes issues/bad readings for the overall battery.

— Reply to this email directly, view it on GitHub https://github.com/wlcrs/huawei_solar/issues/791#issuecomment-2285237364, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB7H64VYAY7J6NLNSR5NQD3ZRFXCDAVCNFSM6AAAAABMI37S2SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBVGIZTOMZWGQ . You are receiving this because you authored the thread.Message ID: @.***>

github-actions[bot] commented 2 months ago

This issue has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this issue will be closed in 10 days

TopoDiFogna commented 2 months ago

Reported also in the major dependency of this component: https://gitlab.com/Emilv2/huawei-solar/-/issues/36

Maybe we can close this issue and follow on that repo since the warning originates there

github-actions[bot] commented 1 month ago

This issue has been automatically marked as stale because it has been open 30 days with no activity. Remove stale label or comment or this issue will be closed in 10 days

wlcrs commented 1 month ago

Should be fixed in v1.5.0 alpha 1 and above.

wlcrs / huawei_solar