UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[mcm1957]

@crunchip
Dein Fork wechslet das Kommunikationsprotokolla, verwendet nun (auch ?) RPC, greift in die adminUIConfig ein usw. Das hat eigentlich nichts mehr mit einem Fix für MQTT Prefix zu tun.
Sorry.

Ich hab in absehbarere Zeit sicher keine Zeit diese Menge an Änderungen zu reviewen sodass man das auf 10000+ User loslassen kann. Bitte kontaktiere @haus-automatisierung was er dazu sagen kann bzw. ob er sich das ansehen will / kann.

Wenn du wirklich helfen willst, dann schau mal ob das Anpassen des mqtt acks bei dir hilft - oder nicht bzw. ob das Nebeneffekte auslöst.

mcm1957

P.S. Copilot ist an sich gut - ich verwende ihn auch intensiv. 90% der Änderungen sind OK. Er hat mir aber auch schon völlig frei erfundene Dinge eingebaut.

[crunchip]

@mcm1957 sagte in Test Adapter shelly - ALPHA Versionen:

90% der Änderungen sind OK. Er hat mir aber auch schon völlig frei erfundene Dinge eingebaut.

ich weiss, habe ich auch damit Erfahrung gemacht.

@mcm1957 sagte in Test Adapter shelly - ALPHA Versionen:

hat eigentlich nichts mehr mit einem Fix für MQTT Prefix zu tun

sehe ich zwar anders, deshalb ja die Änderungen

aber unterm Strich, wie oben schon

@crunchip sagte in Test Adapter shelly - ALPHA Versionen:

eventuell doch mal Lust ein Auge draufzuwerfen, ob was sinnvolles dabei rumgekommen ist, bzw den ein oder anderen Ansatz aus dem Fork in Betracht zu ziehen.
Vielleicht war das ganze auch ein "Griff ins Klo",

nicht mehr und nicht weniger

@mcm1957 sagte in Test Adapter shelly - ALPHA Versionen:

Wenn du wirklich helfen willst

habs verstanden und daher belassen wir es damit, kein Problem und ja, werd ich testen und Rückmeldung geben

[mcm1957]

@crunchip

Ich glaub ich sollte mein Feedback anders formulieren.

Zuerst sollte ich sagen "DANKE dass du mithilfst ioBroker stabiler zu machen". Sowohl als Tester als auch mit Codeänderungsvorschlägen. Das hätte ich unbedingt als erstes loswerden sollen da es am Wichtigsten ist.

Im Prinzip hat mich dein Änderungsvorschlag vom Umfang her überfahren. Und ich hab daher eher unangemessen reagiert. Ich durchschaue den Shelly Code bei weitem nicht komplett. Und du kannst selbst - wie du schreibst die Änderungen codemäßig nicht kommentieren. Es ist mir daher unmöglich die umfangreichen Änderungen die der Copilot da vorgenommen hat mit mäßigem Zeitaufwand auch nur grob zu beurteilen. Und was das konkrete mqtt Problem betrifft kann ich auf Grund des Änderungsumfangs das nicht halbwegs sicher extrahieren.

Den Adapter hat - soweit ich weiß - zu großen Teilen @haus-automatisierung entwickelt. Ich muss daher betreffend deine Änderungsvorschläge auf ihn verweisen. Insbesondere die Verwendung neuer Kommunikationsschnittstellen zu den Shellies muss man evaluieren.

Bitte lösch keinesfalls deinen Fork damit Mathias eine Chance hat sich das anzusehen.

Und nochmals DANKE für deine Mithilfe.

[crunchip]

@mcm1957 alles gut,
ich beabsichtige nicht einen PR zu erstellen und kann auch anderweitig nicht viel dazu beitragen. Bin ja auch nicht vom Fach. Darum geht's auch gar nicht.

Hatte lediglich meine Gedanken in die KI getippt zum Fehlerbild

Error in function setMqttPrefixHttp (Gen 1)

Was ja kein Gen1 Gerät ist.

Daraus entstand dann "Mein Projekt" und hab aus Spaß nen Fork erstellt. Das Resultat daraus war eben dieser Umbau.

Letztendlich hab ichs hier gepostet mit den Gedanken

• wenn Lust und Laune besteht, man ja einen Blick drauf werfen kann, also kein Zwang
• als Fachmann den Code weitgehenst versteht
• gerade als eingefleischter Spezialist manchmal "Betriebsblind" ist und ein Aussenstehender ne andere Sicht und Denkweise hat,
  aber auch weiss, das ein kompletter Umbau sehr aufwendig wäre und es die Zeit gar nicht zu lässt.
  Oder anders Formuliert... Kosten/Nutzenfaktor... Warum sollte man..., läuft doch so
  und Matthias auch nicht beabsichtigt den Adapter umzumodeln.
• eventuelle interessante codeschnipsel dabei sein könnten, die zur Verbesserung bei tragen

Ps: Texte fasst man meist falsch auf oder kommt falsch rüber

[mcm1957]

@crunchip
Und ja - ich werde deinen Fork sicher auch noch mal ansehen. Gute Ideen sind ja sicher sinnvoll. Bin nur zur Zeit irgendwie im Grenzlastbetrieb (selbstgemacht ...).

[mcm1957]

Es gibt eine neue ALPHA 10.4.2-alpha.0

Da die Labor / Github Release keine Rückmeldungen über massive Probleme gezeigt habe, stelle ich diese Anpassungen nun allgemein für einen Alpha Test bereit:

Diese Release enthält eine Änderung betreffend MQTT Connection. Die Änderung sollte helfen das Problem 'Unable to get MQTT Prefix' zu beheben. Bitte um Rückmeldung falls ihr diesbezügliche Änderungen bemerkt und insbesondere auch wenn andere neue Nebenwirkungen beobachtet werden,

[mcm1957]

ACHTUNG:
Leider gibt es mit der 10.4.2-alpha.0 offensichtlich Probleme bei batteriebetriebenen Geräte:
https://github.com/iobroker-community-adapters/ioBroker.shelly/issues/1289

Danke HGLab für die Meldung incl. ausführlichem Log

[mcm1957]

@mcm1957 said in Test Adapter shelly - ALPHA Versionen:

ACHTUNG:
Leider gibt es mit der 10.4.2-alpha.0 offensichtlich Probleme bei batteriebetriebenen Geräte:
https://github.com/iobroker-community-adapters/ioBroker.shelly/issues/1289

Bitte neue Version 10.4.2-alpha.1 testen.

Zumindest mein Shelly Button 1 funktioniert mit dieser Version wieder. Allerdings habe ich sonst keine weiteren MQTT Geräte GEN1 mit Batteriebetrieb verfügbar.