Test Adapter HAM 5.2.0 (GitHub)

Fabio

@apollon77 ich danke dir ganz herzlich.
So, dank dir läuft jetzt auch die 5.2.2 perfekt mit dem homebridge-tahoma@2.241 Plug in.

iPhilBln

@apollon77 ja bin mir da absolut sicher.
Ich bearbeite die über VS Code mit nem ssh Zugang.

Dein Wrapper Modul lädt die Module unter folgendem Pfad:
/opt/iobroker/node_modules/iobroker.ham/node_modules/homebridge-xiaomi-fan

Die index.js sieht aktuell wie folgt aus:

 /*----------========== HOMEBRIDGE STATE SETTERS/GETTERS ==========----------*/

  getPowerState(callback) {
    let isFanOn = false;
    if (this.fanDevice && this.fanDevice.isFanConnected()) {
      isFanOn = this.fanDevice.isPowerOn();
    }
    this.logInfo(`DEBUG: getPowerState() -> Callback: ${isFanOn}`);
    callback(null, isFanOn ? Characteristic.Active.ACTIVE : Characteristic.Active.INACTIVE);
  }

  setPowerState(state, callback) {
    this.logInfo(`DEBUG: setPowerState() -> value: ${state}`);
    if (this.fanDevice && this.fanDevice.isFanConnected()) {
      let isPowerOn = state === Characteristic.Active.ACTIVE;
      // only fire the setPowerOn method when we want to turn off the fan or the fan is off
      // the rotaion speed slider fires this method many times even when the fan is already on so i need to limit that
      if (isPowerOn === false || this.fanDevice.isPowerOn() === false) {
        this.fanDevice.setPowerOn(isPowerOn);
      }
      this.logInfo(`DEBUG: setPowerState() -> Callback: ${isPowerOn}`);
      callback();
    } else {
      this.logInfo(`DEBUG: setPowerState() -> Callback: cannot set power state`);
      callback(this.createError(`cannot set power state`));
    }
  }

  getFanState(callback) {
    let fanState = Characteristic.CurrentFanState.INACTIVE;
    if (this.fanDevice && this.fanDevice.isFanConnected()) {
      fanState = this.fanDevice.isPowerOn() ? Characteristic.CurrentFanState.BLOWING_AIR : Characteristic.CurrentFanState.IDLE
    }
    this.logInfo(`DEBUG: getFanState() -> Callback: ${fanState}`);
    callback(null, fanState);
  }

  getRotationSpeed(callback) {
    let fanRotationSpeed = 0;
    if (this.fanDevice && this.fanDevice.isFanConnected()) {
      fanRotationSpeed = this.fanDevice.getRotationSpeed();
      fanRotationSpeed = this.adjustToPercentageRange(fanRotationSpeed);
    }
    this.logInfo(`DEBUG: getRotationSpeed() -> Callback: ${fanRotationSpeed}`);
    callback(null, fanRotationSpeed);
  }

  setRotationSpeed(value, callback) {
    this.logInfo(`DEBUG: setRotationSpeed() -> value: ${value}`);
    if (this.fanDevice && this.fanDevice.isFanConnected()) {
      // use debounce to limit the number of calls when the user slides the rotation slider
      if (this.rotationSpeedTimeout) clearTimeout(this.rotationSpeedTimeout);
      this.rotationSpeedTimeout = setTimeout(() => this.fanDevice.setRotationSpeed(value), 500);
      this.logInfo(`DEBUG: setRotationSpeed() -> Callback: ${value}`);
      callback();
    } else {
      this.logInfo(`DEBUG: setRotationSpeed() -> Callback: cannot set rotation speed`);
      callback(this.createError(`cannot set rotation speed`));
    }
  }

Hab es eben nochmal über das Terminal kontrolliert dass auch die richtige index.js editiert wird in VS Code.

Ich logge mit folgendem Befehl:
tail -f /opt/iobroker/log/iobroker.current.log | grep ham.0 > ham.log

Danach kopiere ich es rüber auf meine Mac mit dem Befehl:
scp ham.log philippmielke@10.0.10.4:/Users/philippmielke/Desktop/ham.log

Im Log steht dann folgendes:
states können gesetzt werden: ham.log.zip
states können NICHT gesetzt werden: ham_kaputt.log.zip

apollon77

@iphilbln sagte in Test Adapter HAM 5.2.0 (GitHub):

DEBUG:

Hm ... haste mal nach "DEBUG: set" gegreppt in dem File ... da ist was drin

2022-09-14 13:25:00.723 - [34mdebug[39m: ham.0 (111330) [9/14/2022, 1:25:00 PM] [xiaomifan] [Standventilator] DEBUG: setPowerState() -> value: 0
2022-09-14 13:25:31.921 - [34mdebug[39m: ham.0 (111330) [9/14/2022, 1:25:31 PM] [xiaomifan] [Standventilator] DEBUG: setRotationSpeed() -> value: 30

... ... ...

Gefühlt sendet der das nur raus wenn das gerät auch an ist ...

iPhilBln

@apollon77 du kannst dir das logfile auch einfach in den Chrome Browser ziehen und dann nach Begriffen durchsuchen. So mach ich das aktuell.

rechts wird einem auch angezeigt wo er die Begriffe gefunden hat. Finde ich recht praktisch.

apollon77

Ok, dann hoffe ich das die 5.2.3 jetzt endgültig alle Issues löst

Fabio

@apollon77 Guten Morgen,
musste nur noch im Ham Adapter von homebridge-tahoma@2.241 auf homebridge-tahoma ändern.
Alles läuft sehr gut, keine Fehler Daten werden alle empfangen.

apollon77

@fa-bio Die "Gefahr" wenn Du es ohne Version angibst ist halt das pot der Start des Adapters wieder ein update installiert und was nicht geht ... als vorsichtig sein

Fabio

@apollon77 ja das dachte ich auch, aber als ich den Ham Adapter heute morgen aktualisiert habe, hat er im Log ein Fehler ausgeben wegen der Version Angabe. Dadurch konnte ich dann die Markisen nicht steuern, also habe ich dann die Version Angabe entfernt und dann funktionierte alles wieder wie immer.

apollon77

@fa-bio haste den Fehler noch? Log is ja unter /opt/iobroker/log/...

Fabio

@apollon77 nein leider gelöscht. Jetzt steht da auch wieder von alleine die Version mit drin drin.

iPhilBln

@apollon77 hier wie versprochen mein Feedback zur Race condition:

Adapter abonniert jetzt zuverlässig die states.
Was mich irritiert hat, dass wenn in der config.JSON des Geräts die einzelnen Optionen auf false stehen und man den Datenpunkt im iobroker steuern möchte weil er noch aus einer älteren config existiert, führt das zum Absturz des ham Adapters. Soll heißen, es existieren Datenpunkte im iobroker die zum Absturz des Adapters führen wenn sie auf false in der config stehen.
Ansonsten funktioniert jetzt aber alles einwandfrei. Nochmals riesen Dank.

apollon77

@fa-bio Hm ... aber von alleine tut sie das nicht wirklich ... strange. Naja hauptsache es geht

apollon77

@iphilbln Wie sieht denn der Absturz aus? Bitte mal error log posten ... das sollte an sich nicht passieren ...

iPhilBln

@apollon77 Schick ich dir wenn ich von der Arbeit zurück bin.

Fabio

@apollon77 was soll ich dir sagen so war es aber.

apollon77

@iphilbln Ahhh ... habs lokal mal getestet ... denke wenn du heim kommst hste ne 5.2.4

ETA für 5.2.4 ca. 20-30Mins (die automatisierten tests dauern eewewig)

Fabio

@apollon77 so hab es jetzt mal auf meinem Reservesytem gemacht er macht das alles von ganz alleine egal welche Version ich vorher angeben hatte.


ham.0
2022-09-15 09:01:50.845	info	Adapter uses Wrapper mode

ham.0
2022-09-15 09:01:50.843	info	All Libraries installed/updated

ham.0
2022-09-15 09:01:50.801	info	found 0 vulnerabilities

ham.0
2022-09-15 09:01:50.796	info	run `npm fund` for details

ham.0
2022-09-15 09:01:50.793	info	18 packages are looking for funding

ham.0
2022-09-15 09:01:50.792	info	

ham.0
2022-09-15 09:01:50.788	info	added 75 packages, and audited 76 packages in 1m

ham.0
2022-09-15 09:00:47.803	info	npm install homebridge-tahoma@2.2.28 --production --loglevel error (System call)

ham.0
2022-09-15 09:00:47.801	info	Install/Update homebridge-tahoma@2.2.28

ham.0
2022-09-15 09:00:47.799	info	Install/Update the following Libraries: homebridge-tahoma@2.2.28

ham.0
2022-09-15 09:00:47.762	info	starting. Version 5.2.3 (non-npm: ioBroker/ioBroker.ham) in /opt/iobroker/node_modules/iobroker.ham, node: v16.17.0, js-controller: 4.0.23

apollon77

@fa-bio sagte in Test Adapter HAM 5.2.0 (GitHub):

2022-09-15 09:00:47.799 info Install/Update the following Libraries: homebridge-tahoma@2.2.28

ALso laut dem steht in der konfig diese version

Fabio

@apollon77 ja korrekt hab es gerade geprüft. Kann laut Ham Adapter jetzt auf Version 2.2.41 gehen. Schon komisch

apollon77

@fa-bio Zeig mal die Konfig nochmal vom Adapter bitte

NEWS

Test Adapter HAM 5.2.0 (GitHub)

Support us

1.0k
Online

30.4k
Users

76.7k
Topics

1.2m
Posts

NEWS

Test Adapter HAM 5.2.0 (GitHub)

Support us

1.0kOnline

30.4kUsers

76.7kTopics

1.2mPosts

1.0k
Online

30.4k
Users

76.7k
Topics

1.2m
Posts