tesla-style-solar-power-card icon indicating copy to clipboard operation
tesla-style-solar-power-card copied to clipboard

Values shown are 10x the actual sensor value

Open MatteoGalet opened this issue 2 years ago • 9 comments

Hello, I noticed on my install the values shown in the bubbles are 10x the actual values calculated from the flows. EG. sensors that constitute the flows are read correctly in HomeAssistant, and add up to 3.2kW (3200w), but bubble shows 32kW (32000w). What is going on there?

MatteoGalet avatar Jun 30 '22 23:06 MatteoGalet

Same issue here, mine is showing 95.3KW when it should be 0.95kw

OneScuzzie2k avatar Jul 08 '22 08:07 OneScuzzie2k

In matter of fact the more entries I put in the more strange values I get!

OneScuzzie2k avatar Jul 08 '22 08:07 OneScuzzie2k

Update: seems like if I use "w" as unit for the flow sensors, the sensor is multiplied by 10x. If I use unit "W" to define the sensor, all is good. Go figure...

MatteoGalet avatar Jul 08 '22 10:07 MatteoGalet

Thanks sorted for me too

OneScuzzie2k avatar Jul 08 '22 16:07 OneScuzzie2k

getting same issue can you expand on how you fixed?

image

tomcoleman avatar Jul 15 '22 17:07 tomcoleman

getting same issue can you expand on how you fixed?

Well, this looks like another different problem...

MatteoGalet avatar Jul 16 '22 02:07 MatteoGalet

with no replies in years to the main code or support / bug questions i think the dev has dropped this project - it could have been really good

tomcoleman avatar Jul 16 '22 05:07 tomcoleman

they exchanged "w" for "W" as stated in the documentation.

And not responding to every configuration error people have does not mean that the project is dead. And just for your information @tomcoleman this is an open source project, meaning 2 things. 1. Most probably it's a hobby for someone and 2. anyone who needs something can contribute.

But going back to the problem, I might be able to change this soon, but not during my vacation, sorry

reptilex avatar Jul 21 '22 16:07 reptilex

they exchanged "w" for "W" as stated in the documentation.

And not responding to every configuration error people have does not mean that the project is dead. And just for your information @tomcoleman this is an open source project, meaning 2 things. 1. Most probably it's a hobby for someone and 2. anyone who needs something can contribute.

But going back to the problem, I might be able to change this soon, but not during my vacation, sorry

cool lets make it the best

tomcoleman avatar Jul 22 '22 05:07 tomcoleman

Is that line probably the source of the 10x value error?

https://github.com/reptilex/tesla-style-solar-power-card/blob/55b06a1890eb059fad12ded85d9d9f1174b4f7ca/src/TeslaStyleSolarPowerCard.ts#L635

edit: sorry, no looks like that's just the width calculation of energy flow

mgerald21 avatar Nov 01 '22 11:11 mgerald21

no, that should divide by 1000 so that when you don't have kW but only W it calculates with kW internally

reptilex avatar Nov 01 '22 14:11 reptilex