Sonof Obi Switch und Tasmota Prellen

#6071321
Lesenswert?

Luigi A. schrieb:
> Ist da keine Software Entprellung
> implementiert?

Kann gut sein, viele Entprellversionen taugen leider nichts.
Zuverlässig ist nur eine Filterung durch Mehrfachabtastung, d.h. in 
einem Timerinterrupt (z.B. 10ms) wird die Anzahl aufeinanderfolgender 
gleicher Zustände ausgewertet.
1
Prinzip (>=4 Abtastungen):
2
Taste:    01010101111101010100000
3
                    |          |
4
Entprellt:00000000001111111111100

Torsten schrieb:
> Doch. Guckst du hier:
> https://github.com/arendst/Tasmota/blob/development/tasmota/support_button.ino

Puh, knapp 300 Zeilen, das ist ja riesig.
Auf den ersten Blick kann ich nicht erkennen, wo da entprellt werden 
soll.
#6071447
Lesenswert?

Peter D. schrieb:
> Auf den ersten Blick kann ich nicht erkennen, wo da entprellt werden
> soll.

Is etwas unglücklich mit reingewurschtelt:
1
if (NOT_PRESSED == button) {
2
          Button.hold_timer[button_index] = 0;
3
        } else {
4
          Button.hold_timer[button_index]++;
5
          if (Settings.flag.button_single) {                   // SetOption13 (0) - Allow only single button press for immediate action
6
            if (Button.hold_timer[button_index] == loops_per_second * hold_time_extent * Settings.param[P_HOLD_TIME] / 10) {  // SetOption32 (40) - Button held for factor times longer
7
//              Settings.flag.button_single = 0;
8
              snprintf_P(scmnd, sizeof(scmnd), PSTR(D_CMND_SETOPTION "13 0"));  // Disable single press only
9
              ExecuteCommand(scmnd, SRC_BUTTON);
10
            }
Das dürfte je nach Laufzeit (wann gedrückt), mehr schlecht als recht 
funktionieren?! :´(
#6071474
Lesenswert?

Teo D. schrieb:
> Is etwas unglücklich mit reingewurschtelt:

Da ist ne ziemliche Menge reingewuselt. Ich mag riesige if/else Ketten 
überhaupt nicht. Da verliert man sehr leicht den Überblick. D.h. es 
können sich schnell Fehler einschleichen und es ist schwer zu 
überprüfen, ob der Code wirklich das Gewünschte macht.
Ich unterteile lieber in kleine Unterfunktionen, die man für sich 
implementieren und überprüfen kann.
Tief verschachtelte if/else Ketten vermeide ich generell.
Falls wirklich komplexe Abläufe nötig sind, bevorzuge ich switch/case.


Teo D. schrieb:
> if (NOT_PRESSED == button) {
>           Button.hold_timer[button_index] = 0;

Das heißt dann wohl, beim Loslassen wird nicht entprellt.
#6071612
Lesenswert?

Hallo Luigi,

die zweite Generation von Obi und der Schalter für draußen haben 
Probleme mit reboots beim Schalten. Ist wohl ein Hardwareproblem mit der 
reset-Leitung. Man kann das an der uptime erkennen. Bei mir hat das 
genau die von Dir beschriebenen Symptome.

https://github.com/arendst/Tasmota/issues/4829
https://github.com/arendst/Tasmota/issues/5511
https://forum.iobroker.net/topic/17060/obi-funk-steckdosenumbau-esp8266-generation2-eckig

Gruß,
stbu

Antwort schreiben

Bitte melde dich an, um einen Beitrag zu schreiben.

oder

Mit Google-Account einloggen

Die Registrierung ist kostenlos und dauert nur eine Minute.

Jetzt registrieren