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

installedBatteryCapacity documentation wrong (Ah instead of W) #100

Open
eikowagenknecht opened this issue Oct 24, 2023 · 1 comment
Open

Comments

@eikowagenknecht
Copy link
Contributor

eikowagenknecht commented Oct 24, 2023

In the documentation it says:

...
                    "installedBatteryCapacity": <installed Battery Capacity in W>,
...

Battery capacities are never measured in W, it should be Wh, kWh or Ah.

My system reports

get_system_info():
{
  "installedBatteryCapacity": 33,
}

So at least for the S10XCOMPACT14 it is 33 Ah.

With a voltage of around 400V (this might not be exactly the right number to use, but in the right range) this seems to result in 13,2 kWh, which is at least in the range of the official (net / "usable energy") value of 11,2kWh.

The docs for the battery values contain no units at all:

                    "designCapacity": <design capacity>,
                    "deviceConnected": <device connected>,
                    "deviceInService": <device in service>,
                    "deviceName": <device name>,
                    "deviceWorking": <device working>,
                    "eodVoltage": <eod voltage>,
                    "errorCode": <error code>,
                    "fcc": <full charge capacity>,
                    "index": <batIndex>,
                    "maxBatVoltage": <max battery voltage>,
                    "maxChargeCurrent": <max charge current>,
                    "maxDischargeCurrent": <max discharge current>,
                    "maxDcbCellTemp": <max DCB cell temp>,
                    "minDcbCellTemp": <min DCB cell temp>,
                    "moduleVoltage": <module voltage>,
                    "rc": <rc>,
                    "readyForShutdown": <ready for shutdown>,
                    "rsoc": <relative state of charge>,
                    "rsocReal": <real relative state of charge>,
                    "statusCode": <status code>,
                    "terminalVoltage": <terminal voltage>,
                    "totalUseTime": <total use time>,
                    "totalDischargeTime": <total discharge time>,
                    "trainingMode": <training mode>,
                    "usuableCapacity": <usuable capacity>
                    "usuableRemainingCapacity": <usuable remaining capacity>

Might be good to add correct units there as well. For reference about the units, these are the values shown in RscpGui:

image
image

@vchrisb
Copy link
Collaborator

vchrisb commented Oct 24, 2023

Yes, W is not correct.
For me it is Wh and the value is exactly the expected one. But this doesn't make sense in your case.

A PR to improve the docs is always appreciated.

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

No branches or pull requests

2 participants