[Javascript] Midas (Aquatemp) Poolheizung

[oxident]

@michael-roling Konnte doch schon testen und muss leider berichten, dass es bei mir ohne getDeviceID nicht mehr funktioniert.

"TypeError: Cannot read properties of undefined (reading '0')\n at updateDeviceStatus (/opt/iobroker/node_modules/iobroker.midas-aquatemp/build/lib/updateDeviceStatus.js:65:60)\n at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n at async updateToken (/opt/iobroker/node_modules/iobroker.midas-aquatemp/build/lib/token.js:75:7)\n at async MidasAquatemp.onReady (/opt/iobroker/node_modules/iobroker.midas-aquatemp/build/main.js:78:5)"

Wenn ich wieder den Stand vor Deinem letzten Commit (also ab f116b75) nutze, dann ist alles okay.

Das Blöde ist, dass es sein könnte, dass wir bestimmte ProductIDs anders behandeln müssen. Also so, wie auch beim APILevel. Komischerweise schafft die Aquatemp-App es, über alle Geräteklassen hinweg den "richtigen Weg" zu finden.

[sunnylaila]

@oxident

Hallo bei mir funktioniert es jetzt super du musst denke ich nur aus der App deine Geräte Mac eingeben

[sunnylaila]

@michael-roling

Danke dir für deine Mühen echt cool das der Adapter jetzt so läuft

[oxident]

@sunnylaila Cool, danke. Das hatte ich völlig übersehen. Auch die Tatsache, dass die MAC dem DeviceCode entspricht

@michael-roling Genialer Workaround!

Es sei jedoch gesagt, dass MAC/DeviceCode unbedingt in Großbuchstaben und ohne Trennzeichen eingegeben werden muss. Also z. B. 34EAF111AABB

[oxident]

@michael-roling Es scheint so, als würde bei der aktuellen Version die Erkennung der ProductIDs nicht mehr klappen. Will sagen, isAquaTemp_Poolsana() führt bei mir zu false ... und dadurch werden die falschen Parameter ausgewertet.

Ansonsten alles prima.

[Michael Roling]

@oxident schaue ich mir später an und gucke wie ich das anpassen kann. Das ganze war auch nur auf die Schnelle

[sunnylaila]

@michael-roling
er verbindet sich ja jetzt bei mir auch sauber und ließt auch Werte aus siehe Log

midas-aquatemp.0
2024-07-21 17:56:17.892 info DeviceDetails: {"sessionid":null,"error_code":"0","error_msg":"Success","error_msg_code":"","totalSize":null,"totalPage":null,"nextPage":null,"objectResult":[{"value":"1","code":"Power"},{"value":"1","code":"Mode"},{"value":"0","code":"Manual-mute"},{"value":"0","code":"T01"},{"value":"0","code":"T02"},{"value":"0","code":"2074"},{"value":"0","code":"2075"},{"value":"0","code":"2076"},{"value":"0","code":"2077"},{"value":"0","code":"H03"},{"value":"0","code":"Set_Temp"},{"value":"12.0","code":"R08"},{"value":"40.0","code":"R09"},{"value":"12.0","code":"R10"},{"value":"40.0","code":"R11"},{"value":"26.0","code":"R01"},{"value":"25.0","code":"R02"},{"value":"26.0","code":"R03"},{"value":"0","code":"T03"},{"value":"0","code":"1158"},{"value":"0","code":"1159"},{"value":"8","code":"F17"},{"value":"2","code":"H02"},{"value":"0","code":"T04"},{"value":"0","code":"T05"},{"value":"0","code":"T06"},{"value":"0","code":"T07"},{"value":"0","code":"T14"},{"value":"0","code":"T17"},{"value":"17.0","code":"T1"},{"value":"29.0","code":"T2"},{"value":"29.0","code":"T3"},{"value":"31.5","code":"T4"},{"value":"36.0","code":"T5"},{"value":"0","code":"T6"},{"value":"0","code":"T7"}],"isReusltSuc":true}

aber unter Objekte

siehe Bild wird nur set temp gefüllt

[Michael Roling]

@sunnylaila ich schaue mal eben rein.

Okay habe den Fehler gefunden

[Michael Roling]

@sunnylaila @oxident So ich habe jetzt noch mal eine Update erstellt, und zwar muss man jetzt zusätzlich wenn man die DeviceMac nutzen möchte eine Checkbox dahinter anhaken. Ansonsten würde der ganz normale Code wie vorher auch ausgeführt. Zudem habe ich das Problem fürs erste behoben das die Daten nicht aktualisiert werden bei @sunnylaila bzw dadurch sollte der Fehler von @oxident auch behoben sein

Ich war zur voreilig, einen Moment noch

Scheint doch zu funktionieren, bitte Testen

[sunnylaila]

@michael-roling

die Objekte werden bei mir leider noch nicht gefüllt

[Michael Roling]

@sunnylaila weil aktuell nichts an Daten rein kommt. Bzw. welche Daten meinst du ?

[sunnylaila]

@michael-roling

zb die Daten von temp in und temp out
laut log würde ich denken das es t2 und t3 sind die aber nicht in Objekte übernommen wird

2024-07-21 19:15:34.799	info	DeviceDetails: {"sessionid":null,"error_code":"0","error_msg":"Success","error_msg_code":"","totalSize":null,"totalPage":null,"nextPage":null,"objectResult":[{"value":"1","code":"Power"},{"value":"1","code":"Mode"},{"value":"0","code":"Manual-mute"},{"value":"0","code":"T01"},{"value":"0","code":"T02"},{"value":"0","code":"2074"},{"value":"0","code":"2075"},{"value":"0","code":"2076"},{"value":"0","code":"2077"},{"value":"0","code":"H03"},{"value":"0","code":"Set_Temp"},{"value":"12.0","code":"R08"},{"value":"40.0","code":"R09"},{"value":"12.0","code":"R10"},{"value":"40.0","code":"R11"},{"value":"26.0","code":"R01"},{"value":"25.0","code":"R02"},{"value":"26.0","code":"R03"},{"value":"0","code":"T03"},{"value":"0","code":"1158"},{"value":"0","code":"1159"},{"value":"8","code":"F17"},{"value":"2","code":"H02"},{"value":"0","code":"T04"},{"value":"0","code":"T05"},{"value":"0","code":"T06"},{"value":"0","code":"T07"},{"value":"0","code":"T14"},{"value":"0","code":"T17"},{"value":"17.0","code":"T1"},{"value":"29.0","code":"T2"},{"value":"29.0","code":"T3"},{"value":"28.5","code":"T4"},{"value":"30.0","code":"T5"},{"value":"0","code":"T6"},{"value":"0","code":"T7"}],"isReusltSuc":true}

[Michael Roling]

@sunnylaila Versuch jetzt noch mal, ich habe in meiner Heizung geschaut gehabt und habe deswegen den falschen Wert eingetragen.

[sunnylaila]

@michael-roling

ja genau das war es, jetzt geht es und trägt die Werte auch in Objekte ein
vielen vielen Dank für deine Mühen.
Brauchst du den Test Account noch?

Gruß

[Michael Roling]

@sunnylaila grundsätzlich brauche ich den nicht mehr. Aber ich würde vorschlagen du testest morgen noch mal in Ruhe , auch mit werte setzen usw. Wenn alles geht kannst du den löschen. Du kannst den natürlich auch sofort löschen und gegebenenfalls neu erstellen wenn benötigt

[sunnylaila]

@michael-roling
ok machen wir so, werde es noch ein paar tage lassen falls du etwas testen möchtest.

[Michael Roling]

@oxident Moin. Hast du eine eine Liste mit Geräten die mit dem Adapter funktionieren? Oder Links zum Hersteller , oder so?

[mcm1957]

Der ADAPTER für midas-aquatemp wurde vom Dev aus den Repositoriies zurückgezogen u wird - wenn ich die Information nicht falsch verstanden habe - NICHT weiterentwickelt.

Ref https://github.com/ioBroker/ioBroker.repositories/pull/3890

MiRo1310 created this issue in ioBroker/ioBroker.repositories

closed Add midas-aquatemp to latest #3890

[Michael Roling]

@mcm1957 Nicht ganz, erstmal läuft alles ja. Und es ist nicht zwingend notwendig das dieser aktuell ins Latest kommt. Issues würde ich natürlich bearbeiten.

Ich werde dieses natürlich noch nachholen, aber im Moment schaffe ich das nicht. Aber der Winter kommt ja bald.

[mcm1957]

Von der direkten Installation von Github wird jedenfalls abgeraten

Der Vollständigkeit und als Info - NICHT als Kritik

Von direkten Installation von GitHub - insbesondere auf produktiven Systemen - wird explizit abgeraten. GitHub Versionen können sich jederzeit (auch kurzfristig) ändern und durchaus auch in sich inkonsistent und fehlerhaft sein. Versionsangaben von GitHub Installationen sind Schall und Rauch da die Versionsnummer zumindest bei Verwendung der standardmäßigen Umgebung erst im Zuge der Releaseerstellung geändert wird.

Auf explizite Aufforderung durch den Entwickler kann eine GitHub Installation zur Fehlereingrenzung oder zum Test neuer Funktionalität - unter Inkaufnahme des erhöhten Risikos - natürlich erfolgen.

Adapter die nur via GitHub oder npm (also NICHT aus einem der beiden Repositories) installierbar sind sind mit erhöhter Vorsicht zu betrachten. Hier sollte der Entwickler drum ersucht werden eine Aufnahme in die Repositories zu veranlassen indem z.B. ein Issue im Adapterrepository erstellt wird.

Und falls es irgendwie unklar ist:

ioBroker unterstützt folgende Arten von Installation:

• aus dem STABLE Repository

  Das sind Adapter Releases die keine groben Fehler aufweisen (sollten). Natürlich kann es auch dort Fehler geben, die Behebung davon obliegt dem jeweiligen Dev und kann ggF auch dauern.

• aus dem LATEST Repository

  Das sind Adapter Releases die neu erstellt wurden und nur rudimentär getestet sind - oft auch als BETA Releases bezeichnet. Releases aus dem LATEST sind primär für unsere zahlreichen freiwilligen Tester gedacht. BETA / LATEST Releases können durchaus Fehler aufweisen - auch wenn ich davon ausgehe dass jeder aintainer dies zu vermeiden versucht. Vom Einsatz auf produktiven Systemen wird abgeraten außer man braucht irgenein neues Feature (z.B. neues Gerät) unbedingt. Hier muss dann jeder Entscheiden was ihm wichtig ist.

• direkt von GITHUB

  Von Installationen direkt aus Giuthub wird definitiv abgeraten außer auf Anweisung des Maintainers und für den Fall dass man gemeinsam mit diesem etwas testen will. Details siehe oben.

• direkt von npm

  Diese Installation kann erforderlich sein, wenn man eine bestimmte Version installieren möchte / muss. Im Normalfall sollte man direkte npm Installationen ebenso meiden wie direkte GitHub Installationen - ausgenommen um z.B. zu einer bekannten Version downzugraden - obwohl hier wenigstens ein definierter Stand garantiert ist.

Adapter die NUR via npm und/oder nur via GitHub installierbar sind sollte man meiden - diese wurden nicht mal einem rudimentären Review unterzogen und sollten mit dem Attribut "vollständig auf eigenes Risiko verwenden" installiert werden. Hier empfiehlt es sich den Developer zu ersuchen eine Aufnahme in die Repos zu veranlassen. Wenn dieser darauf nicht reagiert sollte man von einer eher kurzen und unklaren Lebensdauer des Adapters ausgehen. Support meiner-/unsererseits für solche Adapter ist mit sicherheit minimalistisch.