[Aufruf] hm-rega Erweiterung

[lobomau]

Ich habe diesen Rega Adapter 1.6.3 installiert. Ich habe eine CCU2 mit 2.29.22.

ioBroker läuft auf ubuntu-server mit

npm version 3.10.10

node version v6.11.3

Bei mir ist auch das Problem, dass ich DutyCycle zwar anwählen kann es aber aber nicht sichern kann oder etwas in das Feld eintragen kann.

Ausprobiert habe ich schon js-controller Neustart und Host Neustart.

Ansonsten scheint der Rega Adapter so zu laufen wie vorher auch.

[AlGu]

Hallo lobomau,

danke erst mal fürs Installieren und Testen. Ich habe das Problem mit der Einstellen im Adapter jetzt identifiziert und vorerst folgenden Workaround dafür:

• Nach dem Installieren des Adapters über die URL und dem öffnen der Konfiguration im "Instanzen"-Reiter muss auch eine schon vorher vorhandene Einstellung geändert werden, damit die neuen Felder bearbeitet werden können. z.B. eine andere Checkbox "Polling" oder dessen "Intervall" ändern. Danach sollten dann auch die "DutyCycle" Einstellungen möglich sein und mit Abgespeichert werden. Das ganze muss nur einmal so gemacht werden, bis die neuen Einstellungen "DutyCycle" und dessen "Intervall" in ioBroker gespeichert sind. Danach werden beim Laden der Konfig auch die passenden Events abonniert und alles funktioniert wie gewohnt.

Viele Grüße

AlGu

[AlGu]

Nun zum eigentlichen Problem: (ggf. an die Profis gerichtet:)

Wenn ich eine neue Einstellung in dem Adapter einbaue (also in "admin/index.html") dann wir ja beim öffnen der Konfig die Methode "function load(settings, onChange)" ausgeführt. Leider ist dann ja zu diesem Zeitpunkt noch nicht meine neu hinzugekommen Einstellung in der "settings" Property und wird daher nicht mit "setValue" initialisiert und die Events "change" nicht gesetzt.

Ich denke dieses Problem werden schon mehrere Adapter-Entwickler gehabt haben und vielleicht gibt es da ein einheitliches Vorgehen das zu lösen?!

Alternativ kann ich natürlich im Code prüfen ob eine Einstellung nicht in den "settings" beim "load" vorhanden ist und diese dann einzeln initialisieren. Das ist dann aber nicht mehr so schön dynamisch wie bisher und man muss bei jedem neuen Feld daran denken dies dort wieder mit einzubauen. z.B.:

function load(settings, onChange) {
	for (var key in settings) {
		if (settings.hasOwnProperty(key)) setValue(key, settings[key], onChange);
	}

	if(settings["syncDutyCycle"] == null) {
		setValue("syncDutyCycle", false, onChange);
	}
	if(settings["pollingIntervalDC"] == null) {
		setValue("pollingIntervalDC", 60, onChange);
	}

	...
}

Viele Grüße

AlGu

[AlCalzone]

@AlGu:

Ich denke dieses Problem werden schon mehrere Adapter-Entwickler gehabt haben und vielleicht gibt es da ein einheitliches Vorgehen das zu lösen?! `
Relevant: http://forum.iobroker.net/viewtopic.php?f=24&t=7914

tl;dr: Problem ist bekannt. Eine elegante Lösung lässt noch auf sich warten:
> Geofency fixed that by handling it alone. read io-package and parse instanceObjects and create the objects by it's own, but I think js-controller should handle that
Wobei, das war bezogen auf instanceObjects. Bin mir gerade nicht 100% sicher, wie es mit .native (Konfig) aussieht.

[apollon77]

Undefiniert ist "undefined" und nicht "null" Versuch das mal

Idee wäre auch eher die Fehlenden Settings vor der Schleife in das Array mit dem Defaultwert einzufügen.

also eher ein "if (settings.syncDutyCycle===undefined) settings.syncDutyCycle = false;" und dann sid sie für die schleife drin

[AlCalzone]

@apollon77:

Undefiniert ist "undefined" und nicht "null" `
== null prüft sowohl auf undefined als auch null.

=== null funktioniert nur wenn der Wert exakt null ist.

Alternativen wären

typeof settings.syncDutyCycle === "undefined"

und

!("syncDutyCycle" in settings)

[apollon77]

hehe … ok, hast Recht

[AlGu]

Hallo zusammen,

ich habe jetzt die Version 1.6.4 Veröffentlicht mit einem Fix für die Adapter-Konfiguration nach einem Update:

• Wenn der Adapter von einer älteren Version (vor 1.6.0) auf diese Version aktualisiert wird, dann sollten jetzt die neuen Einstellungen auch direkt änderbar sein und nicht mehr erst mit anderen Settings zusammen.

Ich hoffe es gibt noch jemanden, der den Adapter jetzt Testet der noch von der Version kleiner als 1.6.0 kommt. Dann kann ich sicher sein, dass der Adapter nun auch in der Konfig-Seite richtig funktioniert.

@apollon77 und AlCalzone:

Wenn es irgendwann ein einheitliches Vorgehen für Adapter-Default-Werte gibt, dann baue ich das gerne wieder um. Ich denke eine wie auch immer gebaute "adapterDefault"-Liste könnte in Zukunft auch für einen Button "Reset Defaults" oder so nützlich sein.

Viele Grüße

AlGu

[eric2905]

Moin,

@AlGu:

ich habe jetzt die Version 1.6.4 Veröffentlicht…. `
gilt die URL aus dem ersten Posting noch?

Wenn ich die Nutze, bleibt es bei der Version 1.6.3.

Gruß,

Eric

[AlGu]

Moin,

ja die URL ist immer noch dir richtige. Sorry hatte die Version in einer Datei nicht korrekt hochgezählt.

Habe ich nachgeholt.

Danke und Gruß

AlGu

[eric2905]

Jetzt klappt es. Und alles läuft, wie es soll.

Gruß,

Eric

Von unterwegs getippert

[apollon77]

Habs gerade erstmals installiert, also von ner 1.5xer … klappt alles, konnte es einstellen. Super!

[apollon77]

PS:Bitte nur noch am Logging schrauben. Es werden gerade wohl alle Neuen Werte als info geloggt, das ist ein bissl viel … debug reicht dafür. SDag kurz wenn Du mit allem zufrieden bist dann checke ich den PullRequest nochmal genau und gebe Ihn frei!

Super Arbeit!

Ingo F

[eric2905]

Ist mir auch gerade aufgefallen … das Log fliegt nur so vorbei [emoji16]

Von unterwegs getippert

[AlGu]

Ich habe Version 1.6.5 veröffentlicht mit der einzigen Anpassung, dass die Log-Einträge mit "debug" gemacht werden.

Vielen Dank noch mal an alle Tester und wenn es in den nächsten Tagen keine weiteren Fehler oder so gibt, dann mache ich den PullRequest

Viele Grüße

AlGu

[looxer01]

Hi,

ich habe gerade auf Rega 1.6.5 upgedated und den DutyCycle aktiviert mit polling 30s.

Ich habe node version 4.8.1 mit NPM 2.15.11 und eine CCU2 mit version 2.29.22 mit aktivierter community Rega.

Dazu dann noch zwei Gateways. (die Runden)

Ich bekomme einen Fehler. Es sieht so aus, dass die Gateways nicht eingelesen werden ?

Also, den Thread habe ich mir durchgelesen. Ich meine, dass ich alle Voraussetzungen erfülle.

@AlGu Hast du eine eine Ahnung woran es liegen könnte ?

hm-rega.0	2017-09-19 15:13:42.903	error	
hm-rega.0	2017-09-19 15:13:42.903	error	Cannot parse answer for dutycycle: {ADDRESS KEQ0102260 CONNECTED 1 DEFAULT 1 DESCRIPTION CCU2-Coprocessor DUTY_CYCLE 6 FIRMWARE_VERSION 2.8.5 TYPE CCU2} {ADDRESS KEQ0851563 CONNECTED 1 DEFAULT 0 DESC

Edit: noch ein Hinweis. Im Moment nutze ich das TCL Script auf der CCU2. Ich bekomme dort Werte für die Zentrale und für die beiden gateways.

Bei der CCU2 steht in der Fehlermeldung, dass sie einen DC 6 hat. Was korrekt war. Die States werden aber nicht angelegt.

[AlGu]

Hallo loxxer01,

kannst du das Skript hier bitte einmal auf deiner CCU2 (z.B. bei Skript Testen) ausführen und mir das Ergebnis hier (oder per PN) schicken. Dann kann ich damit vielleicht feststellen, warum bei dir das Parsen der Bidcos Antwort nicht funktioniert.

string stderr;
string stdout;
system.Exec("/bin/sh -c '" # 'echo "load tclrpc.so; puts [xmlrpc http://127.0.0.1:2001/ listBidcosInterfaces ]" |tclsh' # "'", &stdout, &stderr);
WriteLine(stdout);

Danke und Gruß AlGu

[looxer01]

…erledigt

der Report sieht auf der CCU Seite ok aus.

Auf der HM Seite nicht sehe ich aber die CCU und einen LanAdapter nicht

Ich verstehe es nicht. Hat möglicherweise was mit meiner Installation zu tun

[AlGu]

Ich denke ich habe den Fehler schon gefunden. Ich Teste das bei mir noch mit deinen Rückgabewerten und dann sag ich Bescheid wenn ich eine Version veröffentliche die du dir dann installieren kannst.

Gruß AlGu

[AlGu]

Hallo loxxon1,

ich habe in meinem Fork: https://github.com/AlGu1/ioBroker.hm-rega einen fix für den Fehler gebaut und veröffentlicht. Wenn du möchtest, kannst du diese Version bei dir gerne mal testen. Das Problem lag an dem Einlesen der Daten, dabei habe ich nicht an allen Stellen mit einem String wie z.B. "Lan Interface" (mit Leerzeichen) gerechnet. Ich habe das Einlesen jetzt etwas umgebaut, so dass es stabiler und toleranter sein sollte.

Danke und Gruß AlGu