Pattern analog überblenden - nicht immer

[PeterVT11]

Hallo,

ich bin beim Test darauf gekommen, dass das analoge Überblenden der Pattern-Funktion nicht immer funktioniert.
Nämlich dann nicht, wenn der Mode "PM_SEQUENZ_W_RESTART" verwendet wird und das Pattern über einen Taster gestartet wird.

In diesem Fall arbeitet das Pattern wie, wenn der Analog-Modus nicht gesetzt wäre. Drückt man den Taster dauernd, dann geht auch das Überblenden (aber nur, bis der Taster wieder los gelassen wird).

Getestet mit V3.5.0C1, trat aber auch mit 3.4.0B3 auf.

[PeterVT11]

Hallo,
Ich wollte mal nachfragen, ob es hier eine Lösung geben könnte. Bisher gab es leider keine Reaktion.
Bitte nicht falsch verstehen, ich weiß, dass Lösungen Zeit brauchen. Ich wollt das nur mal wieder in Erinnerung bringen.

[PeterVT11]

Hallo,

das Problem ist nicht der Pattern_Configurator (der erstellt das Pattern korrekt). Auch nicht der Prog_Generator (der gibt das Pattern ordnungsgemäß weiter). Das Problem ist die Umsetzung in den MLL-internen Modulen.

Ausgangslage:
Ein Pattern mit analogen Überblenden. Es wird mit dem Mode "PM_SEQUENZ_W_RESTART" wird über einen Taster gestartet.

Problem:
Solange der Taster gedrückt bleibt, läuft der Farbwechsel mit überblenden der Werte. Läßt man den taster los, so läuft der Farbwechsel ohne Überblenden, d.h. er schaltet von einem Wert direkt zum anderen.

[Eckhart]

Ja klar Peter!

...natürlich ist das ein bug im zentralen Code! Da fehlt die Überprüfung von Run2endAndWait um auch dann die Zwischenzustände der LEDs zu generieren.

Hier ist ein Patch:

diff --git a/src/AnalogPattern.cpp b/src/AnalogPattern.cpp
index c4c59ec..08a6f47 100644
--- a/src/AnalogPattern.cpp
+++ b/src/AnalogPattern.cpp
@@ -608,7 +608,7 @@ void MobaLedLib_C::Proc_AnalogPattern(uint8_t TimeCnt, bool AnalogMode)
//if (!Initialize && AnalogMode) Dprintf("%i %i", (int)v0, (int)v1); // Debug 24.10.19:
}
// Update one LED
- if (AnalogMode && (Inp_Is_On(Inp) || (HSV_mode && !Initialize))) // 09.09.18: Hier war Inp > INP_TURNED_OFF oder so // 22.10.18: Added: HSV_mode to correctly turn off the LEDs
+ if (AnalogMode && (Inp_Is_On(Inp) || (HSV_mode && !Initialize) || (Run2endAndWait && dp->State != PT_INACTIVE))) // 09.09.18: Hier war Inp > INP_TURNED_OFF oder so // 22.10.18: Added: HSV_mode to correctly turn off the LEDs
{
// 20.10.19: Added: !Initialize because otherwise the Cave_Illumination shows random colors at startup
#pragma GCC diagnostic push // 17.11.20: Disable warning "v0 uninitialized..."
#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"

Du musst im File AnalogPattern.cpp die Zeile mit dem "-" durch die Zeile mit dem "+" ersetzen.

Gruß, Eckhart

[PeterVT11]

Hallo Eckhart,

vielen Dank für den Patch.

Code: Alles auswählen

        if (AnalogMode && (Inp_Is_On(Inp) || (HSV_mode && !Initialize) || (Run2endAndWait && dp->State != PT_INACTIVE))) // 09.09.18: 

Das "Run2endAndWait && dp->State != PT_INACTIVE" in Zeile 610 von AnalogPattern.cpp war wohl die Lösung.
Werde es weiter testen.