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 Energy Throughput" seems off. #299

Closed
Ndrinta opened this issue May 30, 2023 · 5 comments · Fixed by #300
Closed

"BMS Energy Throughput" seems off. #299

Ndrinta opened this issue May 30, 2023 · 5 comments · Fixed by #300
Labels
bug Something isn't working

Comments

@Ndrinta
Copy link

Ndrinta commented May 30, 2023

Hey @canton7,
may be worthwhile having a look at the scaling for BMS Energy Throughput while you're at it, I'm on version 1.5.2 at the moment.
It seems little off to me that ~700kWh have gone through my battery today!

image

After quick inspection, seems to be a U16 starting at 11049 in my H1.

Thanks!

@canton7
Copy link
Collaborator

canton7 commented May 30, 2023

Thanks! We did originally fix the scale on this, then I seem to have reverted it 4825394.

@canton7 canton7 added the bug Something isn't working label May 30, 2023
@TonyM1958
Copy link

TonyM1958 commented May 30, 2023

FWIW, I tracked this for a while and its in watts, scale : 0.001. Its supposed to be the energy counter that drives BMS Cycle Count, but it is uint16 and wraps every 65kWh, making it hard to use for anything useful.

The metric that I use that makes more sense is a template Battery Energy Total. This is the sum of Charge Energy Total and Discharge Energy Total, similar to the above, but uint32 so it does not wrap,and tracks battery use since install. As BMS Cycle Count also counts from installation, it then makes sense to track Battery Energy per Cycle by dividing Battery Energy Total by BMS Cycle Count.

@canton7
Copy link
Collaborator

canton7 commented May 30, 2023

Yeah, there's a discussion in #110.

We disable this sensor by default these days, precisely because it wraps and is of questionable value -- it's only in for back compat reasons really.

@Ndrinta
Copy link
Author

Ndrinta commented May 30, 2023

Great that will be fixed, thanks, but after seeing the discussion I believe I'll disable it at once.
I think we should close this, I'll let you the onor.

@canton7
Copy link
Collaborator

canton7 commented May 30, 2023

It's already closed :)

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

Successfully merging a pull request may close this issue.

3 participants