Neuauflage des Moduls 98_Siro.pm

Begonnen von Byte09, 17 Mai 2019, 06:06:34

Vorheriges Thema - Nächstes Thema

Byte09

Guten Morgen,

da das 'alte' Siromodul mein erstes Fhemmodul war und ich jetzt einiges anders/effektiver machen würde habe ich mich entschlossen , das Modul komplett zu erneuern.

In der neuen Version werden einige Funktionen wegfallen , von denen ich denke , dsas sie ungenutzt sind ( z.B die verschiedenen Betriebsmodi ) .

Da in der neuen Version u.A alle Attribute und z.T auch Readings anders benannt werden ist ein einfacher Wechsel zwischen den Versionen nicht so einfach möglich.d.H ich werde das Modul in der Testphase anders benennen müssen , was aber zur Folge hat , das die Test version auch nur mit einer gepachten Version des Signaulduino-moduls funktionieren wird.
Diese werde ich zusammen mit der ersten Testversion hier zur Verfügung stellen.

Mit der ersten offiziellen Version werde ich den alten Modulnamen übernehmen und eine Umstellung der vorhandenen Devices auf neue Bezeichnungen etc. wird dann automatisch erfolgen.

Gerne greife ich Wünsche und Anregungen auf, wenn Sie allgemein Sinnig und umsetzbar sind.

gruss Byte09


edit:

eine Verteilung des Moduls erfolgt nicht mehr über das FHEM-SVN . Eine Installation muss über GitHub erfolgen.
update all https://raw.githubusercontent.com/Byte009/FHEM-Siro/master/controls_siro.txt

CoolTux

Guten Morgen Thomas  ;D

1. Die ermittelte neue Position des Rolllos erst nach Beendigung der Fahrt als Reading setzen.


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

betateilchen

Zitat von: Byte09 am 17 Mai 2019, 06:06:34
d.H ich werde das Modul in der Testphase anders benennen müssen , was aber zur Folge hat , das die Test version auch nur mit einer gepachten Version des Signaulduino-moduls funktionieren wird.
Diese werde ich zusammen mit der ersten Testversion hier zur Verfügung stellen.

Lege doch die Datei unter dem richtigen Namen in ./contrib ab und poste dann Links zum Download aus Trac.

Dateien in Threads ablegen, finde ich immer extrem umständlich. Einen Trac-Link kann man einfach kopieren und per wget auf der Zielplattform in der Konsole verwenden. Das ist meist einfacher, als eine Datei manuell kopieren zu müssen.
-----------------------
Formuliere die Aufgabe möglichst einfach und
setze die Lösung richtig um - dann wird es auch funktionieren.
-----------------------
Lesen gefährdet die Unwissenheit!

Byte09

das erste Grundgerüst steht - als kompletter 'Neuaufbau'

Im Screenshot - geänderte Readings - geanderte Attribute etc.

ich denke zu Ende der Woche werde ich die erste Version des neuen Moduls zur Verfügung stellen.

gruss Byte09


CoolTux

Cool. Du nimmst pct für Positions Angabe? Ist der set Befehl dann auch pct? Das wäre wichtig zu wissen?

Darf man sich den Code anschauen? Hast Du da was auf GitHub?


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Byte09

Zitat von: CoolTux am 20 Mai 2019, 18:27:29
Cool. Du nimmst pct für Positions Angabe? Ist der set Befehl dann auch pct? Das wäre wichtig zu wissen?

Darf man sich den Code anschauen? Hast Du da was auf GitHub?


Grüße

ja - zumindest im Moment - hatte ich in der bisherigen Version allerdings nicht und weiss auch noch nicht , ob das der Weisheit letzter Schluss ist.

klar kannst du Ihn dir anschauen , ist aber noch sehr rudimentär.
ich stelle ihn gleich in das GIT und gebe dir den Link

gruss Thomas

Byte09

#6
Zitat von: CoolTux am 20 Mai 2019, 18:27:29

Darf man sich den Code anschauen? Hast Du da was auf GitHub?


Grüße

https://github.com/Byte009/Siro

gruss Thomas


edit
alte Befehle werde ich aber intern auf die neuen mappen, um nicht alle Abhängigkeiten durcheinander zu bringen ( notify etc.)

CoolTux

Zitat von: Byte09 am 20 Mai 2019, 18:43:08
https://github.com/Byte009/Siro

gruss Thomas


edit
alte Befehle werde ich aber intern auf die neuen mappen, um nicht alle Abhängigkeiten durcheinander zu bringen ( notify etc.)

Super,

Ich danke Dir. Ich schaue es mir gerne an. Ja mappen ist ne gute Idee, zu mindest die ersten 6 Monate. Ich frage auch wegen ASC (AutoShuttersControl)


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Byte09

Zitat von: CoolTux am 20 Mai 2019, 19:03:58
Super,

Ich danke Dir. Ich schaue es mir gerne an. Ja mappen ist ne gute Idee, zu mindest die ersten 6 Monate. Ich frage auch wegen ASC (AutoShuttersControl)



Grüße

ist PCT für die Funktion mit ASC ungeeignet ? ... spielt für mich fast keine Rolle, d.H ich kann es anpassen wie es am besten passt und zusammenarbeitet ?!

habe mich mit ASC noch nicht wirklich beschäftigt.

Gruss Thomas

CoolTux

Zitat von: Byte09 am 20 Mai 2019, 19:30:32
ist PCT für die Funktion mit ASC ungeeignet ? ... spielt für mich fast keine Rolle, d.H ich kann es anpassen wie es am besten passt und zusammenarbeitet ?!

habe mich mit ASC noch nicht wirklich beschäftigt.

Gruss Thomas

ASC ist es eigentlich egal. Kannst es also so lassen. Es gibt aber TYPEN wo wir den set Befehl schon am TYPE festgemacht haben. Das muss ich dann halt irgendwann ändern. Ist aber für Dich und Deine Arbeit so jetzt nicht wichtig. Wichtig ist das ich es weiß und dann passt das.
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Byte09

ich habe den code im github gerade auf heutigen stand aktualisiert.

fahrt runter - ok
fahrt hoch -ok
positionsanfahrt - ok
wechsel der richtung oder position während der fahrt - ok

favoritenanfahrt - nA
proc - nA
up/down-for-timer - nA
parse ( empfang der FB ) -nA

gruss Byte09

Byte09

#11
update im GIT.

fahrt runter - ok
fahrt hoch -ok
positionsanfahrt - ok
wechsel der richtung oder position während der fahrt - ok

favoritenanfahrt - ok
favorit setzen - ok
favorit löschen - ok

proc - z.T. ok

up/down-for-timer - nA
parse ( empfang der FB ) -nA

Logeinträge zur Zeit nur verbose5 - davon aber reichlich

gruss Byte09

CoolTux

Hallo Thomas,

Weißt schon ob Du mein pull request einbauen tust?


Grüße
Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net

Byte09

Zitat von: CoolTux am 22 Mai 2019, 19:38:58
Hallo Thomas,

Weißt schon ob Du mein pull request einbauen tust?


Grüße

::)

auch auf die gefahr hin , das ich jetzt doof dastehe  ??? . Kannst du mir mal auf die Sprünge helfen was du meinst  .

( gott wie peinlich  ;) )

gruss Thomas

CoolTux

Du musst nicht wissen wie es geht! Du musst nur wissen wo es steht, wie es geht.
Support me to buy new test hardware for development: https://www.paypal.com/paypalme/MOldenburg
My FHEM Git: https://git.cooltux.net/FHEM/
Das TuxNet Wiki:
https://www.cooltux.net