Solaredge Local incorrect temperature conversion in dev branch

[adamj12b]

Home Assistant release with the issue:

0.99.3
Last working Home Assistant release (if known):

Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io

Component is installed in the custom component folder for testing. Stock install otherwise.

Component/platform:

Main Component - https://www.home-assistant.io/components/solaredge_local/
#26708 - Caused Issue

Description of problem:
@scheric Temperatures are being converted to Celsius when the inverter is providing temperatures in Fahrenheit. This is causing Home Assistant to convert the temperatures for display in Fahrenheit causing them to be off. Example. Inverter is reporting 78F, HA is displaying 172F (78C converts to 172F)

I change:
"inverter_temperature": [ "invertertemperature", "Inverter Temperature", TEMP_CELSIUS, "mdi:thermometer", ],

to:
"inverter_temperature": [ "invertertemperature", "Inverter Temperature", TEMP_FAHRENHEIT, "mdi:thermometer", ],

This proved my suspicions of double conversion of the temperature, but is not a fix as it would cause a similar issue with inverter reporting in Celsius.

See attached images of what the inverter is displaying and that HA is displaying.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

[scheric]

Are those images taken at the same time?
My program does not convert the temperature. Does HA convert stuff on its own without telling me?

Do you think the optimizer temperature is off to?

Would you mind changing the logger mode to debug and giving me the output or executing this python and giving me the output. And compare these values With HA and with the HTML page.

from solaredge_local import SolarEdge
se_client = SolarEdge("http://[inverter_ip]")

status = se_client.get_status() 
maintenance = se_client.get_maintenance() 

print(f"optimizer data:\n {maintenance.diagnostics.inverters.primary.optimizer[0]}")
print(f"country code: {status.country}")
print(f"inverter data:\n {status.inverters}")

possible fixes:

1. Calculate the temperature so they are only visible in one unit (hotfix)
2. Change the unit during setup. (longer fix)
3. Remove temperature readings (only if people really hate it)

notes:

My HA instance got corrupted so i can not test anything at this moment.
I am located in Europe so my inverter only logs in °C

[adamj12b]

@scheric I am not sure how to verify the optimizer temperature so it is hard to verify that.

The previous images were taken at the same time.

I have run the python script with results below. I have removed serial numbers.

`optimizer data:
serialNumber: "xxxxxxxxx"
online: true
lastReport {
year: 2019
month: 9
day: 30
hour: 14
minute: 53
second: 46
}
outputV: 25
inputV: 56
inputC: 4
temperature {
value: 42
}

country code: 12
inverter data:
primary {
dspSn: "xxxxxxxxx"
voltage: 395.17840576171875
optimizersStatus {
total: 47
online: 47
}
temperature {
value: 159
units {
farenheit: true
}
}
subSystem: 6144
}
`
Here is a screenshot of the inverter status page at the same time as the python script was run.

My Observations:
Optimizer temperature is reported as 42. When converting this to Fahrenheit I get 107F. I would agree based on weather today that the optimizers could be this warm.

Inverter temperature has a flag for Fahrenheit, which may be useful but the optimizers seem to be celsius all the time.

When the optimizers were all off last night, the system reported 32F which makes sense for 0C.

Temperature information should not be omitted from the component IMO.

Additional requests
I would also like to add DC voltage, AC Voltage, AC Frequency, Optimizer Status(total/online), and Import/Export meter sensors. I believe these would be better placed in a new pull request?

[probot-home-assistant]

Hey there @drobtravels, mind taking a look at this issue as its been labeled with a integration (solaredge_local) you are listed as a codeowner for? Thanks!

[adamj12b]

Here is what I am seeing for optimizer information in HA. Inverter is reporting 102F.