Skip to content
This repository has been archived by the owner on May 17, 2021. It is now read-only.

Add support for 'On/Off plug-in unit' in hue binding #3202

Merged
merged 1 commit into from
Oct 8, 2015

Conversation

tobiaswegner
Copy link
Contributor

This patch adds support for hue devices that offer only on/off switching, no dimming. They deliver no brightness information so without this change, an exception is thrown. The patch returns 0 or 254 based on the "on" state of the hue device.

@kaikreuzer
Copy link
Member

@tobiaswegner Just out of curiosity: What hue devices are there that aren't dimmable?

@tobiaswegner
Copy link
Contributor Author

Plug adapters like this http://www.busch-jaeger-katalog.de/6710-0-0004,artikel.html or wall mounted actuators like http://www.busch-jaeger-katalog.de/6710-0-0002,artikel.html that are connected e.g. to your multimedia system instead of a lamp.
I think even with the genuine Philips plug adapter http://www.philips.de/c-p/6916531PH/livingwhites-adapter dimming can be disabled via a mechanical switch if dimming makes no sense with the particular load.

@kaikreuzer
Copy link
Member

Ah, cool, I didn't know the Busch Jäger products yet! But 120 EUR for a smart plug? They should revisit their price list...

I think even with the genuine Philips plug adapter http://www.philips.de/c-p/6916531PH/livingwhites-adapter dimming can be disabled

You are right, I am actually using quite some of them (but never tried out the switch mode and how that behaves).

@tobiaswegner
Copy link
Contributor Author

Ah, cool, I didn't know the Busch Jäger products yet! But 120 EUR for a smart plug? They should revisit their price list...

Yes, they are only a few weeks old. They price is as expected from Busch-Jaeger, but at least it is noticeable lower than the KNX Powernet devices. Anyway, I agree, it is no bargain buy.

@teichsta teichsta added this to the 1.8.0 milestone Oct 8, 2015
teichsta added a commit that referenced this pull request Oct 8, 2015
Add support for 'On/Off plug-in unit' in hue binding
@teichsta teichsta merged commit 8bc758d into openhab:master Oct 8, 2015
@teichsta
Copy link
Member

teichsta commented Oct 8, 2015

Thanks @tobiaswegner! Is that functionality already implemented in the ESH version of the binding? I know it's a rewrite but it might be worth having into that code. Would you mind @tobiaswegner checking double checking the ESH code?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants