bigmoby / fglair_for_homeassistant

❄️ FGLair heat pump controller integration
MIT License
52 stars 4 forks source link

temperathure in app not the same as on HA #15

Closed Yakie996 closed 1 year ago

Yakie996 commented 1 year ago

Hi,

The temperature in the app displays 18C and in HA it displays 20C and rarely updates.

Kind regards

chiefcomm commented 1 year ago

@Yakie996 please install version v0.1.4 or higher and add the following to configuration.yaml.

logger:
  default: critical
  logs:
    custom_components.fglair_heatpump_controller.climate: debug
    homeassistant.components.climate: debug

I'm really interested in the lines below from your logs and the attributes from HA.

2023-01-13 07:39:14.484 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Display_temperature json: {'value': 7500, 'key': 93872419}
2023-01-13 07:39:14.484 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Display_temperature: 7500
2023-01-13 07:39:14.484 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Region: au

image

bigmoby commented 1 year ago

Yes @chiefcomm also for me there's an offset of +2 celsius degree

chiefcomm commented 1 year ago

@bigmoby the API from where we extract the current temp does seems to report different data than the mobile app. Can you confirm that the data in the logs is exactly the same as what shows in HA?

2023-01-14 09:31:59.611 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Display_temperature json: {'value': 7400, 'key': 93872419}
2023-01-14 09:31:59.611 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Display_temperature: 7400
2023-01-14 09:31:59.611 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Region: us

image

chiefcomm commented 1 year ago

same again in Celsius

2023-01-14 09:45:12.479 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Display_temperature json: {'value': 7400, 'key': 93872419}
2023-01-14 09:45:12.479 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Display_temperature: 7400
2023-01-14 09:45:12.480 DEBUG (MainThread) [custom_components.fglair_heatpump_controller.climate] Region: us

image

image

bigmoby commented 1 year ago

Hi @chiefcomm , current_temperature: 19.4 and in the logs Display_temperature json: {'value': 6700, 'key': 80400804} So the conversion is right 👍, but in fgliar app 17.0 Cis shown. I think 17 degrees is the correct room's temperature.

chiefcomm commented 1 year ago

Thanks at least it confirms that everything is converting OK - as the API is undocumented and has been reverse engineered this is the only viable option.

The API gives 3 temperature I could extract only one made sense as the current temperature, looking at other reverse engineered plugins all are using the display_temperature. Most likely this isn't the same sensor the the Mobile App is using (it is impossible to tell as the API is all we have) - the question, is this a big deal and can people live with this or do we turn it into an option to display (or not).

adjust_temperature an input display_temperature an output outdoor_temperature an output (in my case none reported)

At this minute with my A/C turned off the Mobile App is saying 28.0°C and HA is displaying 24.3°C, for sure 28.0°C is nearer the real temp in Sydney today :-)

bigmoby commented 1 year ago

Hi @chiefcomm and thanks again for your contributions! 🙌 All my "envy" of Sydney's current temperatures aside, here in Sicily (Italy) is winter and too cold for me! 😜 Please could you give me feedback about this "daily" release? https://github.com/bigmoby/fglair_for_homeassistant/compare/v0.1.6...v0.1.7

chiefcomm commented 1 year ago

Hi @bigmoby oh nice I never managed to get to Sicily in my travels, I'll add it to my list of places to visit. Latest release is one way to handle it and gives more flexibility so yes lets go with that and see if it solves for people. I may have mentioned earlier in that I think this is probably a completely different sensor in the unit, for me the variance has been so minor I never bothered to worry about it (I never look at the mobile app).

Yakie996 commented 1 year ago

Hi, sorry I dit not had the time last couple of days, thanks for update!

chiefcomm commented 1 year ago

Looks good @bigmoby, stuck indoors today it's a bit warm outside (about 31°C) so sitting inside with A/C on and testing the new release out. As I said earlier I have very marginal differences between the App 23.0°C vs HA 22.9°C but the offset does appear to work in test were I added a +0.2°C offset :-)

image

image

image

bigmoby commented 1 year ago

Hi @bigmoby oh nice I never managed to get to Sicily in my travels, I'll add it to my list of places to visit.

Sicily is a wonderful place! You're welcome! 🙌 ...but in summer it's better! 😅 Oook, so I suppose we could close this issue 🚀