search

meffken / OSSCR

Open Source Smart Curtain Rod
6 stars 0 forks source link

Ein paar Verbesserungsvorschläge #5

Open EinRainerZufall opened 2 years ago

EinRainerZufall commented 2 years ago

Hi, Ich habe mir jetzt nur mal deinen Code durchgelesen und noch nicht selber getestet. Ich hätte aber ein paar Verbesserungsvorschläge,

Als letzten hätte ich die Frage ob ich einen Fork machen darf um zu probieren ob man OSSCR mit HomeSpan in HomeKit integrieren kann?

ansonsten ist das sehr gute und solide Arbeit 👍🏻

mfg Tim

meffken commented 2 years ago

Moin Tim, Freut mich, dass du dir meine Bastelei angesehen hast und Freude daran hast! Die Verbesserungsvorschläge klingen sehr sinnvoll, ich habe aber auch sehr lange nicht mehr in den Code geschaut… Wenn du Ihn verbessern magst, würde ich mich voll drüber freuen! Ich würde mich auch eher in der Konstruktion sehen, als bei der Programmiererei... hatte mich schon fast geschämt meinen Try-and-Error Code hochzuladen 😃 Der Verbesserungsvorschlag mit dem anderen Stepper-Befehl klingt super sinnvoll. Das ist dann auf jeden Fall die elegantere Lösung! Du darfst jegliche Änderung vornehmen und natürlich freue ich mich auch über einen Fork von dir! Ansonsten gibt es anscheinend auch eine Gardinen-Funktion in Tasmota, das habe ich aber nie getestet…

Danke für die Blumen und die Nachricht von dir! Marius Wendt  

    Student – allgemeiner Maschinenbau (Hochschule Hannover / Matr. Nr.: 1470534) mobil: (0)176 / 83 53 08 43     Contruct Event GbR https://construct-event.de/ Haltenhoffstraße 36 30167 Hannover     Vorstandsmitglied Hannoversche Kunstplattform e.V.  https://hannoversche-kunstplattform.de/ Haltenhoffstraße 36 30167 Hannover

Von: Tim Gesendet: Sonntag, 24. April 2022 19:58 An: meffken/OSSCR Cc: Subscribed Betreff: [meffken/OSSCR] Ein paar Verbesserungsvorschläge (Issue #5)

Hi, Ich habe mir jetzt nur mal deinen Code durchgelesen und noch nicht selber getestet. Ich hätte aber ein paar Verbesserungsvorschläge, • Ich würde die Microschalter als NC benutzten, damit könntest du das Spiel mit stepcounter ±50 dir sparen und der ESP würde auch einen Kabelbruch erkennen • wenn du statt stepper1.moveTo(stepcounter), stepper1.runToPosition(stepcounter) benutzt sollte der WDT vom ESP nicht mehr anspringen Als letzten hätte ich die Frage ob ich einen Fork machen darf um zu probieren ob man OSSCR mit HomeSpan in HomeKit integrieren kann? ansonsten ist das sehr gute und solide Arbeit 👍🏻 mfg Tim — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>