Sensors Unknown 0.8.4 >

safepay / sensor.fronius

A Fronius Sensor for Home Assistant

MIT License

80 stars 32 forks source link

Sensors Unknown 0.8.4 > #7

Closed cathelest closed 4 years ago

cathelest commented 5 years ago

Hello,

Sorry to be a pain,

Just noticed that something breaks my setup since 0.8.4 >

My config

sensor:

platform: fronius_inverter ip_address: 192.168.1.75 device_id: 1 scan_interval: 20 powerflow: True

0.8.3 Seems to work great, Screenshot

0.8.4 > (Tested 0.8.4, 0.8.5, 0.8.6)

Thanks again for taking time develop this,

Ben

safepay commented 5 years ago

It's probably just not getting values when HA is started/restarted after sunset.

When the inverter data is restored the next day after sunrise, the values should populate.

I probably need to have some sort of "1st run" detection and insert zero's.

bosbefok commented 4 years ago

I've noticed this same issue as well. No values are returned if the system isn't generating AC power. I would have thought at least the energy values should be there. Definitely setting some defaults would be a good idea. Cheers

cathelest commented 4 years ago

Iv jumped back to 0.8.3 from memory and works fine, not sure what changed after that, can u jump back and see aswell?

safepay commented 4 years ago

The change was adding the pause overnight to stop polling. This was to remove errors in the logs.

You can work around this by overriding the pause timing using the start and stop times parameters.

I am unsure of the exact nature of the problem, though. There is not enough detail.

What does "at least the energy values should be there" mean? Which specific values are those?

bosbefok commented 4 years ago

I was referring to sensor.fronius_day_energy, sensor.fronius_year_energy, sensor.fronius_total_energy. All of which were returning unknown if the hassio instance was restarted after there was no AC production. I haven't tried 0.8.7 yet, but will update shortly and let you know of any issues I encounter.

Such a useful addon, thanks heaps for your effort.

bosbefok commented 4 years ago

0.8.7 is running great. Haven't noticed any issues. Would you be able to add a units parameter for the power to allow the user to choose between W and kW so they don't need to setup template sensors. Cheers

safepay commented 4 years ago

Which power sensors do you specifically want to have converted?

Would they be individually converted or is there a single conversion that you would apply to several of them?

bosbefok commented 4 years ago

Which power sensors do you specifically want to have converted?

Would they be individually converted or is there a single conversion that you would apply to several of them?

sensor.fronius_ac_power, sensor.fronius_grid_usage, sensor.fronius_house_load. I'd just like the option to show these 3 in kW instead of W.

safepay commented 4 years ago

sensor.fronius_ac_power, sensor.fronius_grid_usage, sensor.fronius_house_load. I'd just like the option to show these 3 in kW instead of W.

I've uploaded a new sensors.py file to the master branch for you to try.

Use power_units same as for units, but with W, kW or MW. Default is still W.

If this works for you, I'll do a release.

Cheers.

safepay commented 4 years ago

This issues seems to be resolved