Units about tesla-style-solar-power-card

[dgomes]

Therefore the importance of showing the unit.

This is very common in various energy monitor interfaces.

But again, this can be a configuration variable.

[msim888]

+1
Great job!
Would be very good to have that option - I found how to switch to W instead kW but it doesn't work very good so far.

[reptilex]

I think I got this covert now. Fresh from the oven, define use_w_not_kw in your yml configuration like this:
use_w_not_kw: 1
and you will have w instead of kw and it will not be divided by 1000. Your sensors do have to have the w unit though.
Please test it, if it works close the issue ;)

[dgomes]

Thank you soo much for considering this :)

But

• its not W vs kW
• its W if power < 1000 and kW if power > 1000

[reptilex]

won't this look funny when you have 30W solar power and you are using 2 kW? It would confuse the heck out of me. The change is just adding a var to the if statement but I find it's not user friendly that way.

[msim888]

show_w_not_kw: 1 works very well but there is a problem with speed of cycles - it's too high when show_w_not_kw: 1

I guess the code should be something like this:

getSpeed(value) {
var speed = 0;
if (value > 0 && this.w_or_kw !== 'W') {
speed = 0.07 * value;
} else if (value > 0 && this.w_or_kw === 'W') {
speed = 0.00007 * value;
}
return speed;
}

[msim888]

With the code above the speed is normal and everything looks good.
But I have some negative power values - I can use Math.abs() method of course but would be very good to have that option as config option. I mean the option to use negative power values.

[reptilex]

Your unreal "pull request" has been merged ;) Do you need a new release to test it? After that I would close the issue and create a new release

[msim888]

Sorry, I'm not very good familiar with development process and github. Maybe a new release will be better. Thanks!