ColorCircle without artfacts

SmartHome-yourself / m5-dial-for-esphome

M5 Stack Dial - Custom Component for ESPHome

MIT License

57 stars 8 forks source link

ColorCircle without artfacts #1

Closed ArduFox closed 3 months ago

ArduFox commented 3 months ago

Der neue Code zeichnet einen perfekten Farbkreis

danielscheidler commented 3 months ago

Der neue Code zeichnet einen perfekten Farbkreis

Sehr gute Idee Dreiecke zu nutzen. Der erste Ansatz war erst mal quick and dirty um überhaupt voran zu kommen, aber die Dreiecke sind natürlich viel besser, als die mehrfachen Linien. Dazu wohl auch wesendlich performanter. Wenn ich das richtig sehe, hast du aber im main-branch geändert. Daher würde ich das so nur ungern 1:1 übernehmen, da ich im dev-branch bereits massives Refactoring betrieben habe. Den Farbkreis übernehme ich aber auf jeden Fall nachher mal in den Dev branch.

Für kommende Anpassungen also am besten immer den DEV nutzen. Wir können uns gerne auch mal auf Discord austauschen. https://smarthomeyourself.de/discord

danielscheidler commented 3 months ago

Zu deinen Notizen:

weißtöne sind im dev bereits als neuer Mode enthalten
wer unbedingt drauf besteht die wifi daten plain in der yaml zu hinterlegen kann das über die substitutions ja tun. Als Standard sollte aber doch eher die saubere Variante über secrets bestehen bleiben. Steht ja jedem frei, die substitutions einfach mit strings zu überschreiben
Der Wechsel von offline zu disconnected ist normal. Offline = Kein WIFI, Disconnected = Wifi, aber noch keine Verbindung zur HA API. (Dafür muss das Gerät erst in HA integriert werden über einstellungen -> Geräte)
Wenn du noch nichts steuern kannst, fehlt wahrscheinlich in der Integration der Haken für Erlaubnis, HA-Dienste aufzurufen (Einstellungen - Geräte - ESPHome - Dein Gerät >> Konfigurieren >> Haken setzen)

ArduFox commented 3 months ago

Hallo Daniel, ich frische gerade meine CPP und github Kenntnisse auf und nutze Deinen sehr klaren Code, um Erweiterungen in esphome zu verstehen und evtl. etwas beizutragen. Bei aller Freude über Dein github hatte ich tatsächlich den dev Branch übersehen ...

Mein M5Dial läuft mit Deinem Code und meinen kleinen Experimenten und nun schaue ich mal, was die deutlich umfangreichere, aber sicherlich immer mal nicht perfekt stabile, dev-Version so kann.

Neue commits mache ich dann direkt in dev - falls ich wieder mal was beitragen kann. Eine Idee die ich habe, ist ein Menü analog zum M5Dial Beispiel zu integrieren.

Vielen Dank an Dich für die Mühe und Zeit, die Du rein steckst.

ArduFox commented 3 months ago

Autsch, eigentlich wollte ich auch nur einen commit für die Dreiecke machen, nun sehe ich dass mein ganzer Wanderweg mit microcommits angekommen ist - sorry.

danielscheidler commented 3 months ago

Autsch, eigentlich wollte ich auch nur einen commit für die Dreiecke machen, nun sehe ich dass mein ganzer Wanderweg mit microcommits angekommen ist - sorry.

Alles gut. Alle commits laufen ja unter einem Pullrequest. Das heist, ich kann diese als einen einzigen Merge später übernehmen. Das kannst du also gerne genau so für die Zukunft beibehalten.

Mit dem Menü hatte ich auch schon überlegt, zum Beispiel für eine einfachere Geräteauswahl als das durch wischen durch die einzelnen Geräte. Meine Idee wäre, mit einem Langen druck auf die Taste in dieses Menü zu gelangen wo man dann z.b. die Geräte direkt anspringen kann. Vllt lässt sich da aber auch noch mehr mit umsetzen?

Ich habe mal die discussions geöffnet um sich über Ideen auszutauschen: https://github.com/SmartHome-yourself/m5-dial-for-esphome/discussions/5