[Neuer Adapter] LinkedDevices

paul53

immer den gleichen Zustand wie das parentObjekt abbilden muss.

Das ist Ansichtssache. Wenn Du es so dokumentierst, wird es sicherlich akzeptiert und Nachfragen vermieden.

paul53

@Scrounger Der Zustand eines Datenpunktes beinhaltet auch den Zeitstempel. Jetzt erfolgt die Übertragung nur bei einer Wertänderung, nicht jedoch, wenn nur der Zeitstempel aktualisiert wird - wie es bei Tasten (z.B. PRESS_SHORT) der Fall ist.

paul53

Nach 2 Tagen nochmal getestet und es reagiert kein Datenpunkt mehr auf ein Veränderung der anderen Seite (parent / linked). Im Log finde ich nur noch das:

linkeddevices.0	2019-05-09 17:00:32.774	debug	[createLinkedObject] keep custom settings '{"linkeddevices.0":{"enabled":true,"parentId":"hm-rpc.0.NEQ1234567.1.STATE","isLinked":false}}' for linkedObject 'linkeddevices.0.Bad.Licht.Aktor'
linkeddevices.0	2019-05-09 17:00:32.772	debug	[createLinkedObject] using custom name 'Bad Lampe' for: 'linkeddevices.0.Bad.Licht.Aktor' (parentObj: 'hm-rpc.0.NEQ1234567.1.STATE')
linkeddevices.0	2019-05-09 17:00:32.630	debug	[resetAllLinkedObjectsStatus] 'dicLinkedObjectsStatus' items count: 6
linkeddevices.0	2019-05-09 17:00:32.630	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Auto_Mode'
linkeddevices.0	2019-05-09 17:00:32.630	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Control_Mode'
linkeddevices.0	2019-05-09 17:00:32.630	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Manu_Sollwert'
linkeddevices.0	2019-05-09 17:00:32.614	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Licht.Aktor'
linkeddevices.0	2019-05-09 17:00:32.614	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Fenster'
linkeddevices.0	2019-05-09 17:00:32.614	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Licht.Taste_kurz'
linkeddevices.0	2019-05-09 17:00:32.599	info	[initialObjects] started...
linkeddevices.0	2019-05-09 17:00:32.599	debug	[onReady] notDeleteDeadLinkedObjects: 'true'
linkeddevices.0	2019-05-09 17:00:32.552	info	starting. Version 0.0.1 in C:/Program Files/ioBroker/node_modules/iobroker.linkeddevices, node: v8.15.1
linkeddevices.0	2019-05-09 17:00:32.498	debug	statesDB connected
linkeddevices.0	2019-05-09 17:00:32.444	debug	objectDB connected

Scrounger

@paul53 sagte in [Neuer Adapter] LinkedDevices:

@Scrounger Der Zustand eines Datenpunktes beinhaltet auch den Zeitstempel. Jetzt erfolgt die Übertragung nur bei einer Wertänderung, nicht jedoch, wenn nur der Zeitstempel aktualisiert wird - wie es bei Tasten (z.B. PRESS_SHORT) der Fall ist.

Stimmt bei buttons macht der Zeitstempel natürlich sinn. Bitte schick mir mal die raw Daten des buttons.

@paul53 sagte in [Neuer Adapter] LinkedDevices:

Nach 2 Tagen nochmal getestet und es reagiert kein Datenpunkt mehr auf ein Veränderung der anderen Seite (parent / linked). Im Log finde ich nur noch das:

Laut log bleibt der adapter beim initializieren hängen. Hast du den neu gestartet oder komplett zwei Tage am Stück durchlaufen lassen? Restart hilft auch nicht?

paul53

@Scrounger sagte:

die raw Daten des buttons

{
  "type": "state",
  "common": {
    "def": false,
    "type": "boolean",
    "read": false,
    "write": true,
    "role": "button",
    "name": "Taste 2 kurz"
  },
  "native": {
    "CONTROL": "BUTTON.SHORT",
    "DEFAULT": false,
    "FLAGS": 1,
    "ID": "PRESS_SHORT",
    "MAX": true,
    "MIN": false,
    "OPERATIONS": 6,
    "TAB_ORDER": 0,
    "TYPE": "ACTION",
    "UNIT": ""
  },
  "from": "system.adapter.hm-rpc.0",
  "ts": 1556192106793,
  "_id": "hm-rpc.0.PEQ0371234.1.PRESS_SHORT",
  "acl": {
    "object": 1636,
    "state": 1636,
    "owner": "system.user.admin",
    "ownerGroup": "system.group.administrator"
  }
}

Trotz common.def: false ist der Wert immer true.

@Scrounger sagte in [Neuer Adapter] LinkedDevices:

Hast du den neu gestartet

Ja.

Scrounger

@paul53
Stell mal den adapter auf silly und mach nen neustart. Wenn er dann auch nicht komplett durchläuft, d.h. am Ende muss '[initialObjects] finished' im log stehen, dann deaktivere 'Nicht mehr verlinkte Objekte nicht löschen' und nochmal neustarten und logs posten, danke!

paul53

Ergebnis:

linkeddevices.0	2019-05-09 19:20:56.711	debug	[createLinkedObject] keep custom settings '{"linkeddevices.0":{"enabled":true,"parentId":"hm-rpc.0.NEQ1234567.1.STATE","isLinked":false}}' for linkedObject 'linkeddevices.0.Bad.Licht.Aktor'
linkeddevices.0	2019-05-09 19:20:56.710	debug	[createLinkedObject] using custom name 'Bad Lampe' for: 'linkeddevices.0.Bad.Licht.Aktor' (parentObj: 'hm-rpc.0.NEQ1234567.1.STATE')
linkeddevices.0	2019-05-09 19:20:56.622	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 19:20:56.622	debug	[resetAllLinkedObjectsStatus] 'dicLinkedObjectsStatus' items count: 6
linkeddevices.0	2019-05-09 19:20:56.622	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Auto_Mode'
linkeddevices.0	2019-05-09 19:20:56.606	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Control_Mode'
linkeddevices.0	2019-05-09 19:20:56.606	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.HKT.Manu_Sollwert'
linkeddevices.0	2019-05-09 19:20:56.606	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Licht.Aktor'
linkeddevices.0	2019-05-09 19:20:56.606	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Fenster'
linkeddevices.0	2019-05-09 19:20:56.591	debug	[resetLinkedObjectStatus] 'isLinked' status reseted for 'linkeddevices.0.Bad.Licht.Taste_kurz'
linkeddevices.0	2019-05-09 19:20:56.575	info	[initialObjects] started...
linkeddevices.0	2019-05-09 19:20:56.575	debug	[onReady] notDeleteDeadLinkedObjects: 'false'
linkeddevices.0	2019-05-09 19:20:56.544	info	starting. Version 0.0.1 in C:/Program Files/ioBroker/node_modules/iobroker.linkeddevices, node: v8.15.1
linkeddevices.0	2019-05-09 19:20:56.544	debug	statesDB connected
linkeddevices.0	2019-05-09 19:20:56.528	debug	objectDB connected

Scrounger

@paul53
Ok hängt genau an der gleichen Stelle und schlauer werd ich daraus auch nicht.
Versuch mal nen 'iobroker update linkeddevices' durchzuführen und adapter neu starten. Wenn das nicht hilft, dann mal iobroker komplett neu starten.

paul53

@Scrounger sagte:

'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

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 !

NEWS

[Neuer Adapter] LinkedDevices

Support us

1.2k
Online

30.7k
Users

77.3k
Topics

1.2m
Posts

NEWS

[Neuer Adapter] LinkedDevices

Support us

1.2kOnline

30.7kUsers

77.3kTopics

1.2mPosts

1.2k
Online

30.7k
Users

77.3k
Topics

1.2m
Posts