Ankündigung

Einklappen
Keine Ankündigung bisher.

Neues Plugin für KWL Pluggit AP310

Einklappen
X
 
  • Filter
  • Zeit
  • Anzeigen
Alles löschen
neue Beiträge

  • Cannon
    antwortet
    Zitat von Msinn Beitrag anzeigen
    Ist besser. Du siehst in mehreren Plugins eine Liste von Copyright Hinweisen
    Dann mache ich das und lade dann ein Update hoch.

    Einen Kommentar schreiben:


  • Msinn
    antwortet
    Ist besser. Du siehst in mehreren Plugins eine Liste von Copyright Hinweisen, weil nach und nach verschiedene Autoren das Plugin weiter entwickelt haben.

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Zitat von Msinn Beitrag anzeigen
    Inzwischen sehe ich den PR.
    Großartig. Dann habe ich doch mal was richtig gemacht da.

    Zitat von Msinn Beitrag anzeigen
    Du hast den ursprünglichen Copyright Hinweis entfernt. Ist das Plugin ein vollständiges Rewrite, ohne Code aus der alten Version zu verwenden?
    Da sind sicherlich noch kleine Reste übrig. Irgendwie ging das Ganze zwar daraus vor, aber die Basisstruktur ist ja in jedem "Musterplugin" drin. Ich kann das aber wieder rein machen, wenn das richtig ist und so sein soll.

    Einen Kommentar schreiben:


  • Msinn
    antwortet
    Inzwischen sehe ich den PR.

    Dazu habe ich einen Punkt, der mir sofort auffiel: Du hast den ursprünglichen Copyright Hinweis entfernt. Ist das Plugin ein vollständiges Rewrite, ohne Code aus der alten Version zu verwenden?

    Einen Kommentar schreiben:


  • Msinn
    antwortet
    Ich sehe allerdings keine neuen Pull Requests (weder gegen master, noch gegen develop). Ich sehe nur den 2 Tage alten PR #615.

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Ich habe das jetzt noch mal probiert. Leider beim ersten Mal wieder im Master, aber irgendwie kann ich das nicht rückgängig machen. ich hoffe es ist jetzt im Develop drin ... irgendwie stelle ich mich da doof an.

    Einen Kommentar schreiben:


  • Msinn
    antwortet
    Das grundlegende Problem liegt darin, dass Du den PR gegen den master Branch gestellt hast und nicht gegen den den develop Branch. Der master Branch enthält die jeweiligen Releases und keine Entwicklungsstände/Updates. Die gehören in den develop Branch und werden nur bei einem Release vom Entwickler Team in den master Branch übernommen.

    Ein von von mir eingerichteter Bot (github-actions) hat den Fehler auch im PR auch kommentiert.

    Du hast als PR eingestellt, dass Du den gesamten develop Branch von SmartHomeNG in den master Branch mergen möchtest (alle 62 Commits, die im develop seit dem letzten Release gemacht wurden in den master Branch übernehmen, als ein Release machen).

    Deine Dateien sind allerdings nicht mal enthalten, da Du den PR nicht aus Deinem github Repo gestellt hast.

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Zitat von Msinn Beitrag anzeigen
    Wenn Du einen Pull Request stellst, schon.
    Habe ich jetzt mal gemacht. Ob das so richtig ware, vage ich zu bezweifeln. Ich war der Meinung ich kann meine Daten direkt hochladen, geht aber nicht.

    Einen Kommentar schreiben:


  • Msinn
    antwortet
    Wenn Du einen Pull Request stellst, schon.

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Ich habe gerade auf 1.9.2 geupdatet. Aber ist ist immer noch die Uralt-Pluggit-Version drin. Wird meien Version mal mit eingebunden?

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Die requirements habe ich auch hochgeladen. Mit der userdoc kann ich leider nicht dienen. :-( Es ist einfach zu umfangreich. Man müsste sich dazu die Modbus-Dokumentation der Pluggit anschauen. Ich weiß selbst nicht mal mehr welcher Parameter wofür ist und habe den aus der Dokumentation entnommen.

    Einen Kommentar schreiben:


  • bmx
    antwortet
    Ok, was mir ein bischen fehlt ist eine requirements.txt Datei wo die benötigten Module und ggf. deren Versionen drinstehen.
    Außerdem wäre eine userdoc.rst ganz prima. Aber jetzt kommt sowieso erstmal das Bugfix Release 1.9.1 raus und dann kann man dabeigehen

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Zitat von bmx Beitrag anzeigen
    Wenn Du das auf github laden kannst als Dein öffentliches Repository, dann kann man sich das vorn dort aus reinladen als privates Plugin.
    Alternativ kannst Du das als zip packen und in ein gist packen so das man sich das von dort aus laden kann.
    Wenn es das Plugin schon gibt, müsste sich jemand anschauen ob es breaking changes gegeben hat oder ob man das einfach dann in SmartHomeNG/Plugins reinschmeissen kann
    Sei mir nicht böse, aber ich habe die Hälfte der Wörter nicht mal verstanden.

    Kurzum: Ich habe es jetzt ins mein github-Profil geladen (auch wenn ich mich damit null auskenne). Aus dem Beta-Branch kann man sich das laden. Gern kann das auch jemand in SmartHomeNG direkt einbinden (das war dann die Stelle, wo ich nichts verstanden hatte).

    Hier der Link: Pluggit-Plugin

    Einen Kommentar schreiben:


  • bmx
    antwortet
    Wenn Du das auf github laden kannst als Dein öffentliches Repository, dann kann man sich das vorn dort aus reinladen als privates Plugin.
    Alternativ kannst Du das als zip packen und in ein gist packen so das man sich das von dort aus laden kann.
    Wenn es das Plugin schon gibt, müsste sich jemand anschauen ob es breaking changes gegeben hat oder ob man das einfach dann in SmartHomeNG/Plugins reinschmeissen kann

    Einen Kommentar schreiben:


  • Cannon
    antwortet
    Ich habe das pluggit-plugin schon vor einiger Zeit erheblich aufgebohrt und nutze nun fast alle unterstützten Parameter. Zudem lassen sich Dateitypen auch innerhalb von Items umwandeln, beispielsweise DateTime in Date oder sowas. Leider bin ich in den letzten Monaten auch nicht weiter dazu gekommen das ganze zu vollenden. Dennoch ist es derzeit sehr umfangreich, funktioniert und könnte auch genutzt werden. Wo kann ich das hoch laden und wer will testen?

    Im Übrigen kann das komplette plugin als struct einbinden. Das spart Arbeit.

    Einen Kommentar schreiben:

Lädt...
X