jp112sdl / Beispiel_AskSinPP

88 stars 36 forks source link

HM-PB-MP-WM #28

Closed eddy-ch closed 5 years ago

eddy-ch commented 5 years ago

Hy Jerome habe diesen super Sketch gefunden und ihn gleich eingesetzt, leider bleibt er nach unbestimmten Klicks hängen und nur ein reset hilft weiter. habe schon an den Zeiten gedreht, leider ohne Erfolg.

jp112sdl commented 5 years ago

Hi! Das ist komisch... ich hatte sowas schon mal bei einem normalen Wandtaster. Kannst du ungefähr abschätzen, wie häufig das vorkommt?

eddy-ch commented 5 years ago

Das ist sehr unterschiedlich, wenn ich es provozieren viel (viele tastendrücke hintereinander) nach ca. 10 -30 mal bleibt es manchmal hängen. meistens wenn die Frau Licht anmachen will und das ist es frustrierend.

jp112sdl commented 5 years ago

Also die gute Nachricht: Ich konnte es reproduzieren und auch bei mir tritt der Fehler nur sporadisch nach einem nicht nachvollziehbaren Muster auf. Die schlechte: Ich konnte die Ursache noch nicht lokalisieren. Jedoch habe ich eine Idee, die bei mir zumindest offensichtlich Besserung bringt.

Ändere in der ButtonMP.h bitte mal Zeile 29: #define DEBOUNCETIME millis2ticks(50) (warum auch immer ich damals auf 30ms abgewichen bin...)

und füge nach Zeile 102:

sysclock.cancel(ma);

ein. Der Abschnitt sieht dann so aus:

...
      switch ( state() ) {
        case released:
          ma.set(millis2ticks(MULTIPRESSTIME));
          sysclock.cancel(ma);
          sysclock.add(ma);
        case longreleased:
...
eddy-ch commented 5 years ago

vielen Dank, werde es sofort testen und berichten. habe übrigens beim entfernen von auch eine Besserung festgestellt, vlt. hilft es dir weiter.

jp112sdl commented 5 years ago

Das kann ich nahezu ausschließen, da die LowPower in der Activity.h so oder so gezogen wird. Die Referenz im Sketch war eigentlich obsolet.

eddy-ch commented 5 years ago

habe es seit gestern ohne Aussetzer im Einsatz. Denke das du es hinbekommen hasst.

jp112sdl commented 5 years ago

Danke für die Info! Klingt echt gut... ich werds mal in den master-Code pushen.