hif2k1 / battery_sim

Home assistant home battery simulator - allows you to model how much energy you would save with a home battery
106 stars 13 forks source link

Efficiency calculation bug? #52

Closed twasietsmetjeroen closed 1 year ago

twasietsmetjeroen commented 1 year ago

Hi,

I was wondering if the efficiency is calculated right, I would expect that the "Energy saved" would differ to the "Energy in" sensor. Using the data in my screenshots, I expect something like this: "Energy in" "Efficiency" = "Energy saved" 101,028 0,89 = 89,91492

image image

hif2k1 commented 1 year ago

This was a bug, but I think it has been fixed. Are you using the latest version? If so and it's still a problem let me know.

twasietsmetjeroen commented 1 year ago

I think I have the latest version 1.07, the images above are captured using this version. image

hif2k1 commented 1 year ago

Quite right sorry. I had fixed, but not uploaded the fix. Should be fixed now in 1.08

twasietsmetjeroen commented 1 year ago

Thank you, I've updated to 1.08 and tested it with an extra test battery. It is fixed!