Test Adapter time-switch v2.0.x GitHub/Latest
-
Hallo zusammen,
wie wichtig wäre es denn diesen Adapter für VIS-2 läuffähig zu bekommen?
Gruß//Lucky
-
@lucky_esa ich würde mich freuen wenn ich den Adapter mit VIS2 nutzen könnte.
Da ich time-switch benutze kann ich deshalb noch nicht auf VIS2 umsteigen. -
@lucky_esa für mich sehr ... ich mache sämtliche Zeitsteuerungen über den Adapter
Wie bei @mane444 ist das einer Gründe (von 2en) das ich die VIS2 bisher links liegen gelassen habe. -
@lucky_esa Bei mir das gleiche Nutze VIS 2 aus 2 Gründen nicht, weil meine Navigation, und der Time-Switch-Adapter nicht geht
-
Ich habe leider keine Ahnung von VIS und seine widgets aber warum es nicht funktioniert kann ich sehen. Es gibt leider 2 Themen warum es nicht funktioniert.
- Das hat was mit diesem issue zu tun. Da hätte ich einen Workaround.
- Der 2te ist leider etwas schlimmer. Das Thema haben schon einige berichtet, dass man nicht so einfach an die Values eines States kommt. Hier hätte ich einen Workaround der leider etwas umständlich wäre. Man müsste 4 Datenpunkte 2 x auswählen.
-
Die Diskussion zur Weiterentwicklung bzw. zum Adapter TIMER-SWITCH (Lucky_ESA) wurde in einen eignen Thread verlagert:
https://forum.iobroker.net/topic/77355/test-adapter-timer-switch
Bitte hier NUR Beiträge zum älteren Adapter time-switch (ohne "r") verfassen.
-
@mcm1957 Kannst du mir bitte mal einige Fragen beantworten?
- Wenn ich einen PR erstelle, wirst du diesen Mergen (es werden alle Daten geändert)?
- Wenn ja, kannst und wirst du dann ein neues Release erstellen?
- Wenn ja, kannst du dann auch eine neues Release auf NPM ablegen?
- Wenn ja, kannst du dann einen PR für stable erstellen. Würde dann natürlich eure Änderungswünsche umsetzen.
Gruß//Bodo
-
@lucky_esa sagte in Test Adapter time-switch v2.0.x GitHub/Latest:
Wenn ich einen PR erstelle, wirst du diesen Mergen (es werden alle Daten geändert)?
das kann ich dir beantworten mit klaren NEIN
der Martin ist kein eingetragener owner von dem Repo...und der walli545 war schon lange nicht mehr aktiv.. ergo .. der Adapter ist tod. zumindest aus diesem Repo...
-
@lucky_esa said in Test Adapter time-switch v2.0.x GitHub/Latest:
@mcm1957 Kannst du mir bitte mal einige Fragen beantworten?
- Wenn ich einen PR erstelle, wirst du diesen Mergen (es werden alle Daten geändert)?
- Wenn ja, kannst und wirst du dann ein neues Release erstellen?
- Wenn ja, kannst du dann auch eine neues Release auf NPM ablegen?
- Wenn ja, kannst du dann einen PR für stable erstellen. Würde dann natürlich eure Änderungswünsche umsetzen.
Gruß//Bodo
Wie Artek schon geschrieben hat:
NEIN ich kann das (so) nicht und ziemlich sicher kann es auch sonst niemand.
Damit stell ich mal ein paar Gegenfragen
- Wurden deine Änderungen / Erweiterungen in irgendeiner Form mit dem Owner des Repositories (wally545) abgestimmt?
- Wurde in irgendeienr nachvollziehbaren Form versucht wally545 zu kontaktieren (Issue, Mail, ...)?
Generell kann ein Adapter in den Community Bereich übernommen werden. Dazu ist aber entweder die Zustimmung des derzeitigen Owners notwendig oder es muss zumindest angemessen versucht werden diesen zu erreichen, d.h alle bekannten Kommunikationswege und mindestens 1 bis 2 Monate Reaktionszeit. Du würdest es sicher auch nicht wollen, dass einer deienr Adapter einfach mal so verschoben wird nur weil du krank / auf Urlaub bist.
Weiters ist für die Übernahme in den community Bereich zumindest eine rudimentäre Wartbarkeit / ein zumindest interessierter Maintainer erforderlich, da keiner den Community Adapters Bereich als Ablagerort für Zombies sieht. Dieser Punkt wäre ja gegeben und die allenfalls notwendige Timeoutanfrage bei Wally545 würde ich vornehmen wenn ...
Und damit zur konkreten Situation:
Sollte wally545 zustimmen oder nicht erreicht werden, stellt sich die Frage was du geändert hast.An sich hast du einen NEUEN Adapter erstellt. Neues Repo, neuer Name -> neuer Adapter. Wenn du nun diesen unter dem alten Namen veröffentlchen willst, wäre interessant was sich geändert hat.
-) wird eine existierende Konfiguration übernommen?
-) bleiben existierende States zumindest Großteils ident erhalten?
-) was verstehst du unter "es werden alle Daten geändert" ?Hintergrund der Frage ist, ob es sinnvoll ist den alten Adapter zu übernehmen und (ggF) inkompatibel massiv zu ändern oder gleich einen neuen Adapter zu veröffentlichen und den alten Adapter asulaufen zu lassen wenn User sowieso Einstellungen / Vis etc. anpassen müssen.
Für diese Entscheidung fehlen mir derzeit Infos wobei ich gleich sage, dass ich diese nicht alleine treffen werden.
@apollon77 FYI
-
@mcm1957 Leider mache ich immer wieder den gleichen Fehler und konfrontiere, User wie dich, ohne Referenz, sorry...Ich gelobe Besserung.
History:
Bin von diesen Adapter abhängig und da habe ich diesen Issue gesehen.
Hatte dann einen Fork erstellt und schnell festgestellt, dass das schwierig werden wird. Um das alles zu testen habe ich temporär (alles hier im Thread beschrieben) den timer-switch (mit r) angelegt. Wenn alles funktioniert wollte ich ein PR erstellen und den timer-switch löschen (alles hier bereits beschrieben).Du wurdest, laut Mail, von walli546 eingeladen und hast Rechte für NPM. Wenn du diese Einladung akzeptierst, könntest du einen PR Mergen und ein neues Release erstellen.
Es würde theoretisch ausreichen, eine PR zu mergen und kein Release zu erstellen. Oder den Adapter aus dem Beta Repo werfen.
Natürlich werde ich kein neuen Adapter erstellen oder eine Anfrage bezügl. wechsel in den Community Bereich stellen. Du hast ihn ja angeschrieben und wir haben sein Statement.
Nun zurück zu meinen Fragen:
- Wenn ich einen PR erstelle, wirst du diesen Mergen (es werden alle Daten geändert)?
- Wenn ja, kannst und wirst du dann ein neues Release erstellen?
- Wenn ja, kannst du dann auch eine neues Release auf NPM ablegen?
- Wenn ja, kannst du dann einen PR für stable erstellen. Würde dann natürlich eure Änderungswünsche umsetzen.
Wenn du das nicht möchtest, dann lass ich den Fork bestehen.
Gruß//Lucky
-
Danke dass du mich erinnerst. Hab mich nicht mehr an diese Diskussion erinnert.
wally545 schreibt explizit (https://github.com/walli545/ioBroker.time-switch/issues/163#issuecomment-2079179952)
Die ownership des Repos werde ich aber nicht übertragen. Wenn das für euch nicht passt, könnt ihr es gerne per Fork regeln.
Aus meiner Ansicht macht es keinen Sinn einen Adapter auf diese Art und weise zu warten. Es ist voll OK wenn mehrere Personen an einem Adapter arbeiten - aber eben auch(!) der Owner. Wenn dieser nicht kann / will hat es keinen Sinn "nur" einen Maintainer zu registrieren. Der kann wesentliche Aufgaben mittelfristig nicht wahrnehmen - so kann er z.B. weder das Repo übertragen noch kann er Webhooks eintragen (-> weblate) oder Security Token anlegen / eintragen (-> zB. npm Anbindung). Das Ganze ist und bleibt dann eine halbe Sache die dann im ungünstigsten Augenblick klemmt...
Man könnte nun den Adapter per Override / Fork verlegen - gegen den expliziten Willen das Owners fände ich das aber nicht so super.
ERGO:
Lass doch deinen Adapter (fork unter neuem Namen) neu in die Repositories eintragen. Was spricht da dagegen? Von Adaptern die nur via Github installiert werden können wird explizit abgeraten. Damit stünde deine Arbeit nur experimentierfreudigen Usersn zur Verfügung...Alternativ schreib ihn selbst an, ob er dir Zugriffsrechte gibt?
@apollon77
Wie siehst du das? -
Nach kurzer Rücksprache wäre folgende Vorgangsweise am sinnvollsten:
Bitte lass den Adapter unter einem neuen Namen in die Repositories eintragen.
Es gibt dann halt einen time-switch und einen timer-switch. GgF kannst du den Namen auch noch etwas unterschiedlicher gestalten - ist aber dir überlassen.
Dann einfach einen PR für LATES Aufnahme erstellen. Damit sind die beiden Adapter entkppelt und du kannst tun / ändern was du magst bzw. dir sinnvoll erscheint. Da du schreibst "es werden alle Daten geändert" macht es für User sowieso keinen Unetrschied ob sie updaten oder neu installieren - neu installieren bitet sogar den kleinen Vorteil dass gezielter migriert werden kann.
Diskussion zum neuen Adapter bitte dann im entsprechenden Topic:
https://forum.iobroker.net/topic/77355/test-adapter-timer-switch
-
@mcm1957 Werde dann den Adapter als schedule-switcher neu in Repo eintragen. Da der derzeitige eine Prüfung nicht standhalten würde, muss ich jetzt erstmal was ändern.
Könnt ihr bitte den Adapter aus dem Beta Repo entfernen. Dieser nutzt Funktionen die auf "deprecated" gesetzt sind. Werden diese Funktionen entfernt wird der Adapter beim Restart/Neustart crashen.deleteChannelAsync createDeviceAsync createChannelAsync createStateAsyncP.S. Das wäre der PR gewesen.
Gruß//Lucky
-
@lucky_esa
Ja danke,
Machen wir wenn bis js-controller 8 (?) diese Funktionern noch nicht ersetzt sind.
Bis dahin hat der dev Zeit das zu fixen. -
Hier nun eine Version für VIS-2
Sollte auch mit VIS laufen. Wie bereit gesagt, wird der alte Adapter nicht mehr lange funktionieren.
Gruß//Lucky
