Neuauflage des Moduls 98_Siro.pm

[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

[CoolTux]

Guten Morgen Thomas 

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


Grüße

[betateilchen]

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.

[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

[Byte09]

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]

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]

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

[Byte09]

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]

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.

[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]

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

[Byte09]

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]

Kein Problem.

https://github.com/Byte009/Siro/pulls

Grüße