Supportthread Resol-Adapter

[Gargano]

@faz ok, dann brauchen wir die Versions Info unbedingt. Muss ich mal schauen, wie ich das dann in dem Adapter allgemein gültig unterbringe. Die jetzige Lösung funktioniert, ist aber ein Hack und würde mit dem nächsten Update wieder weg sein.
Vielen Dank fürs Testen.

Schalten kannst Du auch unter Actions?

[Faz]

@gargano Ich habe zu Danken, da werde ich mal mir die Daten die wir geändert haben vorerst sichern

[Gargano]

@gargano Schalten kannst Du auch unter Actions?

[Faz]

@gargano ja Funktioniert

[Gargano]

@grizzelbee Hi grizzelbee,
das war ein etwas fummeliger Act mit dem MX-Controller, da der ja 2 Dateien hat aber mit der gleichen Adresse.
Dazu muss man dann im createOptimizer die Version mit angeben.

Leider gibt der MX Controller keine Info über die Version aus. Eine Lösung wäre das in der Konfiguration wählbar zu machen.

Noch ein Punkt:
Im Main ist der Pfad für den VBus so beschrieben

const vbus = require('resol-vbus');

Das hat zur Folge das die VBus LIb im Pfad vom Resol sich befinden muss.

/opt/iobroker/node_modules/iobroker.resol/node_modules/resol-vbus

Bei einem normalen Install vom Vbus befindet sich der VBus aber hier :

/opt/iobroker/node_modules/resol-vbus

Was zur Folge hat, daß der require Pfad so aussehen muss

const vbus = require('../resol-vbus');

Ein Update vom Vbus wird auch in das Verzeichnis gemacht

/opt/iobroker/node_modules/resol-vbus

Viele Grüße

[Grizzelbee]

@gargano

Freut mich riesig, dass ihr das klären konntet.
War schon interessant so als Zuschauer am Spielfeldrand zu stehen.

@Gargano schrieb:

Ich habe ein Issue bei Daniel aufgemacht, daß mit den 2 verschiedenen Adressen (32273 und 32289) ist mir unbekannt.

das war ein etwas fummeliger Act mit dem MX-Controller, da der ja 2 Dateien hat aber mit der gleichen Adresse.
Dazu muss man dann im createOptimizer die Version mit angeben.
Leider gibt der MX Controller keine Info über die Version aus. Eine Lösung wäre das in der Konfiguration wählbar zu machen.

Hmmm. In der Konfig ginge das natürlich - fände ich aber eher unelegant, weil es ja keine generelle Konfig wäre. Kontextsensitive Einstellungen, die nur bei Auswahl einer bestimmten anderen Einstellung angezeigt werden, habe ich im Broker noch nicht gesehen und auch keine Idee, wie man das realisieren könnte.
EDIT: Der FullyBrowser-Adapter macht genau das. Da könnte ich also zur Not spicken wie das geht.
So als Arbeitshypothese (keine Ahnung ob da etwas dran ist) - ist vielleicht die 32273 die V2 und 32289 die V1??
Dann könnte man das doch unterscheiden. Zumindest, wenn ich das Ganze Thema beim Mitlesen richtig verstanden habe.

Bei einem normalen Install vom Vbus befindet sich der VBus aber hier :

Du meinst vom myVBus Adapter?
Wenn ja - würde ich tendenziell bei der Variante:

const vbus = require('resol-vbus');

bleiben wollen, weil die Lib ja ausschließlich vom Resol Adapter verwendet wird, und für meinen Geschmack deshalb genau dorthin (/opt/iobroker/node_modules/iobroker.resol/node_modules/resol-vbus) gehört.
Zum einen um beim Löschen des Adapters auch richtig erwischt zu werden und zum anderen um sich nicht mit anderen Adaptern (hier speziell myVBus), die die selbe Lib verwenden mit der Version der Lib nicht in die Quere zu kommen. Wäre ja blöd, wenn ein Adapter eine spezielle Version einer Lib benötigt und ein anderer Adapter die plump überschreibt und damit den Adapter kaputt macht. Bei einer seltenen lib wie VBus mag das noch kein (großes) Problem sein, aber stell dir das Problem mal zum Beispiel bei Axios vor - da zerschießt ein allgemeines Update ggf. einen Großteil der Adapter.
Wenn es eine neue Version der Lib gibt, die der Adapter benötigt, wird die ja mit dem nächsten Release ausgeliefert/installiert und alles ist gut.

Wie machen wir weiter? Reichst Du einen PR mit den erarbeiteten Änderungen ein? Dann würde ich noch die Abhängigkeiten aktualisieren und ein neues Release bauen - also nachdem wir diesen V1/V2 Problem irgendwie gelöst und ggf. die Konfig angepasst haben - falls nötig.

viele Grüße
grizzelbee

[Gargano]

@grizzelbee
Da ja V1 und V2 die gleichen Adressen haben (32273) kann ich nicht auf die 32289 gehen.
Ich kontaktiere nochmal Daniel, um zu klären ob man die deviceMajorVersion aus dem MX Controller auslesen kann.
Das wäre am besten.
Falls das nicht geht, bleibt nur eine User Auswahl. Die müsste dann entweder alle Einträge vom Resol-Types anzeigen, oder nach erkennen der Adresse nur die anzeigen, die die Adresse haben. In dem Falle hieße es das die Auswahl erst nach dem Starten der Instanz angezeigt wird. Was aber z.B,. Device Watcher auch macht.
Im Fall vom MX würden dann 2 Einträge da stehen : V1 und V2, bei allen anderen nur ein Eintrag .

Ich schick Dir dann ein PR , wenn ich vom Daniel Nachricht habe.

[Grizzelbee]

@gargano

Okay. So genau habe ich nicht mitgelesen. Für mich sah das so aus als würden sich die V1 mit der Adresse 32273 und die V2 mit der Adresse 32289 beim Adapter melden.
Aus dieser Info hätte man dann die passende deviceMajorVersion ableiten und im Code setzen können.
Gibt es vielleicht irgendeine andere Info, die man auf diese Weise ausbeuten könnte? Die DeviceID vielleicht (deviceId":"007E110010" bei V2), Anzahl irgendwelcher Sensoren, Vorhandensein/Abwesenheit eines Sensors, ...?
Keine Ahnung. Ich stocher nur im Nebel, weil ich solche Konfigeinstellungen nicht sonderlich mag, weil die supportanfällig sind. Wer achtet schon auf die Hardwarerevision von irgendetwas und ahnt das das einen Unterschied an irgendeiner Stelle machen könnte? Ich sicher nicht. wenn der selbe Name drauf steht, erwarte ich, das alle Versionen identisch funktionieren.

Aber am Ende: Wenn es nichts gibt, gibt es nichts und dann muss es leider die Konfig retten.

[Gargano]

@grizzelbee sagte in Supportthread Resol-Adapter:

Für mich sah das so aus als würden sich die V1 mit der Adresse 32273 und die V2 mit der Adresse 32289 beim Adapter melden

Das wäre ja auch zu einfach

Das habe ich jetzt gesehen auf Git vom Daniel:

Known issues
The ConfigurationOptimizers do not yet detect the firmware version running on the controller to be configured. That sometimes causes configuration loads and saves to fail because unknown values are read from or written to (e.g. using the "customizer" example on a DeltaSol MX with firmware version 1.11 or below).

Short-term plans
Remove current ConfigurationOptimizer constructs in favor of RESOL's official support.

Also wenn der sowieso weg fällt und es bis jetzt nur den User faz gibt, der den MX Controller nutzt, denke ich sollte wir warten bis Daniel den Short-term plans umgesetzt hat.

[Gargano]

@grizzelbee sagte in Supportthread Resol-Adapter:

ber am Ende: Wenn es nichts gibt, gibt es nichts und dann muss es leider die Konfig retten.

Ich habe Antwort vom Daniel : Auslesen der Majorversion ist nicht möglich. Also bleibt nur die Auswahl durch den User.
Ich mach dann mal die Änderungen und schick Dir eine PR. Du müsstest dann die Auswahl einbauen.
Kannst ja schon mal schauen wie das geht und mir dann mitteilen, welche Infos in welchem Format Du brauchst. (Am Besten JSON ?)
Machen wir einen neuen Branch zum Testen ?

Viele Grüße
Werner

[Grizzelbee]

@gargano
Ich schlage vor, das ich zuerst die Konfig baue - dann kannst Du im Code darauf reagieren. Das dürfte am einfachsten sein. Das werde ich aber frühestens kommende Woche schaffen.

Ich melde mich, wenn ich soweit bin und einen Branch dafür habe.

[Grizzelbee]

@gargano

Hallo Werner,

ich finde gerade ein bisschen Zeit und Ruhe um mich um meine Projekte zu kümmern.
Bleibt es dabei, das wir das zusammen mit der Konfig-Option einbauen, oder warten wir auf das Update von Daniel (Was ziemlich lange dauern könnte, wenn ich mir seine Release-Zyklen so anschaue )?

[Gargano]

@grizzelbee
Daniel hat ja gesagt, daß Major Version auslesen auch in Zukunft nicht gehen wird.
Also machen wir das mit der User Auswahl.
Ich benötige dann eine Funktion, in der ich die Auswahl reinschreiben kann (Json) und eine Funktion, die getriggert wird, wenn der User ausgewählt hat mit Rückgabewert der Auswahl oder den Index.

Viele Grüße
Werner

[Grizzelbee]

@gargano

Okay. Dann machen wir das so.

Die neue Konfig sähe dann so aus:

Auf die beiden neuen Werte kann dann über das normale Adapter-Konfig Objekt zugegriffen werden.

Passt das für dich?

[Gargano]

@grizzelbee Schaut schon gut aus. Machst Du ein Branch ?

[Grizzelbee]

@gargano

Schaut schon gut aus. Machst Du ein Branch ?

Das freut mich - und: Ja - ich mache da in Kürze einen Branch für. Bin (hoffentlich auch in Kürze soweit).
Habe da aber noch eine Frage: Kann ich in den Tooltipp der ControllerMajorVersion irgendwas hilfreiches reinschreiben wie man die ermittelt bzw. wo oder wie man die aus-/ablesen kann?

[Gargano]

@grizzelbee
In dem File Setup-Resol-Types.js stehen ja die Typen drin mit ID und dann zukünftig auch die Major Version.

Z.B. für den MX :

{"id":32273,"setup":"deltasol-mx",majorVersion":1,"data":"resol-deltasol-mx-112-data"},
{"id":32273,"setup":"deltasol-mx2xx","majorVersion":2,"data":"resol-deltasol-mx-2xx-data"},

Identifiziert wird es ja beim Start über die ID. In dem Falle 32273.
Kannst Du eine Identifizierungs - Button anlegen, womit der Controller automatisch gesucht und identifiziert wird ?
Der Rückgabe Wert muß dann die ID sein.

Der Ablauf könnte dann so aussehen :
Controller identifizieren
Identifizierung über die ID ,
Wenn eine Major Version eingetragen ist, dann Auswahl der Version durch den User.
Der Controller Type würde automatisch dann richtig identifiziert.
In dem Falle brauchen wir die Auswahl MX/Other nicht.
Erst dann die Objekte anlegen , da die evtl. sonst falsch sind.

oder
Auswahl vom Controller Type und Version beim Installieren durch den User.
(Ich glaube , das hast Du so vorgesehen, oder ?)

Ich bin für die erste Variante, weil bei Erweiterungen der Controller Versionen nur das Setup Type File angepasst werden muss.

[Grizzelbee]

@gargano

Auswahl vom Controller Type und Version beim Installieren durch den User.
(Ich glaube , das hast Du so vorgesehen, oder ?)

Ja, das war meine erste Idee, weil mir nicht klar war, das man die Daten automatisch ermitteln kann.
Deine Lösung klingt besser.

Kannst Du eine Identifizierungs - Button anlegen, womit der Controller automatisch gesucht und identifiziert wird ?

Ja - das müsste gehen. Gucke ich mir dann die Tage mal an.

[Grizzelbee]

@gargano

So - was lange währt wird endlich was (ob auch gut, wir sich zeigen) .
Schau dir bitte mal im Repo den development Branch an. Da habe ich das mal vorbereitet - in der Hofgfnung das Du darauf aufsetzen kannst.

viele Grüße
grizzelbee

[Gargano]

@grizzelbee
Kannst Du ein paar Zeilen zu den Änderungen sagen ?
Ich habe gesehen : Message Funktion ist hinzugekommen und ein 'getControllersForAdminSelect'.

Viele Grüße
Werner