[Neuer Adapter] LinkedDevices

paul53

'iobroker update linkeddevices' durchzuführen und adapter neu starten.

Gemacht und anschließend PC neu gestartet: Keine Änderung.

Scrounger

@paul53
ok dann lösch mal die Verlinkung mit 'hm-rpc.0.NEQ1234567.1.STATE', wenn er weiter da hängen bleibt, die nächste Verlinkung usw.
Stell mir mal die raw daten des 'hm-rpc.0.NEQ1234567.1.STATE' zur Verfügung.

paul53

@Scrounger Raw data

{
  "_id": "hm-rpc.0.NEQ1234567.1.STATE",
  "type": "state",
  "common": {
    "name": "NEQ1234567.1.STATE",
    "role": "switch",
    "type": "boolean",
    "read": true,
    "write": true,
    "desc": "Manuell erzeugt",
    "def": false
  },
  "native": {},
  "acl": {
    "object": 1638,
    "state": 1638
  },
  "from": "system.adapter.admin.1",
  "user": "system.user.admin",
  "ts": 1557243244184
}

Log nach Deaktivierung Link für STATE

linkeddevices.0	2019-05-09 20:07:49.471	debug	[createLinkedObject] keep custom settings '{"linkeddevices.0":{"enabled":true,"parentId":"hm-rpc.0.NEQ1234567.1.AUTO_MODE","isLinked":false}}' for linkedObject 'linkeddevices.0.Bad.HKT.Auto_Mode'
linkeddevices.0	2019-05-09 20:07:49.470	debug	[createLinkedObject] using custom name 'Bad HKT Auto' for: 'linkeddevices.0.Bad.HKT.Auto_Mode' (parentObj: 'hm-rpc.0.NEQ1234567.1.AUTO_MODE')
linkeddevices.0	2019-05-09 20:07:49.371	silly	[resetAllLinkedObjectsStatus] linkedObjects status {"linkeddevices.0.Bad.Licht.Taste_kurz":false,"linkeddevices.0.Bad.Fenster":false,"linkeddevices.0.Bad.Licht.Aktor":false,"linkeddevices.0.Bad.HKT.M
linkeddevices.0	2019-05-09 20:07:49.371	debug	[resetAllLinkedObjectsStatus] 'dicLinkedObjectsStatus' items count: 6
linkeddevices.0	2019-05-09 20:07:49.371	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Auto_Mode'
linkeddevices.0	2019-05-09 20:07:49.371	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Control_Mode'
linkeddevices.0	2019-05-09 20:07:49.371	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Manu_Sollwert'
linkeddevices.0	2019-05-09 20:07:49.371	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Licht.Aktor'
linkeddevices.0	2019-05-09 20:07:49.355	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Fenster'
linkeddevices.0	2019-05-09 20:07:49.355	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Licht.Taste_kurz'
linkeddevices.0	2019-05-09 20:07:49.339	info	[initialObjects] started...

Scrounger

Mhhh, ok dann muss ich da mal noch bissle mehr error handling einbauen. Bei mir funktioniert des einwandfrei.
Hast du inzwischen mal auf die Node v8.16.0 geupdatet, kann mir zwar nicht vorstellen das es daran liegt, aber sicher ist sicher.

paul53

@Scrounger Habe mal den Adapter deinstalliert, wieder installiert, Links angelegt, ioBroker (PC) neu gestartet: Es werden keine Datenpunkte unter linkeddevices.0 erstellt. Log:

linkeddevices.0	2019-05-09 21:01:38.864	debug	[createLinkedObject] using custom name 'Bad HKT Auto' for: 'linkeddevices.0.Bad.HKT.Auto' (parentObj: 'hm-rpc.0.NEQ1234567.1.AUTO_MODE')
linkeddevices.0	2019-05-09 21:01:38.763	silly	[resetAllLinkedObjectsStatus] linkedObjects status {}
linkeddevices.0	2019-05-09 21:01:38.763	debug	[resetAllLinkedObjectsStatus] 'dicLinkedObjectsStatus' items count: 0
linkeddevices.0	2019-05-09 21:01:38.747	info	[initialObjects] started...
linkeddevices.0	2019-05-09 21:01:38.747	debug	[onReady] notDeleteDeadLinkedObjects: 'true'
linkeddevices.0	2019-05-09 21:01:38.732	info	starting. Version 0.0.1 in C:/Program Files/ioBroker/node_modules/iobroker.linkeddevices, node: v8.15.1

Scrounger

So hab nochmal bissle was angepasst:

timestamp changes werden berücksichtig
weiteres debugging und error logging

Bitte den aktuellen branch ziehen

paul53

@Scrounger sagte:

Bitte den aktuellen branch ziehen

Gemacht und es funktioniert auch, wenn nur der Zeitstempel aktualisiert wird :+1:

paul53

@Scrounger schrieb:

} else if (tsDifference > 250) {

Weshalb > 250 ms ? HomeMatic PRESS_CONT aktualisiert den Zeitstempel alle ca. 250 ms. Damit kann es ein Problem geben.

Was hälst Du davon, anstelle von state.val, state.ack und state.ts für das Vermeiden einer Dauerschleife state.from auszuwerten ? Etwa so:

async onStateChange(id, state) {
       if (state && state.from != 'system.adapter.' + this.namespace) {
		// parentObject 'state' hat sich geändert -> linkedObject 'state' ändern

paul53

@Scrounger Habe mal folgende Änderung an main.js vorgenommen und erfolgreich getestet:

   async onStateChange(id, state) {
      if (state && state.from != 'system.adapter.' + this.namespace) {
         let changedValue = state.val;
         // parentObject 'state' hat sich geändert -> linkedObject 'state' ändern
         if (this.dicLinkedParentObjects && id in this.dicLinkedParentObjects) {
            let linkedObjId = this.dicLinkedParentObjects[id];

            // let linkedObj = await this.getForeignObjectAsync(linkedObjId);
            // var linkedObjState = await this.getForeignStateAsync(linkedObjId);
            // if (linkedObj && linkedObj.common && linkedObj.common.custom && linkedObj.common.custom[this.namespace] && linkedObj.common.custom[this.namespace].conversion) {
            // 	this.log.debug(`[onStateChange] linkedObject state changed ${linkedObj.common.custom[this.namespace].conversion}`)
            // 	changedValue = eval(`${changedValue}${linkedObj.common.custom[this.namespace].conversion}`)  
            // 	this.log.debug(`[onStateChange] parentObject state changed ${changedValue}`)
            // }

	    await this.setForeignStateChanged(linkedObjId, changedValue, state.ack);
	    this.log.debug(`[onStateChange] parentObject state '${id}' changed to '${state.val}' (ack = ${state.ack}) --> set linkedObject state '${linkedObjId}'`)
         }

         // linkedObject 'state' hat sich geändert -> parentObject 'state' ändern
         else if (this.dicLinkedObjectsStatus && id in this.dicLinkedObjectsStatus) {
            // @ts-ignore
            let parentObjId = Object.keys(this.dicLinkedParentObjects).find(key => this.dicLinkedParentObjects[key] === id);
            
            // let parentObjState = await this.getForeignStateAsync(parentObjId);
            // let linkedObj = await this.getForeignObjectAsync(id);
            // if (linkedObj && linkedObj.common && linkedObj.common.custom && linkedObj.common.custom[this.namespace] && linkedObj.common.custom[this.namespace].conversion) {
            // 	let conversion = this.reverseMathString(linkedObj.common.custom[this.namespace].conversion)
            // 	this.log.debug(`[onStateChange] linkedObject state changed ${conversion}`)
            // 	changedValue = eval(`${changedValue}${linkedObj.common.custom[this.namespace].conversion}*-1`)
            // 	this.log.debug(`[onStateChange] linkedObject state changed ${changedValue}`)
            // }

            // 'custom.isLinked = true'
            if (this.dicLinkedObjectsStatus[id] === true) {
		await this.setForeignStateChangedAsync(parentObjId, changedValue, state.ack);
		this.log.debug(`[onStateChange] linkedObject state '${id}' changed to '${state.val}' (ack = ${state.ack}) --> set parentObject state '${parentObjId}'`)
            }
         }
      }
   }

Scrounger

@paul53

@paul53 sagte in [Neuer Adapter] LinkedDevices:

@Scrounger schrieb:
} else if (tsDifference > 250) {
Weshalb > 250 ms ? HomeMatic PRESS_CONT aktualisiert den Zeitstempel alle ca. 250 ms. Damit kann es ein Problem geben.

Die Entprellzeit von 250ms hab ich gewählt, weil der timestamp ja nur bei Buttons oder zyklischen Statusmeldung geändert wird. Das ein Button innerhalb von 250ms zweimal gedrückt wird erscheint mir unrealistisch, genauso das zyklische Statusmeldungen <250ms überhaupt sinnvoll sind.

@paul53 sagte in [Neuer Adapter] LinkedDevices:

Was hälst Du davon, anstelle von state.val, state.ack und state.ts für das Vermeiden einer Dauerschleife state.from auszuwerten ? Etwa so:

Damit hab ich es auch schon mal probiert, werd aber nochmal testen.

@paul53 sagte in [Neuer Adapter] LinkedDevices:

@Scrounger Habe mal folgende Änderung an main.js vorgenommen und erfolgreich getestet:

Werd ich testen.

Scrounger

@paul53 sagte in [Neuer Adapter] LinkedDevices:

@Scrounger Habe mal folgende Änderung an main.js vorgenommen und erfolgreich getestet:

Fast perfekt ;)
Damit Timestamp Änderungen am linkedObject auch übertragen werden, hab ich es noch etwas modifiziert, 'setForeignStateChangedAsync' in 'setForeignStateAsync' geändert:

	async onStateChange(id, state) {
		if (state && state.from != 'system.adapter.' + this.namespace) {
			let changedValue = state.val;
			// parentObject 'state' hat sich geändert -> linkedObject 'state' ändern
			if (this.dicLinkedParentObjects && id in this.dicLinkedParentObjects) {
				let linkedObjId = this.dicLinkedParentObjects[id];

				// let linkedObj = await this.getForeignObjectAsync(linkedObjId);
				// var linkedObjState = await this.getForeignStateAsync(linkedObjId);
				// if (linkedObj && linkedObj.common && linkedObj.common.custom && linkedObj.common.custom[this.namespace] && linkedObj.common.custom[this.namespace].conversion) {
				// 	this.log.debug(`[onStateChange] linkedObject state changed ${linkedObj.common.custom[this.namespace].conversion}`)
				// 	changedValue = eval(`${changedValue}${linkedObj.common.custom[this.namespace].conversion}`)  
				// 	this.log.debug(`[onStateChange] parentObject state changed ${changedValue}`)
				// }

				await this.setForeignStateAsync(linkedObjId, changedValue, state.ack);
				this.log.debug(`[onStateChange] parentObject state '${id}' changed to '${state.val}' (ack = ${state.ack}) --> set linkedObject state '${linkedObjId}'`)
			}

			// linkedObject 'state' hat sich geändert -> parentObject 'state' ändern
			else if (this.dicLinkedObjectsStatus && id in this.dicLinkedObjectsStatus) {
				// @ts-ignore
				let parentObjId = Object.keys(this.dicLinkedParentObjects).find(key => this.dicLinkedParentObjects[key] === id);

				// let parentObjState = await this.getForeignStateAsync(parentObjId);
				// let linkedObj = await this.getForeignObjectAsync(id);
				// if (linkedObj && linkedObj.common && linkedObj.common.custom && linkedObj.common.custom[this.namespace] && linkedObj.common.custom[this.namespace].conversion) {
				// 	let conversion = this.reverseMathString(linkedObj.common.custom[this.namespace].conversion)
				// 	this.log.debug(`[onStateChange] linkedObject state changed ${conversion}`)
				// 	changedValue = eval(`${changedValue}${linkedObj.common.custom[this.namespace].conversion}*-1`)
				// 	this.log.debug(`[onStateChange] linkedObject state changed ${changedValue}`)
				// }

				// 'custom.isLinked = true'
				if (this.dicLinkedObjectsStatus[id] === true) {
					await this.setForeignStateAsync(parentObjId, changedValue, state.ack);
					this.log.debug(`[onStateChange] linkedObject state '${id}' changed to '${state.val}' (ack = ${state.ack}) --> set parentObject state '${parentObjId}'`)
				}
			}
		}
	}

Wenn deine Test erfolgreich sind, dann pack ich es nachher in den branch

paul53

@Scrounger sagte:

'setForeignStateChangedAsync' in 'setForeignStateAsync' geändert:

Habe die Änderungen übernommen und anschließend alle möglichen Richtungen getestet: Es sieht gut aus !

Scrounger

Änderung ist hochgealden, bitte den neusten branch verwenden.
Hab noch das logging bissle erweitert, damit man auch sieht welche state changes erfolgt sind

@paul53 : Danke für den Input!

paul53

@Scrounger Weshalb erfolgt die Konvertierung nur in eine Richtung (parent --> linked) bei "read only" ?
Eine Änderung in linked schreibt trotzdem in parent; allerdings ohne Konvertierung.
Übrigens: Eval ist böse

			convertedValue = convertedValue.toFixed(obj.common.custom[this.namespace].maxDecimal)

erzeugt einen String !!

Scrounger

@paul53
Folgender Gedanke bei readonly Datenpunkte. Nehmen wir z.B. einen Energiecounter mit Einheit Watt. Im linkedObject will ich aber kWh als Einheit haben. Dann läuft das wie folgt ab: Hardware -> parentObject [Wh] -> linkedObject [kWh]
Ich weiß wenn man per skript in das linkedObject schreibt, dann wird es nicht zurück gewandelt, aber man soll ja auch nicht per skript in read datenpunkte schreiben. Wie siehst du das?

Hab noch für read & write / write datenpunkte Umrechnung eingefügt. Damit ist z.B. möglich eine Hue-Lampe "bri" mit 0-254 (=parentObject) in ein linkedObject das als Eingabe 0-100% aktzeptiert umzuwandeln. Bitte aktuellen branch ziehen hab das nochmal grad was dran geändert.

Übrigens: Eval ist böse

Oh ok, wusste ich nicht, werd ich dann überarbeiten. Weiß jemand was man da alternativ benutzen kann?

paul53

@Scrounger sagte:

Wie siehst du das?

Man sollte sich auf eine lineare Wandlung beschränken, die immer so dargestellt werden kann:

linked = a + b * parent

wobei zwei Eigabefelder erforderlich sind: Eines für a und eines für b. Bei Eingabe von b kann man auch ein führendes "/" erlauben und daraus eine Division 1/b machen.
Diese Vorgehensweise hat den Vorteil, dass man die Konvertierung auch von linked nach parent machen kann und man somit nicht auf "read only" Datenpunkte beschränkt ist:

parent = (linked - a) / b

Scrounger

@paul53 sagte in [Neuer Adapter] LinkedDevices:

wobei zwei Eigabefelder erforderlich sind: Eines für a und eines für b. Bei Eingabe von b kann man auch ein führendes "/" erlauben und daraus eine Division 1/b machen.
Diese Vorgehensweise hat den Vorteil, dass man die Konvertierung auch von linked nach parent machen kann und man somit nicht auf "read only" Datenpunkte beschränkt ist:

Das ist im aktuellen Master schon enthalten, deshalb geht ja z.B. das:

Hab noch für read & write / write datenpunkte Umrechnung eingefügt. Damit ist z.B. möglich eine Hue-Lampe "bri" mit 0-254 (=parentObject) in ein linkedObject das als Eingabe 0-100% aktzeptiert umzuwandeln. Bitte aktuellen branch ziehen hab das nochmal grad was dran geändert.

@paul53 sagte in [Neuer Adapter] LinkedDevices:

Man sollte sich auf eine lineare Wandlung beschränken, die immer so dargestellt werden kann:
linked = a + b * parent
wobei zwei Eigabefelder erforderlich sind: Eines für a und eines für b. Bei Eingabe von b kann man auch ein führendes "/" erlauben und daraus eine Division 1/b machen.

Würde dann aber die Flexibilität nehmen und die usabilty ist dann auch nicht mehr so schön. Ich persönlich finde es sehr nice wenn man einfach eine Formel eingeben kann. Muss ich mal drüber nachdenken wie man das noch anders / besser implementieren kann.

Zunächst muss ich mir ne alternative für eval() suchen.
Wie kann ich denn im adapter ne dependency auf z.B. diese library setzen - math.js - so das die lib automatisch mit installiert wird.
Und wie bekomm ich diese dann in die main.js?

paul53

@Scrounger sagte:

nice wenn man einfach eine Formel eingeben kann.

Das ist nur für die Anwender von Vorteil, die mit Formeln umgehen können und hat den Nachteil, dass die umgekehrte Richtung nicht einfach ermittelt werden kann. @Scrounger sagte in [Neuer Adapter] LinkedDevices:

weil das linkedObjekt ja ein Spiegelbild des parentObjects ist und immer den gleichen Zustand wie das parentObjekt abbilden muss.

SBorg

Bisher soweit :+1:
Eben wollte ich dann einen gelinkten in Node-RED auswählen, nur da erscheint er nicht in der Auswahlliste (bei Blockly bspw. schon). Trägt man ihn per Hand ein funktioniert er aber problemlos.

paul53

@SBorg sagte:

einen gelinkten in Node-RED auswählen, nur da erscheint er nicht in der Auswahlliste

Bei mir schon.

Community Forum

NEWS

[Neuer Adapter] LinkedDevices

Support us

208

32.4k

81.4k

1.3m