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

Add 'a'uto Option to 'om' Field #48

Merged
merged 2 commits into from
Jun 22, 2020
Merged

Add 'a'uto Option to 'om' Field #48

merged 2 commits into from
Jun 22, 2020

Conversation

Sunoo
Copy link
Contributor

@Sunoo Sunoo commented Jun 20, 2020

One of the users of my Homebridge plugin reported that at least some COAP-based models use a setting of 'a' in the 'om' field to indicate auto mode, instead of using the 'mode' field.

https://github.com/Sunoo/homebridge-philips-air/issues/9

@Sunoo
Copy link
Contributor Author

Sunoo commented Jun 20, 2020

Sidenote: It would be very helpful to my project to know whether all COAP-based units work this way, or if it's just some subset. If anyone here has any information, I'd love to hear it.

@Cyber1000
Copy link
Contributor

At least for my coap-based device AC2889/10 it doesn't work this way, I'm getting a success message if I send --om a, but nothing happens. Only --mode A (and other mode-switches) work for me.
"Om" only shows the fanspeed for me.

If I want to switch from Auto to manual I need to use a nasty workaround (switch to manual {speed-not-equal-to-current-autospeed} and then to manual speed {what-I-want} - that's the task you have mentioned here.

But there seems to be a lot of different devices ...

@Sunoo
Copy link
Contributor Author

Sunoo commented Jun 21, 2020

@Cyber1000 That’s what I was afraid of, guess it’s gonna be yet another configuration option in my project that people need to determine with trial and error. It would have been nice if Philips had any consistency in their devices. I guess the official app must have some sort of lookup table to determine how to interface with each model or something...

@rgerganov rgerganov merged commit 2c8d9e0 into rgerganov:master Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants