Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fahrenheit support #1

Closed
bbrown430 opened this issue Jun 30, 2024 · 4 comments
Closed

Fahrenheit support #1

bbrown430 opened this issue Jun 30, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@bbrown430
Copy link

Describe The Bug:
I am getting an error message that I believe is due to using Fahrenheit instead of Celsius. In the Homebridge web-ui, I am seeing the proper temperature, but in the iOS Home app, it is saying 153 degrees instead of 67 degrees. This makes it impossible to control, as that is outside of the range.

To Reproduce:

Use an A/C configured for Fahrenheit mode.

Expected behavior:

Proper temperature to show in the iOS Home app.

Logs:

[6/30/2024, 8:26:29 PM] [homebridge-connectlife-ac] This plugin generated a warning from the characteristic 'Cooling Threshold Temperature': characteristic was supplied illegal value: number 67 exceeded maximum of 35. See https://homebridge.io/w/JtMGR for more info.

Plugin Config:

{
    "name": "ConnectLife AC",
    "deviceNickName": "Air Conditioner",
    "loginID": "[email protected]",
    "password": "hidden",
    "platform": "ConnectLifeAcPlugin"
}

**Environment:**

* **Plugin Version**: 1.2.1
* **Homebridge Version**: <!-- homebridge -V -->
* **Node.js Version**: v20.15.0
* **NPM Version**: <!-- npm -v -->
* **Operating System**: Debian GNU/Linux Bookworm (12)
@bbrown430 bbrown430 added the bug Something isn't working label Jun 30, 2024
@ghosty2004
Copy link
Owner

Hi.

Thanks for this issue report. I will take a look tomorrow to see what is wrong. If you have access to your ConnectLife app can you try to change the temperature unit to Celsius by navigating to Your AC -> Settings -> Temperature Unit.

After that, test it and then confirm if the issue is from here.

Thanks.

ghosty2004 added a commit that referenced this issue Jul 1, 2024
@bbrown430
Copy link
Author

Awesome, 1.3.0 has fixed this.

Not necessarily related to this issue, but I guess I'm confused about the expected behavior of the plugin in general.

The plugin reads the data fine, and whatever temperature I have the AC unit set to shows up in the Home app perfectly. However, it does not seem to be able to push data. When I set the AC to off in the home app, or adjust the temperature, nothing changes on my window AC unit. Furthermore, if I change the "Cool to" target, and then force close the app, that does not persist either.

Can you tell me what the expected behavior is for these actions?

@ghosty2004
Copy link
Owner

I tested only with split AC.
So you say you have a window AC and when you change the temperature from the HomeKit app it doesn't work?

The expected behavior is to work.

Maybe I will make a Debug option to output the logs from the external API server (from ConnectLife)

@bbrown430
Copy link
Author

I'll open a new issue to keep this thread clean, and I'll share any additional details that could potentially help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants