[Neuer Adapter] hue-extended

[dslraser]

@Zefau
ich habe nur HUE Lampen, aber davon so einige Sorten.

[Zefau]

@dslraser Ok. Und das Problem besteht auch mit v0.5.0? Was steht im Log? Kannst du die Lampen der Gruppen einzeln erfolgreich schalten?

[dslraser]

@Zefau
die 0.5.0 habe ich noch nicht installiert/getestet

[dslraser]

@Zefau
die 0.5.0 startet bei mir gar nicht

host.ioBroker	2019-08-12 21:11:50.647	info	Do not restart adapter system.adapter.hue-extended.0 because desired by instance
host.ioBroker	2019-08-12 21:11:50.647	error	instance system.adapter.hue-extended.0 terminated by request of the instance itself and will not be restarted, before user restarts it.

[Zefau]

@dslraser siehe oben:

Bitte beachten, dass die Zugangsdaten neu eingegeben werden müssen.

[dslraser]

@Zefau
was meinst Du ? Ich hatte sogar die Instanz deinstalliert und eine neue angelegt (die Instanz wurde beim Update auf 0.5.0 nicht upgedatet, da stand installiert 0 (die alte war aber noch da)
In der leeren Instanz waren auch keine Zugangsdaten drinn, die mußte ich neu eingeben.

[AlCalzone]

@Zefau Ich habe mal einen kurzen Blick auf deinen Quellcode geworfen. Für mich scheint es so, dass die Stabilitätsprobleme daher kommen, dass du auf einen Schlag eine große Menge an Datenpunkten erstellst und befüllst. Dieser Prozess läuft bei deinem Adapter nicht sequentiell, sondern parallel ab und zwingt ioBroker in die Knie bzw. überfüllt den Call Stack.

https://github.com/Zefau/ioBroker.hue-extended/blob/master/hue-extended.js#L332
Hier wird im Endeffekt setState in einer Schleife aufgerufen, ohne darauf zu warten, dass der vorherige Befehl abgeschlossen ist.

Du kannst das Ganze relativ elegant lösen, indem du auf die ...Async-Methoden der Adapter-Klasse zurück greifst und async/await nutzt. Also anstatt

for (...) {
    adapter.setState(id, val);
}

was eine große Anzahl an States parallel befüllen will, kannst du folgendes nutzen:

for (...)  {
    await adapter.setStateAsync(id, val);
}

was die States schön der Reihe nacheinander schreibt.

[Hiltex]

@Zefau sagte in [Neuer Adapter] hue-extended:

@Hiltex Also in deinem speziellen Fall ist ein Skript notwendig, dass den hue Wert von Homekit annimmt, diesen in xy umrechnet und an den hue-extended Adapter schickt?

Wie sich herausgestellt hat ist mein Fall garnicht so speziell. Ich habe den Support von innr angeschrieben und folgende Antwort bekommen:

Von: Innr Servicedesk service@innr.com
Betreff: Aw: Steuerung im hue-Modus nicht möglich?
Datum: 13. August 2019 um 16:29:50 MESZ
An: xxxxxx@xxxxxx.com
Antwort an: Innr Servicedesk service@innr.com

Hallo Sebastian,

Ich habe gerade mit unserem technischen Spezialisten gesprochen und er sagte mir, dass fortgeschrittene Farbtonparameter von den Innr Lampe nicht unterstützt werden.

Ich hoffe, das beantwortet Ihre Frage.

Mit freundlichen Grüßen,

Ajay Mahabir
Innr Servicedesk

Ich bin also nicht alleine betroffen, sondern jeder, der innr-lampen an der hue-Bridge betreibt. Gleiches gilt offensichtlich auch für Tradfri-Lampen.

Ich wäre dir super dankbar, wenn du das im Adapter irgendwie berücksichtigen könntest.

Bei OpenHAB wird das mit Tradfri diskutiert und weiter unten im Thread gibt es auch Bash-Script für die Umrechnung von RGB nach XY. Vielleicht kannst du ja darauf aufbauen.
https://community.openhab.org/t/ikea-tradfri-color-changing-bulb-on-hue-bridge-no-control-of-colors/35413/54

Weitere Erläuterungen und Formeln zur Umrechnung auf Github:
https://gist.github.com/popcorn245/30afa0f98eea1c2fd34d

[Zefau]

@AlCalzone Danke für den Hinweis. Wusste gar nicht, dass die Funktionen async/await unterstützen.

Habe es aber anders gelöst. Die aktuelle Version (die noch nicht online ist), werden die States beim Adapter-Start in ein Array geladen und dann nur aktualisiert, wenn diese den Wert verändert haben. Insofern reduzieren sich die eigentlich Schreibvorgänge drastisch.

Würde insgesamt gerne die mögliche Performance nutzen, wenn es möglich ist, und nicht künstlich ausbremsen. Ich hatte Zwischendurch auch eine Variante mit setTimeout( .. , 0), aber im Gegensatz dazu wird async/await wohl schneller sein.

[AlCalzone]

@Zefau setTimeout(..., 0) hat ein ähnliches Problem wie ohne Timeout. Die Callbacks mit Timeout 0 müsten alle im gleichen Moment aufgerufen werden - die Last ist dann nur in den nächsten Event-Loop-Zyklus verschoben.

[Zefau]

@AlCalzone alles klar, danke für die Hinweise. Ich werde bei mir mal die await/async Variante durchprobieren.

[Zefau]

v0.6.0 auf Github und npm

• Implemented queue for any applied actions
• Implemented user creation in interface configuration (admin panel)
• Added additional actions for color spaces rgb, hsv, cmyk, xyz and hex

---

Bitte nach der Installation via

iobroker url https://github.com/Zefau/ioBroker.hue-extended

unbedingt

iobroker upload hue-extended

[Zefau]

@Hiltex habe in der aktuellen Version v0.6.0 eine Option in den Adapter-Einstellungen hinzugefügt, die bei nicht-Philips Lampen den hue state in xy umrechnet. Die Umrechnung basiert auf der node-hue-api, die auch der hue Adapter nutzt. Allerdings ist die Umrechnung insgesamt nicht vollständig präzise, weshalb es zu Abweichungen kommen kann.

[Zefau]

@dslraser kannst du bitte probieren, ob das mit der v0.6.0 behoben ist?

[dslraser]

@Zefau
morgen probiere ich es

[Hiltex]

@Zefau ich danke dir. Allerdings bin ich derzeit auf Reisen und komme erst Ende nächster Woche nach Hause.

[sigi234]

@Zefau sagte in [Neuer Adapter] hue-extended:

v0.6.0 auf Github und npm

• Implemented queue for any applied actions
• Implemented user creation in interface configuration (admin panel)
• Added additional actions for color spaces rgb, hsv, cmyk, xyz and hex

---

Bitte nach der Installation via

iobroker url https://github.com/Zefau/ioBroker.hue-extended

unbedingt

iobroker upload hue-extended

Installiert, alles ok, teste gerade.

Übrigens Paypal ist ok, aber wenn ich auf das Schraubenschlüsselsymbol klicke, sollte es aber auf der Konfigurationsseite starten...........

[Spegeli]

Wenn Sie diesen adapter, Sie ganz herzlich willkommen zu Spenden

Den Satz sollte man noch mal überarbeiten

[Zefau]

@Spegeli sagte in [Neuer Adapter] hue-extended:

Den Satz sollte man noch mal überarbeiten

Die Übersetzung ist eigl. überarbeitet. Siehe https://github.com/Zefau/ioBroker.hue-extended/blob/master/admin/i18n/de/translations.json für deutsche Übersetzungen.
Scheinbar hat der Adapter diese nicht korrekt in die words.js übernommen. Ich gehe da heute Abend nochmal drüber.

Danke für den Hinweis.

[AlCalzone]

@Zefau Du musst noch ein gulp Skript ausführen, um words.js zu aktualisieren.