Bug in generated_pv_energy_today

[RienduPre]

grott/examples/Home Assistent/grott_ha.py

Line 193 in 198ef47

"value_template": "{{value_json.epvToday| float / 10 }}",

value_json.epvTodaycontains a capital (T) and must be lowercase (t)

[egguy]

Hi, thanks for the bug report.

It seems I was confused by the SPH inverter, which has an epvToday field while the others are epvtoday.

• grott/grottconf.py

  Line 875 in d6f6947

  "epvToday" : {"value" :278, "length" : 4, "type" : "num", "divide" : 10},

I will try to correct it soon in the -current version of the HA addon.

[johanmeijer]

Oké. We can fix this but this can cause an incompatibility for people already using this in their mqtt processing or influxdb.

The solution would be not to change it, but add a second key I think (other name same offset). They can then be used next to it each other.

Backwards compatibility is always a thing to keep in mind ;)

[RienduPre]

WhT’s the status on this?

[egguy]

Hello,
There's now an auto generation of the correct template, this prevents case problem, the epvtoday in the current version is using this function and address this problem.
ex: https://github.com/egguy/grott/blob/feature/ha-extension/examples/Home%20Assistent/grott_ha.py#L167