[Neuer Adapter] BYD HVS

JB_Sullivan

@tueftler17
Kann die neue Version über eine bestehende Version installiert werden, oder sollten alle alten Datenpunkte gelöscht werden (was Schade wäre, da es dort div. Datenbank Verknüpfungen [Influxdb] gibt) ?

EDIT:

Auch bei deinem Adapter gibt es ein Aktualisierungs Problem seit dem Admin 5.x.x/js-controller 3.3.x Update.

Die bestrifft aber offensichtlich nur User die ioB unter Windows laufen haben.
Es kommt aus ioB Adapter Verwaltung heraus zu einem EBUSY Fehler. Ursache ist derzeit unbekannt, da es auch nicht jeden Adapter betrifft.

Wenn man iobroker stoppt und das Adapter Update über die Konsole macht funktioniert es üblicher Weise.

Wie das allerdings bei Konsolen Updates über die Katze ist , wo sich auch die Versionsnummer nicht geändert hat, kann ich erst heute Abend vor Ort ausprobieren.

$ ./iobroker url "https://github.com/christianh17/ioBroker.bydhvs"
install christianh17/ioBroker.bydhvs#7b9189e50efe5b2cb7d366a73ad617d5aba40fad
NPM version: 6.14.11
npm install christianh17/ioBroker.bydhvs#7b9189e50efe5b2cb7d366a73ad617d5aba40fad --loglevel error (System call)
npm ERR! code EBUSY
npm ERR! syscall rename
npm ERR! path C:\iobroker\GLT\node_modules\iobroker.bydhvs
npm ERR! dest C:\iobroker\GLT\node_modules\.iobroker.bydhvs.DELETE
npm ERR! errno -4082
npm ERR! EBUSY: resource busy or locked, rename 'C:\iobroker\GLT\node_modules\iobroker.bydhvs' -> 'C:\iobroker\GLT\node_modules\.iobroker.bydhvs.DELETE'

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\iobroker\GLT\env\npm-cache\_logs\2021-05-14T11_15_30_059Z-debug.log
ERROR: host.GLT(GLT) Cannot install christianh17/ioBroker.bydhvs#7b9189e50efe5b2cb7d366a73ad617d5aba40fad: 4294963214
ERROR: process exited with code 25

JB_Sullivan

.... bei der Gelegenheit - ich habe letzte Woche meinen Speicher von 3 auf 4 Module aufgerüstet.

Auf BYD Seite hat alles einwandfrei funktioniert, was im Nachgang auch bei deiner Adapter Version 1.1.0 der Fall war. Die Temperatur Werte wurde auf 48 Datenpunkte und die Spannungswerte auf 128 Datenpunkte ergänzt.

tueftler17

@jb_sullivan Da habe ich aufgepasst :-). Du kannst alles so lassen. Das Einzige was ich nicht mehr versorge ist "State.ErrorNum", das ist nach "System.ErrorNum" gewandert. Ich habe nur keine Möglichkeit gefunden das im Adapter zu löschen. Daher: Wenn Du diesen Datenpunkt manull löscht ist alles mit den States perfekt.

In alten Versionen wurden möglicherweise zu viele Zellspannungen und Zelltemperaturen als State angelegt. Vielleicht noch die Überflüssigen löschen, ich bin mir aber nicht sicher ob es überhaupt eine Version außerhalb meiner Entwicklungsinstanz gab bei der das passiert ist.

Danke für den schnellen Test!

EDIT: Mein Browser hatte Dein Update nicht angezeigt. Es wäre prima wenn Du es ausprobieren könntest. Ich habe aber keine Idee was ich ändern sollte, es scheint es wird das "Modul-Verzeichnis" umbenannt. Das ist aber ein NPM-Ding, ich wüsste nicht was ich tun sollte kopfkratz.

JB_Sullivan

@tueftler17 sagte in [Neuer Adapter] BYD HVS:

Es wäre prima wenn Du es ausprobieren könntest. Ich habe aber keine Idee was ich ändern sollte, es scheint es wird das "Modul-Verzeichnis" umbenannt. Das ist aber ein NPM-Ding, ich wüsste nicht was ich tun sollte kopfkratz.

Würde ich gerne, ich weiß aber nicht, wie man nur über die Konsole eine GIT Installation anstößt.

Üblicherweise mache ich das immer mit "iobroker upgrade". Dann werden mir die Adapter aufgezeigt die geupdatet werden können. Dein Adapter ist aber nicht offiziell gelistet, somit taucht er nicht in der Liste der updatefähigen Adapter auf - zumal du ja auch die Versionsnummer nicht geändert hast.

Wird also schwierig.

Zum Thema EBUSY - dieses Phänomen betrifft scheinbar nur User die ioB unter Windows laufen haben.

Die Frage ist, was in den Ablaufroutinen anders ist, als bei anderen Adapter. So z.B. der BackitUp Adapter läßt sich aus dem Admin heraus ohne Probleme installieren. Auch der Admin Adapter läßt sich problemlos installieren.

Dann gibt es aber noch eine ganze Reihe andere Adapter (incl. deinem) die bei der Installation den EBUSY Fehler verursachen und welche sich nur bei gestopptem ioBroker Windows Dienst via Konsolen Befehlen aktualisieren lassen.

Wie gesagt, alles erst seit Admin 5.x.x & js-controller 3.3.x

tueftler17

@jb_sullivan Hi! Heute habe ich den Schritt gewagt:

Der Adapter hat eine neue Version (1.2.0)
NPM ist aktualisiert
ich habe um Aufnahme in die offizielle Adapterliste gebeten, hoffe auf baldige "Zustimmung".
Installation / Update ist jetzt entweder über die Katze oder die Kommandozeile möglich

npmi i iobroker.bydhvs

JB_Sullivan

@tueftler17

Nàbend - habe heute mal versucht dein Adapter Update zu machen. Klassisch über Katze geht ja nicht, wegen EBUSY - siehe oben. Also über npm und die Konsole. Dabei kam es zu der folgenden Meldung.

Ist das ein Problem? - Wie ich eben gesehen habe, ist die Version auf 1.2.0 hochgesetzt worden. Adapter läuft soweit und aktualisiert sich auch. State.Erro.num habe ich gelöscht. Sieht eigentlich alles schick aus :+1:

C:\iobroker\GLT>npm install iobroker.bydhvs@1.2.0
npm WARN jscs-loader@0.3.0 requires a peer of jscs@^3.0.3 but none is installed. You must install peer dependencies yourself.
npm WARN xmlhttprequest-ts@1.0.1 requires a peer of @angular/common@>= 5.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN xmlhttprequest-ts@1.0.1 requires a peer of @angular/core@>= 5.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN typedoc@0.20.28 requires a peer of typescript@3.9.x || 4.0.x || 4.1.x but none is installed. You must install peer dependencies yourself.
npm WARN typedoc@0.20.28 requires a peer of typescript@3.9.x || 4.0.x || 4.1.x but none is installed. You must install peer dependencies yourself.

+ iobroker.bydhvs@1.2.0
removed 2 packages, updated 2 packages and audited 2518 packages in 30.807s

139 packages are looking for funding
  run `npm fund` for details

found 200 vulnerabilities (17 low, 109 moderate, 74 high)
  run `npm audit fix` to fix them, or `npm audit` for details

C:\iobroker\GLT>

Thomas Braun

@jb_sullivan sagte in [Neuer Adapter] BYD HVS:

found 200 vulnerabilities (17 low, 109 moderate, 74 high)

Nur aus Neugierde:
Welche Version von nodeJS läuft denn da?
Bei mir wird lediglich das gemeldet:

19 vulnerabilities (7 low, 2 moderate, 10 high)

JB_Sullivan

@thomas-braun 14.16

EDIT - sollte ich irgend etwas was dort vorgeschlagen wurde (audit fix / npm fund) machen?

Thomas Braun

@jb_sullivan
Nee, damit machst du tendienziell eher den ioBroker 'kaputt'. Da würde ich die Finger von lassen.

Die Fixes kommen mit neuen Versionen und aktualisierten Dependencies 'von alleine' rein. Sein nodeJS und die iobroker-Adapter sollte man allerdings aktuell halten.

JB_Sullivan

@thomas-braun Oh der Broker und die Adapter sind aktuell, das kannst du mir glauben.

Habe zwei Session laufen - einmal als Produktivsystem und dann eine Spiegelung davon die gnadenlos geupdatet wird sobald was neues raus kommt. Gibt es über Wochen keine Probleme mit dem "Spiegel" ziehe ich das Produktiv System hinterher.

Aktuell wird aber nur die Spiegelung auf dem laufenden gehalten. Ist mir im Moment zuviel "Umekehr" bzgl. Admin 5.x.x und js-controller.

Aber die Spiegelung , wo ich jetzt auch den BYD Adapter aktualisiert habe, ist Brand aktuell ;)

Thomas Braun

@jb_sullivan
Dein nodeJS aber nicht! :-P

JB_Sullivan

@thomas-braun Naja, im vergleich zu dem was Empfohlen wird aber schon - da will ich nicht zu viele Schritte voraus sein. Nicht das es ggf. dadurch wieder zu neuen unerwarteten Phänomen kommt.

Thomas Braun

@jb_sullivan
Auf diesem ominösen Windows soll das ganze ja auch eher 'pain in the lower back' sein, hab ich mir sagen lassen. Aber mit dem System kenn ich mich ja auch nicht aus.

JB_Sullivan

@thomas-braun sagte in [Neuer Adapter] BYD HVS:

'pain in the lower back'

Jeder spielt das Instrument was er am besten beherrscht und wenn man sich die Installstatistik von ioB anschaut, sind ~ 1/3 Windows Installationen. Wie viele davon aktiv "gespielt" werden und wie viele nur für den Einstieg waren, ist ja eine andere andere Sache.

Aber ICH komme damit gut zurecht und ich denke @sigi234 auch. Der ist ja auch so ein Windows Kandidat. Auf jeden Fall geht es für mich unter Win schneller und sicherer als wenn ich mich in Linux einarbeiten müsste - wo mir auch ein Stück weit die Zeit zu fehlt.

..... aber das ist ein anderes Thema und wir sollten das damit gut sein lassen. Es geht in diesem Thread ja schließlich um den BYDHVS Adapter ;)

tueftler17

@jb_sullivan @Thomas-Braun
Oh, da kenne ich mich bisher auch nicht groß aus. Ich habe im github diverse Warnungen bekommen das die von mir referenzierten Bibliotheken veraltet seien und ich zwecks Erhöhung der Sicherheit bitte updaten möge. Das habe ich brav getan und bei mir gab es dann auch keine weiteren Fehlermeldungen.

Ich muss dazu mal im Forum suchen ob ich etwas dazu finde.

Melde mich...
Tueftler17

JB_Sullivan

@tueftler17 Kannst du ggf. im Adapter die Anzahl der Nackommastellen schon auf 2 oder 3 begrenzen? Ich glaube so viele Stellen brauch man nicht wirklich - oder?

qqolli

@jb_sullivan

Hi,

komisch, bei mir hat der DP "bydhvs.0.State.Power" bereits nur 2 Stellen (nicht in der VIS gerundet).

tueftler17

@qqolli In der nächsten Version werde ich es zwangsweise auf 2 Stellen runden. Jetzt ist es eine Multiplikation von zwei Festkomma-Zahlen und ich dachte nicht, das da so komische Werte herauskommen können.
Aber ich hoffe darauf das sich jemand meinem PR annimmt, dann wäre ich endlich auch im offiziellen Repository zu finden und wollte in der Zeit eigentlich keine Änderungen durchführen.

Außer: Es stört irgendwo in der Weitererarbeitung, dann mache ich es eben - habe es schon auf meinem Rechner korrigiert, aber noch nicht hochgeladen und als neue Version bei NPM erzeugt.

Ansonsten: Bei den anderen Fehlermeldungen nach dem Update der ganzen Abhängigkeiten. Was meint Ihr? Soll ich einfach wieder auf die alte Version zurück gehen? Ich habe keine Features der neuen Versionen genutzt.

Viele Grüße
Tueftler17

tueftler17

@qqolli @JB_Sullivan
Es gibt eine neue Version, 1.2.1. Die Power-Geschichte sollte damit gelöst sein.
Hoffentlich komme ich demnächst wenigstens ins offizielle latest-repository...

qqolli

@tueftler17

Super! Gleich mal installiert und bis jetzt alles gut :-)

Danke für deine prima Arbeit.

Community Forum

NEWS

[Neuer Adapter] BYD HVS

Support us

383

32.7k

82.3k

1.3m