Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Berndeutsch layout 11x10 + Sunset feature #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JCMengisen
Copy link

Added Berndeutsch layout 11x10
Added sunset darkmode (brightness 50%) instead of Nightmode(LED off) as LUT or polynomial

Hallo Edgar
Ich denke als Erstes ist es nicht die Idee, dass wir direkt in deinen Main branch mergen. Vielmehr wird es eher ein Review sein und anschliessend kann ich die neuen Features so anpassen, dass sie kompatibel sind zu deinem Konzept. Als Erstes habe ich den Darkmode einfach direkt in deinen NighMode integriert. Das kann man nautürlich trennen oder schöner machen, Wenn du dieses Feature sowieso nicht möchtest, lass ich es einfach auf meinem branch. Wir sehen! ;-)
=> Ich habe mir noch überlegt ob ich den PR auf draft stelle, bin mir aber nicht sicher ob du dann die Meldungen schon bekommst. => Du darfst es auch auf Draft zurück stellen!

Übrigens cooles Projekt und endlich ewas sinnvolles für die Arduinos!

Claude

Added sunset darkmode (brightness 50%) instead of Nightmode(LED off) as LUT or polynomial
@JCMengisen JCMengisen marked this pull request as draft February 20, 2025 10:21
@JCMengisen JCMengisen marked this pull request as ready for review February 21, 2025 07:21
@techniccontroller
Copy link
Owner

Hi Claude,

vielen Dank für den PR. Wie ich sehe sind in dem PR vier Änderungen drin, die ich gerne etwas aufteilen würde:

  • Änderung auf 10 Zeilen, statt 11: das ist keine relevante Änderung, die habe ich bereits als Beschreibung auf meiner Webseite -> kein PR
  • Berndeutsch: kann als neue Datei wordclockfunctions.ino_berndeutsch direkt auf main branch gemergt werden -> neuer PR mit nur dieser neuen Datei
  • Sunset feature: sehr gutes feature, würde ich als feature branch veröffentlichen. Ich würde allerdings die latitude und timezone Daten nutzen die ich über den API request beim starten der Uhr bekommen. Gerne kannst du erstmal deine variante mit lookup Tabelle als PR erstellen, ich kann dann meine Änderungen zusätzlich einfügen -> neuer PR
  • reduzierte Helligkeit im Nightmode: ich würde die Helligkeit während des Nightmode als Setting einfügen (default 0%) -> ich kann hierfür einen PR machen.

Allgemein würde ich dich bitte meinen main branch vor eine PR einmal in deinen branch zu mergen, damit die neuen Änderungen mit drin sind.

Viele Grüße
Edgar

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.

2 participants