Skip to content

Conversation

@gbrehmer
Copy link

kleine Optimierung, wenn dx = dmax
Findet passenden Wert aus verfügbaren Ganzen, Viertel und Achtel Schritten

@gbrehmer
Copy link
Author

gbrehmer commented Jan 30, 2019

Musste noch die platformio.ini anpassen und die 3rd party libs hinzufügen. Er hat sie nicht automatisch erkannt. Hab die aktuellste PlatformIO Version. Ist aber auch mein erstes PlatformIO Projekt, also kann natürlich sein, dass es da auch noch andere Lösungen gibt (Umgebungsvariablen, Einstellungen o.ä.). Ansonsten hab ich nach Tests noch für näherungsweise bessere Werten zwischendrin mit Float-Werten gerechnet. Gibt noch kleine Probleme am Ende zu 0, da kurz noch eine andere Farbe an ist, obwohl die Hauptfarbe am längsten laut der Schritte an sein sollte (also man dimmt einen grünlichen Ton auf/ab und am Ende leuchtet noch kurz rot oder blau allein schwach und grün ist schon aus.

@gbrehmer
Copy link
Author

gbrehmer commented Feb 5, 2019

Kann es sein, dass 4 fach und 8 fach Schritte im Vergleich etwas langsamer sind (= eventuell öfter neu berechnet = mehr Berechnungsdelay o.ä.)? Ich hab im worker noch nicht alle Fälle durchblickt, aber es würde erklären, dass die Farben die ich in den 4 fach statt 1 fach Bereich verschiebe länger brauchen

Gerrit Brehmer and others added 2 commits December 27, 2020 21:09
@gbrehmer
Copy link
Author

@LechnerRobert ich hatte wohl irgendeinen Denkfehler damals, zumindest weiß ich auch nicht mehr warum die vorherige Lösung korrekt sein sollte. Ich war wahrscheinlich durch nicht die nicht linearen Dimmkurven durcheinander geraten.
Habe jetzt eine Lösung für lineare Dimmkurven, geht bestimmt schöner vom Code her, aber funktioniert ;) Gerade beim Pulsieren von Farben fand ich es zuvor sehr störend, dass die Farbe erst am Zielwert korrekt war.
Vielleicht gibt es auch noch eine Lösung mit Berücksichtigung der Dimmkurve != linear, aber dafür müsste man wohl die Dimmkurve und die Dimmschrittgröße berücksichtigen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant