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

BMS values missing from sensor.inverter_battery after upgrade #325

Open
toruonu opened this issue Dec 15, 2024 · 3 comments
Open

BMS values missing from sensor.inverter_battery after upgrade #325

toruonu opened this issue Dec 15, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@toruonu
Copy link

toruonu commented Dec 15, 2024

Description

I was running 24.11.02 (I think, can't remember that preciselyl). Then discovered yesterday that out of the blue solarman has a huge amount of updates and decided to update all the way to 24.12.14.

However after restarting HA I discovered, that a crucial data attribute is no longer available. I have Deye SG04LP3 and the battery voltage tends to be reported wrong after the discharge ends. So I was using the attributes from sensor.inverter_battery that had the BMS current and BMS voltage. However those are no longer available. Now it contains only the nominal voltage and max cycle count that seem to be given in the configuration.

When did they disappear and how can I get the actual BMS values now?

Why I need it is that I have the battery charging set to be between 47V and 58.5V. When it discharges down to 47V it nicely stops discharge and settles in the BMS. However the sensor.inverter_battery_voltage seems to drop relatively fast to somewhere around 36V. However if I go to the inverter and check the battery BMS information from the battery LCD display the packs are nicely at 46-47V region with cells relatively balanced. No clue why Deye misreports the voltage. But the attributes that took it from BMS previously had the right numbers so I used those in all my automations and code.

Reproduction Steps

I guess install a prior version and upgrade.

Log

Not relevant

Version

24.12.14

Home Assistant Version

2024.12.3

@toruonu toruonu added the bug Something isn't working label Dec 15, 2024
@amiko24
Copy link

amiko24 commented Dec 15, 2024

I have had a similar problem since the last update. I have read out the registers from address 10032, where the data of the individual battery packs are located. Unfortunately, this no longer works. I have extended ayml file, here as an example
` - name: "Battery 1 Cycle number"
class: "battery"
state_class: "measurement"
uom: " "
scale: 1
rule: 1
registers: [0x2748]
icon: "mdi:battery"

  - name: "Battery 2 Cycle number"
    class: "battery"
    state_class: "measurement"
    uom: " "
    scale: 1
    rule: 1
    registers: [0x276E]
    icon: "mdi:battery"       

  - name: "Battery 1 Max Cel V"
    class: "voltage"
    state_class: "measurement"
    uom: "V"
    scale: 0.001
    rule: 1
    registers: [0x2746]
    icon: "mdi:battery"`

How it was before the update
Davor

this is after the update
Danach

@davidrapan
Copy link
Owner

Hi @toruonu and @amiko24, I'm aware of broken attributes and I'll fix it soon.

But I'm really not sure what could be causing issues w/ regular reading of registers and w/ this so little information I can't even guess.

BTW do not use class: "battery" more than once per (file) device. 😉

@davidrapan davidrapan self-assigned this Dec 15, 2024
@davidrapan
Copy link
Owner

Attributes are fixed in v24.12.16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants