Merge branch 'main' of ssh://192.168.178.10:2022/ranse/wordclock

.gitignore

@@ -11,6 +11,9 @@
 # Own folders
 _unused/
 
+# Own files
+log.txt
+
 # Prerequisites
 *.d
 
@@ -62,4 +65,15 @@ _unused/
 modules.order
 Module.symvers
 Mkfile.old
-dkms.conf
+dkms.conf
+
+# Python
+venv
+.venv
+
+# Logs
+*.log
+
+# Local backup
+*.*_
+*.*bak*

README.md

@@ -1,96 +1,49 @@
 # Important Note:
-This project has been unofficially forked from https://github.com/techniccontroller/wordclock_esp8266 which was initially created by techniccontroller. Copyright and licensing is respected.
+This project has been unofficially forked from https://github.com/techniccontroller/wordclock_esp8266 which was initially created by techniccontroller. Copyright and licensing is respected. Very many thanks for the initial code!
 
 
 # Wordclock 2.0
 
 Wordclock 2.0 with ESP8266 and NTP time
 
-More details on my website: https://techniccontroller.com/word-clock-with-wifi-and-neopixel/
-
+More details on techniccontroller's website: https://techniccontroller.com/word-clock-with-wifi-and-neopixel/
 
 **Languages**
 
-The Wordclock is available in **German**, **English** and **Italian** language. By default the language is German. 
-To use the English or Italian language please replace the file *wordclockfunctions.ino* with *wordclockfunctions.ino_english* or *wordclockfunctions.ino_italian*.
-The code compiles only with one file named *wordclockfunctions.ino*. So please rename the file you want to use to *wordclockfunctions.ino* and replace the existing file.
+The Wordclock is available in **German** language.
 
 
 ## Features
-- 6 modes (Clock, Digital Clock, SPIRAL animation, TETRIS, SNAKE, PONG)
-- time update via NTP server
-- automatic summer/wintertime change
-- easy WIFI setup with WifiManager
-- configurable color
-- configurable night mode (start and end time)
-- configurable brightness
-- automatic mode change
-- webserver interface for configuration and control
-- physical button to change mode or enable night mode without webserver
-- automatic current limiting of LEDs
+- Time update via NTP server
+- Games: Pong, Snake, Tetris, ...
+- Automatic summer/wintertime change
+- Easy wifi setup with WifiManager
+- Configurable color
+- Configurable night mode (start and end time)
+- Configurable brightness
+- Automatic mode change
+- Webserver interface for configuration and control (web address: wordclock.local)
+- Automatic current limiting of LEDs
 
 ## Pictures of clock
 ![modes_images2](https://user-images.githubusercontent.com/36072504/156947689-dd90874d-a887-4254-bede-4947152d85c1.png)
 
-## Screenshots of webserver UI
-![screenshots_UI](https://user-images.githubusercontent.com/36072504/158478447-d828e460-d4eb-489e-981e-216e08d4b129.png)
-
 ## Quickstart
 
-1. Clone the project into the sketch folder of the Arduino IDE, 
-2. Rename the file "example_secrets.h" to "secrets.h". You don't need to change anything in the file if you want uses the normal WiFi setup with WiFiManager (see section "Remark about the WiFi setup").
-3. Install the additional libraries and flash it to the ESP8266 as usual (See section [*Upload program to ESP8266*](https://github.com/techniccontroller/wordclock_esp8266/blob/main/README.md#upload-program-to-esp8266-with-arduino-ide) below). 
-4. The implemented WiFiManager helps you to set up a WiFi connection with your home WiFi -> on the first startup it will create a WiFi access point named "WordclockAP". Connect your phone to this access point and follow the steps which will be shown to you. 
-5. After a successful WiFi setup, open the browser and enter the IP address of your ESP8266 to access the interface of the webserver. 
-6. Here you can then upload all files located in the folder "data". Please make sure all icons stay in the folder "icons" also on the webserver.
-
-
-<img src="https://techniccontroller.com/wp-content/uploads/filemanager1-1.png" height="300px" /> <img src="https://techniccontroller.com/wp-content/uploads/filemanager2-1.png" height="300px" /> <img src="https://techniccontroller.com/wp-content/uploads/filemanager3-1.png" height="300px" />
-
-## Install needed Libraries
-
-Please download all these libraries as ZIP from GitHub, and extract them in the *libraries* folder of your Sketchbook location (see **File -> Preferences**):
-
-- https://github.com/adafruit/Adafruit-GFX-Library
-- https://github.com/adafruit/Adafruit_NeoMatrix
-- https://github.com/adafruit/Adafruit_NeoPixel
-- https://github.com/tzapu/WiFiManager
-- https://github.com/adafruit/Adafruit_BusIO
+1. Clone the project into the a directory.
+2. Open directory in VSCode with PlatformIO extension installed.
+3. Current dependencies (2024-07-21):
+- Platform espressif8266 @ 2.6.3 (required: espressif8266 @ 2.6.3)
+- Libraries
+- ├── FastLED @ 3.7.0 (required: fastled/FastLED @ 3.7.0)
+- ├── FastLED NeoMatrix @ 1.2.0 (required: marcmerlin/FastLED NeoMatrix @ ^1.2)
+- │⠀⠀⠀└── Framebuffer GFX @ 1.1.0 (required: Framebuffer GFX)
+- │⠀⠀⠀│⠀⠀⠀├── Adafruit GFX Library @ 1.11.9 (required: Adafruit GFX Library)
+- │⠀⠀⠀│⠀⠀⠀│⠀⠀⠀└── Adafruit BusIO @ 1.16.1 (required: Adafruit BusIO)
+- ├── WiFiManager @ 0.16.0 (required: tzapu/WiFiManager @ ^0.16.0)
+- └── base64 @ 1.4.0 (required: densaugeo/base64 @ ^1.4.0)
 
 
 ## Remark about the WiFi setup
 
-Regarding the Wifi setting, I have actually implemented two variants: 
-1. By default the WifiManager is activated. That is, the word clock makes the first time its own WiFi (should be called "WordclockAP"). There you simply connect to the cell phone and you can perform configuration of the WiFi settings conveniently as with a SmartHome devices (Very elegant 😊)
-2. Another (traditional) variant is to define the wifi credentials in the code (in secrets.h). 
-    - For this you have to comment out lines 230 to 251 in the code of the file *wordclock_esp8266.ino* (/\* before and \*/ after) 
-    - and comment out lines 257 to 305 (/\* and \*/ remove)
-
-
-## Remark about Logging
-
-The wordclock send continuously log messages to the serial port and via multicast UDP. If you want to see these messages, you have to 
-
-- open the serial monitor in the Arduino IDE (Tools -> Serial Monitor). The serial monitor must be set to 115200 baud.
-
-OR
-
-- run the following steps for the multicast UDP logging:
-
-1. starting situation: wordclock is connected to WLAN, a computer with installed Python (https://www.python.org/downloads/) is in the same local area network (WLAN or LAN doesn't matter).
-3. open the file **multicastUDP_receiver.py** in a text editor and in line 81 enter the IP address of the computer (not the wordclock!).
-```python	
-# ip address of network interface
-MCAST_IF_IP = '192.168.0.7'
-```
-4. execute the script with following command: 
-
-```bash
-python multicastUDP_receiver_analyzer.py
-```
-
-5. now you should see the log messages of the word clock (every 5 seconds a heartbeat message and the currently displayed time). 
-If this is not the case, there could be a problem with the network settings of the computer, then recording is unfortunately not possible.
-
-6. If special events (failed NTP update, reboot) occur, a section of the log is saved in a file called *log.txt*. 
-In principle, the events are not critical and will occur from time to time, but should not be too frequent.
+By default the WifiManager is activated. That is, the word clock makes the first time its own WiFi (should be called "WordclockAP"). There you simply connect to the cell phone and you can perform configuration of the WiFi settings conveniently as with SmartHome devices.

include/animation_functions.h

@@ -4,6 +4,8 @@
 #include <Arduino.h>
 #include "wordclock_constants.h"
 
+extern bool spiral_direction;  // Direction of sprial animation
+
 enum Direction
 {
     RIGHT,

include/base64_wrapper.h

@@ -1,26 +0,0 @@
-/*
-Copyright (C) 2016 Arturo Guadalupi. All right reserved.
-
-This library is free software; you can redistribute it and/or modify it under the terms of the GNU Lesser General Public License as published by the Free Software Foundation; either version 2.1 of the License, or (at your option) any later version.
-
-This library is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License for more details.
-*/
-
-#ifndef BASE64_WRAPPER_H
-#define BASE64_WRAPPER_H
-
-class Base64Class{
-  public:
-    int encode(char *output, char *input, int inputLength);
-    int decode(char * output, char * input, int inputLength);
-    int encodedLength(int plainLength);
-    int decodedLength(char * input, int inputLength);
-
-  private:
-    inline void fromA3ToA4(unsigned char * A4, unsigned char * A3);
-    inline void fromA4ToA3(unsigned char * A3, unsigned char * A4);
-    inline unsigned char lookupTable(char c);
-};
-extern Base64Class Base64;
-
-#endif /* BASE64_WRAPPER_H */

include/diagnosis.h

@@ -0,0 +1,26 @@
+#ifndef DIAGNOSIS_H
+#define DIAGNOSIS_H
+
+#include <Arduino.h>
+#include "led_matrix.h"
+#include "udp_logger.h"
+
+class Diagnosis
+{
+public:
+    Diagnosis();                                     // constructor
+    Diagnosis(UDPLogger *logger, LEDMatrix *matrix); // constructor
+
+    String handle_command(const String &command);
+    String print_device_info();
+    String print_sketch_info();
+    String print_last_reset_details();
+    String print_matrix_fps();
+
+private:
+    UDPLogger *_logger;
+    LEDMatrix * _matrix;
+    void print(const String &s);
+};
+
+#endif // DIAGNOSIS_H

include/led_matrix.h

@@ -1,24 +1,28 @@
 #ifndef LEDMATRIX_H
 #define LEDMATRIX_H
 
+#ifndef FASTLED_INTERNAL
+#define FASTLED_INTERNAL
+#endif
+
 #include <Arduino.h>
 #include <Adafruit_GFX.h>
-#include <Adafruit_NeoMatrix.h>
+#include <FastLED_NeoMatrix.h>
 #include "wordclock_constants.h"
 #include "udp_logger.h"
 
 #define DEFAULT_CURRENT_LIMIT 9999
 
 extern const uint32_t colors_24bit[NUM_COLORS];
-
 class LEDMatrix
 {
 public:
-    LEDMatrix(Adafruit_NeoMatrix * matrix, uint8_t brightness, UDPLogger * logger);
+    LEDMatrix(FastLED_NeoMatrix *matrix, uint8_t brightness, UDPLogger *logger);
     static uint16_t color_24_to_16bit(uint32_t color24bit);
     static uint32_t color_24bit(uint8_t r, uint8_t g, uint8_t b);
     static uint32_t interpolate_color_24bit(uint32_t color1, uint32_t color2, float factor);
     static uint32_t wheel(uint8_t WheelPos);
+    uint16_t get_fps(void);
     void draw_on_matrix_instant();
     void draw_on_matrix_smooth(float factor);
     void flush(void);
@@ -31,26 +35,26 @@ public:
     void setup_matrix();
 
 private:
-    Adafruit_NeoMatrix * _neomatrix;
-    UDPLogger * _logger;
+    FastLED_NeoMatrix *_neomatrix;
+    UDPLogger *_logger;
 
     uint8_t _brightness;
     uint16_t _current_limit;
 
     // target representation of matrix as 2D array
-    uint32_t _target_grid[MATRIX_HEIGHT][MATRIX_WIDTH] = {0};
+    uint32_t _target_grid[MATRIX_HEIGHT][MATRIX_WIDTH];
 
     // current representation of matrix as 2D array
-    uint32_t _current_grid[MATRIX_HEIGHT][MATRIX_WIDTH] = {0};
+    uint32_t _current_grid[MATRIX_HEIGHT][MATRIX_WIDTH];
 
-    // target representation of minutes indicator leds
+    // target representation of minutes indicator LEDs
     uint32_t _target_minute_indicators[4] = {0, 0, 0, 0};
 
-    // current representation of minutes indicator leds
+    // current representation of minutes indicator LEDs
     uint32_t _current_minute_indicators[4] = {0, 0, 0, 0};
 
     void _draw_on_matrix(float factor);
     uint16_t _calc_estimated_led_current(uint32_t color);
 };
 
-#endif  /* LEDMATRIX_H */
+#endif /* LEDMATRIX_H */

include/ntp_client_plus.h

@@ -1,90 +0,0 @@
-#ifndef NTPCLIENTPLUS_H
-#define NTPCLIENTPLUS_H
-
-#include <Arduino.h>
-#include <WiFiUdp.h>
-
-#define UNIX_TIMESTAMP_1900 2208988800UL // careful: positive value
-#define NTP_PACKET_SIZE 48
-#define NTP_DEFAULT_LOCAL_PORT 1337
-#define MAX_NTP_CONN_TRIES 50       // 50 * NTP_RECEIVE_WAIT_TIME_MS => 500ms
-#define NTP_RECEIVE_WAIT_TIME_MS 10 // 10ms
-
-typedef enum
-{
-    NTP_UPDATE_TIMEOUT = -1,
-    NTP_UPDATE_SUCCESS = 0,
-    NTP_UPDATE_DIFFTOOHIGH = 1,
-    NTP_UPDATE_TIME_INVALID = 2
-} NtpReturnValue;
-
-/**
- * @brief Own NTP Client library for Arduino with code from:
- * - https://github.com/arduino-libraries/NTPClient
- * - SPS&Technik - Projekt WordClock v1.02
- *
- */
-class NTPClientPlus
-{
-public:
-    NTPClientPlus(UDP &udp, const char *pool_server_name, int utcx, bool sw_change);
-    bool is_leap_year(unsigned int year);
-    bool update_sw_change();
-    int get_hours_12() const;
-    int get_hours_24() const;
-    int get_minutes() const;
-    int get_month(int dayOfYear);
-    int get_seconds() const;
-    int update_ntp();
-    long get_time_offset();
-    String get_formatted_date();
-    String get_formatted_time() const;
-    unsigned int get_day_of_week();
-    unsigned int get_year();
-    unsigned long get_epoch_time() const;
-    unsigned long get_secs_since_1900() const;
-    void calc_date();
-    void end();
-    void set_pool_server_name(const char *pool_server_name);
-    void set_time_offset(int time_offset);
-    void setup_ntp_client();
-
-private:
-    UDP *_udp;
-    bool _udp_setup = false;
-
-    bool _sw_change = 1;
-    const char *_pool_server_name = "pool.ntp.org"; // Default time server
-    int _utcx = 0;
-    IPAddress _pool_server_ip;
-    long _time_offset = 0;
-    unsigned int _port = NTP_DEFAULT_LOCAL_PORT;
-
-    unsigned long _update_interval = 60000; // In ms
-
-    unsigned int _date_day = 0;
-    unsigned int _date_month = 0;
-    unsigned int _date_year = 0;
-    unsigned int _day_of_week = 0;
-    unsigned long _current_epoc = 0; // In s
-    unsigned long _last_secs_since_1900 = 0;
-    unsigned long _last_update = 0;     // In ms
-    unsigned long _secs_since_1900 = 0; // seconds since 1. Januar 1900, 00:00:00
-
-    unsigned char _packet_buffer[NTP_PACKET_SIZE] = {0};
-    void send_ntp_packet();
-    void set_summertime(bool summertime);
-
-    static const unsigned long milliseconds_per_second = 1000;
-    static const unsigned long minutes_per_hour = 60;
-    static const unsigned long seconds_per_day = 86400;
-    static const unsigned long seconds_per_hour = 3600;
-    static const unsigned long seconds_per_minute = 60;
-
-    // number of days in months
-    unsigned int _days_per_month[13] = {0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
-};
-
-void wait(unsigned long time_ms);
-
-#endif /* NTPCLIENTPLUS_H */

include/own_font.h

@@ -1,66 +1,20 @@
-#ifndef OWNFONT_H
-#define OWNFONT_H
+#ifndef OWN_FONT_H
+#define OWN_FONT_H
 
-uint8_t numbers_font[10][5] = { {0b00000111,
-                                 0b00000101,
-                                 0b00000101,
-                                 0b00000101,
-                                 0b00000111},
-                                {0b00000001,
-                                 0b00000001,
-                                 0b00000001,
-                                 0b00000001,
-                                 0b00000001},
-                                {0b00000111,
-                                 0b00000001,
-                                 0b00000111,
-                                 0b00000100,
-                                 0b00000111},
-                                {0b00000111,
-                                 0b00000001,
-                                 0b00000111,
-                                 0b00000001,
-                                 0b00000111},
-                                {0b00000101,
-                                 0b00000101,
-                                 0b00000111,
-                                 0b00000001,
-                                 0b00000001},
-                                {0b00000111,
-                                 0b00000100,
-                                 0b00000111,
-                                 0b00000001,
-                                 0b00000111},
-                                {0b00000111,
-                                 0b00000100,
-                                 0b00000111,
-                                 0b00000101,
-                                 0b00000111},
-                                {0b00000111,
-                                 0b00000001,
-                                 0b00000001,
-                                 0b00000001,
-                                 0b00000001},
-                                {0b00000111,
-                                 0b00000101,
-                                 0b00000111,
-                                 0b00000101,
-                                 0b00000111},
-                                {0b00000111,
-                                 0b00000101,
-                                 0b00000111,
-                                 0b00000001,
-                                 0b00000111}};
+#include <Arduino.h>
 
-uint8_t chars_font[2][5] = {    {0b00000010,
-                                 0b00000010,
-                                 0b00000010,
-                                 0b00000010,
-                                 0b00000010},
-                                {0b00000111,
-                                 0b00000101,
-                                 0b00000111,
-                                 0b00000100,
-                                 0b00000100}};
+uint8_t numbers_font[10][5] = {{0b00000111, 0b00000101, 0b00000101, 0b00000101, 0b00000111},
+                               {0b00000001, 0b00000001, 0b00000001, 0b00000001, 0b00000001},
+                               {0b00000111, 0b00000001, 0b00000111, 0b00000100, 0b00000111},
+                               {0b00000111, 0b00000001, 0b00000111, 0b00000001, 0b00000111},
+                               {0b00000101, 0b00000101, 0b00000111, 0b00000001, 0b00000001},
+                               {0b00000111, 0b00000100, 0b00000111, 0b00000001, 0b00000111},
+                               {0b00000111, 0b00000100, 0b00000111, 0b00000101, 0b00000111},
+                               {0b00000111, 0b00000001, 0b00000001, 0b00000001, 0b00000001},
+                               {0b00000111, 0b00000101, 0b00000111, 0b00000101, 0b00000111},
+                               {0b00000111, 0b00000101, 0b00000111, 0b00000001, 0b00000111}};
 
-#endif  /* OWNFONT_H */
+uint8_t chars_font[2][5] = {{0b00000010, 0b00000010, 0b00000010, 0b00000010, 0b00000010},
+                            {0b00000111, 0b00000101, 0b00000111, 0b00000100, 0b00000100}};
+
+#endif /* OWN_FONT_H */

include/pong.h

@@ -79,8 +79,8 @@ private:
     UDPLogger *_logger;
     uint8_t _gameState = 0;
     uint8_t _numBots = 0;
-    uint8_t _playerMovement[PLAYER_AMOUNT] = {0};
-    Coords _paddles[PLAYER_AMOUNT][PADDLE_WIDTH] = {0};
+    uint8_t _playerMovement[PLAYER_AMOUNT];
+    Coords _paddles[PLAYER_AMOUNT][PADDLE_WIDTH];
     Coords _ball = {0, 0};
     Coords _ball_old = {0, 0};
     int _ballMovement[2] = {0, 0};

include/snake.h

@@ -20,7 +20,7 @@
 #ifdef DEBOUNCE_TIME
 #undef DEBOUNCE_TIME
 #endif
-#define DEBOUNCE_TIME 300 // in ms
+#define DEBOUNCE_TIME 250 // in ms
 
 #define X_MAX 11
 #define Y_MAX 11
@@ -76,7 +76,7 @@ private:
     uint8_t _userDirection = 0;
     uint8_t _gameState = 0;
     Coords _head = {0, 0};
-    Coords _tail[MAX_TAIL_LENGTH] = {0};
+    Coords _tail[MAX_TAIL_LENGTH];
     Coords _food = {0, 0};
     unsigned long _lastDrawUpdate = 0;
     unsigned long _lastButtonClick = 0;

include/time_manager.h

@@ -0,0 +1,80 @@
+#ifndef TIME_MANAGER_H
+#define TIME_MANAGER_H
+
+#include <Arduino.h>
+#include <time.h>
+#include "udp_logger.h"
+
+typedef enum
+{
+    TIME_UPDATE_FAILED = 0,
+    TIME_UPDATE_OK = 1,
+    TIME_UPDATE_PENDING = 2,
+} TimeUpdateState;
+
+typedef enum
+{
+    TM_INIT = 0,
+    TM_INITIAL_SYNC = 1,
+    TM_NORMAL = 2,
+    TM_SYNC_OVERDUE = 3,
+    TM_SYNC_TIMEOUT = 4,
+    TM_SETUP_FAILED = 5,
+} TimeManagerState;
+
+class TimeManager
+{
+#define NTP_MAX_UPDATE_TIME_US (5 * 1000 * 1000) // 5000ms max update time
+
+public:
+    // constructors
+    TimeManager();
+    TimeManager(const char *tz,
+                const char *ntp_server,
+                bool (*is_wifi_connected)(void),
+                uint32 ntp_max_offline_time_s,
+                UDPLogger *logger);
+
+    // init
+    void init();
+
+    // callback
+    void time_set_cb(void); // callback which is called when NTP time was set
+
+    // ntp methods
+    bool ntp_sync_successful(void) const; // was there a NTP sync once?
+    bool ntp_sync_overdue(void);          // function to check if NTP sync is overdue
+    bool ntp_sync_timeout(void);          // function to check if maximum time since last NTP sync has been reached
+    TimeUpdateState get_time();           // main time update method, called in loop
+
+    // getter for time values
+    bool isdst(void) const; // true if summertime (daylight saving time)
+    int day(void) const;
+    int hour(void) const;
+    int minute(void) const;
+    int month(void) const;
+    int year(void) const;
+    struct tm time_info(void) const;
+
+    // getter
+    TimeManagerState tm_state(void) const; // get current state
+
+    // logging
+    void log_time() const;                    // log _time_info
+    void log_time(struct tm time_info) const; // log argument time_info
+
+private:
+    void _set_up_ntp(void);           // set up NTP server
+    bool (*_is_wifi_connected)(void); // function to check if wifi is connected
+
+    const char *_ntp_server = "pool.ntp.org";           // ntp server address
+    const char *_tz;                                    // timezone
+    struct tm _time_info = {0, 0, 0, 0, 0, 0, 0, 0, 0}; // structure tm holds time information
+    time_t _now = 0;                                    // local time value
+    time_t _ntp_sync_timestamp_s = 0;                   // timestamp of last successful ntp sync
+    TimeManagerState _tm_state = TM_INIT;               // Main state
+    UDPLogger *_logger;                                 // logger instance
+    uint32 _ntp_max_offline_time_s;                     // maximum time in seconds which is considered ok since last NTP update
+};
+
+#endif /* TIME_MANAGER_H */

include/wordclock_constants.h

@@ -6,64 +6,74 @@
 // ----------------------------------------------------------------------------------
 // CONSTANTS
 // ----------------------------------------------------------------------------------
-#define AP_SSID "WordclockAP"                            // SSID name of Access Point
-#define NTP_SERVER_URL "de.pool.ntp.org"                 // NTP server address
+#define AP_SSID "WordclockAP"                                // SSID name of Access Point
+#define NTP_SERVER_URL "de.pool.ntp.org"                     // NTP server address
+#define MY_TZ "CET-1CEST-2,M3.5.0/02:00:00,M10.5.0/03:00:00" // Timezone
+
 #define HOSTNAME (String("wordclock"))                   // Local hostname
 #define LOGGER_MULTICAST_IP (IPAddress(230, 120, 10, 2)) // IP for UDP server
-#define LOGGER_MULTICAST_PORT 8123                       // Port for UDP server
-#define HTTP_PORT 80                                     // Standard HTTP port
+#define LOGGER_MULTICAST_PORT (8123)                     // Port for UDP server
+#define HTTP_PORT (80)                                   // Standard HTTP port
 
-// EEPROM layout
-typedef enum
-{
-    ADR_NM_START_H = 0,
-    ADR_NM_END_H = 4,
-    ADR_NM_START_M = 8,
-    ADR_NM_END_M = 12,
-    ADR_BRIGHTNESS = 16,
-    ADR_MC_RED = 20,
-    ADR_MC_GREEN = 22,
-    ADR_MC_BLUE = 24,
-    EEPROM_SIZE = 30
-} EepromLayout_en;
+// ESP8266 Pins
+#define FASTLED_PIN (14)  // pin to which the LEDs are attached
+#define BUTTON_PIN (5)    // pin to which the button is attached
 
-#define NEOPIXEL_PIN 14 // pin to which the NeoPixels are attached
-#define BUTTON_PIN 5    // pin to which the button is attached
+// Time limits
+#define HOUR_MAX (23)
+#define MINUTE_MAX (59)
+
+#define MINUTES_IN_HOUR (60)
+#define HOURS_IN_DAY (24)
 
 // Night mode
-#define NIGHTMODE_START_HR 23
-#define NIGHTMODE_START_MIN 0
-#define NIGHTMODE_END_HR 7
-#define NIGHTMODE_END_MIN 0
+#define NIGHTMODE_START_HR (23)
+#define NIGHTMODE_START_MIN (0)
+#define NIGHTMODE_END_HR (7)
+#define NIGHTMODE_END_MIN (0)
 
 // Timings in us
-#define PERIOD_ANIMATION_US (200 * 1000)             // 200ms
-#define PERIOD_HEARTBEAT_US (1 * 1000 * 1000)        // 1s
-#define PERIOD_MATRIX_UPDATE_US (100 * 1000)         // 100ms
-#define PERIOD_NIGHTMODE_CHECK_US (20 * 1000 * 1000) // 20s
-#define PERIOD_NTP_UPDATE_US (30 * 1000 * 1000)      // 30s
-#define PERIOD_PONG_US (10 * 1000)                   // 10ms
-#define PERIOD_SNAKE_US (50 * 1000)                  // 50ms
-#define PERIOD_STATE_CHANGE_US (10 * 1000 * 1000)    // 10s
-#define PERIOD_TETRIS_US (50 * 1000)                 // 50ms
-#define PERIOD_TIME_VISU_UPDATE_US (1 * 1000 * 1000) // 1s
-#define TIMEOUT_LEDDIRECT_US (5 * 1000 * 1000)       // 5s
+#define PERIOD_ANIMATION_US (200 * 1000)                   // 200ms
+#define PERIOD_CLOCK_UPDATE_US (1 * 1000 * 1000)           // Must be 1s! Do not change!
+#define PERIOD_HEARTBEAT_US (1 * 1000 * 1000)              // 1s
+#define PERIOD_MATRIX_UPDATE_US (33 * 1000)                // 33ms
+#define PERIOD_NIGHTMODE_CHECK_US (30 * 1000 * 1000)       // 30s
+#define PERIOD_TIME_UPDATE_US (1 * 1000 * 1000)            // 1000ms
+#define PERIOD_PONG_US (10 * 1000)                         // 10ms
+#define PERIOD_SNAKE_US (50 * 1000)                        // 50ms
+#define PERIOD_STATE_CHANGE_US (10 * 1000 * 1000)          // 10s
+#define PERIOD_TETRIS_US (50 * 1000)                       // 50ms
+#define TIMEOUT_LEDDIRECT_US (5 * 1000 * 1000)             // 5s
+#define PERIOD_BRIGHTNESS_UPDATE_US (5 * 60 * 1000 * 1000) // 300s
 
 #define SHORT_PRESS_US (100 * 1000)           // 100ms
 #define LONG_PRESS_US (2 * 1000 * 1000)       // 2s
 #define VERY_LONG_PRESS_US (10 * 1000 * 1000) // 10s
 
 // Current limit
-#define CURRENT_LIMIT_LED 2500 // limit the total current consumed by LEDs (mA)
+#define CURRENT_LIMIT_LED (2500) // limit the total current consumed by LEDs (mA)
+
+// Brightness ranges range: 0 - 255
+#define DEFAULT_BRIGHTNESS (40)
+#define MIN_BRIGHTNESS (10)
+#define MAX_BRIGHTNESS UINT8_MAX
 
 // LED smoothing
-#define DEFAULT_SMOOTHING_FACTOR 0.5
+#define DEFAULT_SMOOTHING_FACTOR (0.5f)
 
 // Number of colors in colors array
-#define NUM_COLORS 7
+#define NUM_COLORS (7)
+#define COLOR_ORDER GRB // WS2812B color order
 
 // LED matrix size
-#define MATRIX_WIDTH 11
-#define MATRIX_HEIGHT 11
+#define MATRIX_WIDTH (11)
+#define MATRIX_HEIGHT (11)
+#define NUM_MATRIX (MATRIX_WIDTH * (MATRIX_HEIGHT + 1))
+
+// NTP macros
+#define BUILD_YEAR (__DATE__ + 7)                   // Will expand to current year at compile time as string.
+#define NTP_MINIMUM_RX_YEAR (atoi(BUILD_YEAR) - 1)  // Will expand to current year minus one at compile time.
+#define NTP_START_YEAR (1900)                       // NTP minimum year is 1900
+#define NTP_MAX_OFFLINE_TIME_S (7 * 24 * 3600)      // Watchdog value, maximum offline time before a restart is triggered
 
 #endif /* WORDCLOCK_CONSTANTS_H */

include/wordclock_esp8266.h

@@ -2,14 +2,19 @@
 #define WORDCLOCK_ESP8266_H
 
 #include <Arduino.h>
+#include <stdlib.h> 
 #include <ESP8266WebServer.h>
 #include "led_matrix.h"
 #include "udp_logger.h"
 
-extern UDPLogger logger;
-extern LEDMatrix led_matrix;
-extern ESP8266WebServer webserver;
+#define RANGE_LIMIT(X, MIN, MAX) (((X) < (MIN)) ? (MIN) : (((X) > (MAX)) ? (MAX) : (X)))
+#define RANGE_LIMIT_SUB(X, MIN, MAX, SUB) (((X) < (MIN)) ? (SUB) : (((X) > (MAX)) ? (SUB) : (X)))
 
+#define EEPROM_SIZE (sizeof(EepromLayout_st) / sizeof(uint8_t))
+
+// ----------------------------------------------------------------------------------
+//                                     TYPEDEFS
+// ----------------------------------------------------------------------------------
 typedef struct
 {
     int start_hour;
@@ -18,6 +23,29 @@ typedef struct
     int end_min;
 } NightModeTimes_st;
 
+typedef struct
+{
+    uint8_t red;
+    uint8_t green;
+    uint8_t blue;
+    uint8_t alpha;  // note: unused
+} Color_st;
+
+typedef struct
+{
+    uint8_t static_brightness;   // user-controlled static brightness of LEDs
+    uint8_t dyn_brightness_min;  // user-controlled min brightness of LEDs
+    uint8_t dyn_brightness_max;  // user-controlled max brightness of LEDs
+    bool flg_dynamic_brightness; // flag if user wants to use daytime dynamic brightness
+} Brightness_st;
+
+typedef struct
+{
+    NightModeTimes_st night_mode_times;
+    Brightness_st brightness_values;
+    Color_st color_values;
+} EepromLayout_st;
+
 typedef enum
 {
     ST_CLOCK,
@@ -30,25 +58,32 @@ typedef enum
     NUM_STATES
 } ClockState_en;
 
-int EEPROM_read_address(int address);
+// ----------------------------------------------------------------------------------
+//                             FUNCTIONS DECLARATIONS
+// ----------------------------------------------------------------------------------
+bool check_wifi_status(void);
 String leading_zero2digit(int value);
+uint8_t calculate_dynamic_brightness(uint8_t min_brightness, uint8_t max_brightness, int hours, int minutes, bool summertime);
+uint8_t update_brightness(void);
 void check_night_mode(void);
-void check_wifi_status(void);
-void EEPROM_write_to_address(int address, int value);
+void cold_start_setup(void);
+void draw_main_color(void);
 void handle_button(void);
 void handle_command(void);
 void handle_current_state(void);
 void handle_data_request(void);
 void handle_led_direct(void);
-void load_main_color(void);
-void ntp_time_update(uint32 *last_ntp_update_us);
+void limit_value_ranges(void);
+void log_data(void);
 void on_state_entry(uint8_t state);
+void read_settings_from_EEPROM(void);
 void reset_wifi_credentials(void);
 void send_heartbeat(void);
+void set_dynamic_brightness(bool state);
 void set_main_color(uint8_t red, uint8_t green, uint8_t blue);
 void set_night_mode(bool on);
-void state_change(uint8_t newState);
+void state_change(ClockState_en new_state);
 void update_matrix(void);
-void update_state_machine(void);
+void write_settings_to_EEPROM(void);
 
 #endif /* WORDCLOCK_ESP8266_H */

platformio.ini

@@ -12,18 +12,21 @@
 default_envs = nodemcuv2
 
 [env]
-platform = espressif8266
+platform = espressif8266@2.6.3
 board = nodemcuv2
 framework = arduino
 lib_deps = 
-	adafruit/Adafruit NeoMatrix@^1.3.0
-	adafruit/Adafruit NeoPixel@^1.11.0
-	tzapu/WiFiManager@^0.16.0
+	densaugeo/base64@^1.4.0
+	fastled/FastLED@3.7.6
+	marcmerlin/FastLED NeoMatrix@^1.2
+	tzapu/WiFiManager@^2.0.17
+build_flags =
+	-DFASTLED_ESP8266_RAW_PIN_ORDER
 
 [env:nodemcuv2]
 monitor_speed = 115200
 monitor_filters = direct
 
-; [env:nodemcuv2_ota]
-; upload_protocol = espota
-; upload_port = http://ESP-DBA7AA
+[env:nodemcuv2_ota]
+upload_protocol = espota
+upload_port = wordclock.local

res/webserver/index.html

@@ -251,15 +251,23 @@
         }
 
         .show {
-            height: 200px;
+            height: 250px;
             transition: height 1s;
         }
 
+        .show-container {
+            display: inline
+        }
+
+        .hide-container {
+            display: none
+        }
+
         .number-container {
             display: flex;
             flex-direction: row;
             justify-content: left;
-            align-items: center;
+            /* align-items: center; */
             margin: 10px;
         }
 
@@ -279,9 +287,21 @@
     <h1 id="headline">WORDCLOCK 2.0</h1>
 
     <div id="settings-container" class="settings-container">
-        <div class="number-container">
-            <label for="brightness">Brightness:</label>
-            <input type="range" id="brightness" name="volume" min="10" max="255">
+        <div class="visible-brightness-container" id="static_brightness_container">
+            <div class="number-container">
+                <label for="static_brightness">Static brightness:</label>
+                <input type="range" id="static_brightness" name="static_brightness" min="10" max="255">
+            </div>
+        </div>
+        <div class="hidden-brightness-container" id="dyn_brightness_container">
+            <div class="number-container">
+                <label for="min_brightness">Min. brightness:</label>
+                <input type="range" id="min_brightness" name="min_brightness" min="10" max="255">
+            </div>
+            <div class="number-container">
+                <label for="max_brightness">Max. brightness:</label>
+                <input type="range" id="max_brightness" name="max_brightness" min="10" max="255">
+            </div>
         </div>
         <div class="number-container">
             <label for="nm_start" style="align-self: flex-start">Nightmode Start Time: </label>
@@ -332,14 +352,12 @@
             <input name="Nightmode" id="Nightmode" type="checkbox" class="toggle">
         </div>
     </div>
-    <div class="checkbox-container">
-        <label for="AutoChange" style="align-self: flex-start">Automatic mode change</label>
+    <div class="checkbox-container" onclick="toggleBrightnessOptions()">
+        <label for="DynBrightness" style="align-self: flex-start">Dynamic brightness</label>
         <div>
-            <input name="AutoChange" id="AutoChange" type="checkbox" class="toggle">
+            <input name="DynBrightness" id="DynBrightness" type="checkbox" class="toggle">
         </div>
-
     </div>
-
     <div class="main-container hidden" id="colorcontainer">
         <div class="verticalline">
         </div>
@@ -503,44 +521,47 @@
                 }
                 modebuttons[state].classList.add("active");
 
-                // set checkbox states
+                // set Nightmode checkbox states
                 var ckb_nightmode = document.querySelector('input[id="Nightmode"]');
-                if (myVar.nightMode == "1") {
-                    console.log("nightMode == 1");
+                if (myVar.night_mode == "1") {
+                    console.log("night_mode == 1");
                     ckb_nightmode.checked = true;
                 }
                 else {
-                    console.log("nightMode == 0");
+                    console.log("night_mode == 0");
                     ckb_nightmode.checked = false;
                 }
                 ckb_nightmode.addEventListener('change', () => {
                     if (ckb_nightmode.checked) {
-                        sendCommand("./cmd?nightmode=1");
+                        sendCommand("./cmd?night_mode=1");
                     } else {
-                        sendCommand("./cmd?nightmode=0");
+                        sendCommand("./cmd?night_mode=0");
                     }
                 });
 
-                var ckb_stateautochange = document.querySelector('input[id="AutoChange"]');
-                if (myVar.stateAutoChange == "1") {
-                    console.log("stateAutoChange == 1");
-                    ckb_stateautochange.checked = true;
+                // set DynBrightness checkbox states
+                var ckb_dynbrightness = document.querySelector('input[id="DynBrightness"]');
+                if (myVar.dyn_brightness == "1") {
+                    console.log("dyn_brightness == 1");
+                    ckb_dynbrightness.checked = true;
                 }
                 else {
-                    console.log("stateAutoChange == 0");
-                    ckb_stateautochange.checked = false;
+                    console.log("dyn_brightness == 0");
+                    ckb_dynbrightness.checked = false;
                 }
-                ckb_stateautochange.addEventListener('change', () => {
-                    if (ckb_stateautochange.checked) {
-                        sendCommand("./cmd?stateautochange=1");
+                ckb_dynbrightness.addEventListener('change', () => {
+                    if (ckb_dynbrightness.checked) {
+                        sendCommand("./cmd?dyn_brightness=1");
                     } else {
-                        sendCommand("./cmd?stateautochange=0");
+                        sendCommand("./cmd?dyn_brightness=0");
                     }
                 });
 
                 document.getElementById("nm_start").value = myVar.nightModeStart.replace("-", ":");
                 document.getElementById("nm_end").value = myVar.nightModeEnd.replace("-", ":");
-                document.getElementById("brightness").value = parseInt(myVar.brightness);
+                document.getElementById("static_brightness").value = parseInt(myVar.static_brightness);
+                document.getElementById("min_brightness").value = parseInt(myVar.min_brightness);
+                document.getElementById("max_brightness").value = parseInt(myVar.max_brightness);
 
                 updateDisplay(parseInt(myVar.modeid));
                 console.log(myVar);
@@ -566,7 +587,7 @@
             for (const element of maincontainer) {
                 element.classList.add("hidden");
             }
-            if (myVar != null && myVar.stateAutoChange == "0") {
+            if (myVar != null) {
                 switch (modeid) {
                     case 0: // clock
                         document.getElementById("colorcontainer").classList.remove("hidden");
@@ -603,7 +624,9 @@
         function saveSettings() {
             var nmStart = document.getElementById("nm_start");
             var nmEnd = document.getElementById("nm_end");
-            var brightnessElmt = document.getElementById("brightness");
+            var staticBrightnessElmt = document.getElementById("static_brightness");
+            var minBrightnessElmt = document.getElementById("min_brightness");
+            var maxBrightnessElmt = document.getElementById("max_brightness");
             var resetWifi = document.getElementById("reset_wifi");
 
             var cmdstr = "./cmd?setting=";
@@ -611,25 +634,68 @@
             cmdstr += "-";
             cmdstr += nmEnd.value.replace(":", "-");
             cmdstr += "-";
-            cmdstr += brightnessElmt.value;
+            cmdstr += staticBrightnessElmt.value;
             cmdstr += "-";
             cmdstr += Number(resetWifi.checked);
+            cmdstr += "-";
+            cmdstr += minBrightnessElmt.value;
+            cmdstr += "-";
+            cmdstr += maxBrightnessElmt.value;
 
             console.log(cmdstr);
             sendCommand(cmdstr);
             toggleSettings();
         }
 
+        function toggleBrightnessOptions() {
+            var static_b = document.getElementById("static_brightness_container");
+            var dyn_b = document.getElementById("dyn_brightness_container");
+
+            var ckb_dynbrightness = document.querySelector('input[id="DynBrightness"]');
+
+            if (ckb_dynbrightness.checked == true) {
+                static_b.classList.remove("show-container");
+                static_b.classList.add("hide-container")
+                dyn_b.classList.remove("hide-container");
+                dyn_b.classList.add("show-container")
+            }
+            else {
+                dyn_b.classList.remove("show-container");
+                dyn_b.classList.add("hide-container");
+                static_b.classList.remove("hide-container");
+                static_b.classList.add("show-container");
+            }
+        }
+
         function toggleSettings() {
             var container = document.getElementById("settings-container");
             if (container.classList.contains("show")) {
                 container.classList.remove("show");
             }
             else {
+                toggleBrightnessOptions();
                 container.classList.add("show");
             }
         }
 
+        function SetMax() {
+            var slider = document.getElementById("mySlider");
+
+            if ("max" in slider) {  // Google Chrome, Safari from version 5 and Opera
+                slider.min = 500;
+            } else {
+                // Safari before version 5
+                slider.setAttribute("max", 500);
+            }
+
+            OnSliderChanged(slider);
+        }
+
+        function OnSliderChanged(slider) {
+            var sliderValue = document.getElementById("sliderValue");
+            sliderValue.innerHTML = slider.value;
+        }
+
     </script>
 </body>
 

scripts/http_diagnosis.py

@@ -0,0 +1,13 @@
+import requests
+
+r = requests.get("http://wordclock.local/cmd?diag=reset_info")
+print(r.status_code)
+print(r.text)
+
+r = requests.get("http://wordclock.local/cmd?diag=sketch_info")
+print(r.status_code)
+print(r.text)
+
+r = requests.get("http://wordclock.local/cmd?diag=device_info")
+print(r.status_code)
+print(r.text)

scripts/multicastUDP_receiver.py

@@ -4,13 +4,15 @@ import queue
 import socket
 import struct
 import sys
+from pathlib import Path
 
+LOG_PATH = Path("C:/temp/wordclock_log.txt")
 
 def setup_logging():
     FORMAT = "%(asctime)s %(message)s"
     logging.basicConfig(format=FORMAT, level=logging.INFO)
     logger = logging.getLogger()
-    handler = logging.StreamHandler(sys.stdout)
+    handler = logging.FileHandler(LOG_PATH)
     handler.setLevel(logging.INFO)
     logger.addHandler(handler)
     return logger
@@ -42,33 +44,9 @@ mreq = struct.pack("4s4s", group, socket.inet_aton(get_ip_address()))
 sock.setsockopt(socket.IPPROTO_IP, socket.IP_ADD_MEMBERSHIP, mreq)
 
 logger.info("Ready")
-saveCounter = 0
-
-buffer = queue.Queue(20)
 
 # Receive/respond loop
 while True:
     data, address = sock.recvfrom(1024)
     data_str = data.decode("utf-8").strip()
     logger.info(data_str)
-    data_str = datetime.now().strftime("%b-%d-%Y_%H%M%S") + ": " + data_str
-    buffer.put(data_str)
-    if buffer.full():
-        buffer.get()
-
-    if "NTP-Update not successful" in data_str or "Start program" in data_str:
-        f = open("log.txt", "a")
-        while not buffer.empty():
-            f.write(buffer.get())
-            f.write("\n")
-        f.close()
-        saveCounter = 20
-
-    if saveCounter > 0:
-        f = open("log.txt", "a")
-        f.write(data_str)
-        f.write("\n")
-        if saveCounter == 1:
-            f.write("\n")
-        f.close()
-        saveCounter -= 1

src/connectivity/diagnosis.cpp

@@ -0,0 +1,93 @@
+#include "diagnosis.h"
+
+Diagnosis::Diagnosis()
+{
+    _logger = nullptr;
+}
+
+Diagnosis::Diagnosis(UDPLogger *logger, LEDMatrix *matrix) // constructor
+{
+    _logger = logger;
+    _matrix = matrix;
+}
+
+String Diagnosis::handle_command(const String &command)
+{
+    if (command == "device_info")
+    {
+        return print_device_info();
+    }
+    else if (command == "sketch_info")
+    {
+        return print_sketch_info();
+    }
+    else if (command == "reset_info")
+    {
+        return print_last_reset_details();
+    }
+    else if (command == "matrix_fps")
+    {
+        return print_matrix_fps();
+    }
+    else
+    {
+        // Handle unknown command
+        String unknown_command = "Diagnosis: Unknown command!\n";
+        print(unknown_command);
+        return unknown_command;
+    }
+}
+
+String Diagnosis::print_device_info()
+{
+    // Retrieve and print device information
+    String device_info = "Device Information:\n";
+    device_info += "Chip ID: " + String(ESP.getChipId()) + "\n";
+    device_info += "Flash Chip ID: " + String(ESP.getFlashChipId()) + "\n";
+    device_info += "Flash Chip Size: " + String(ESP.getFlashChipSize()) + " bytes\n";
+    device_info += "Free Heap Size: " + String(ESP.getFreeHeap()) + " bytes\n";
+    device_info += "Free Sketch Space: " + String(ESP.getFreeSketchSpace()) + " bytes\n";
+    device_info += "SDK Version: " + String(ESP.getSdkVersion()) + "\n";
+    print(device_info);
+    return device_info;
+}
+
+String Diagnosis::print_sketch_info()
+{
+    // Retrieve and print sketch information
+    String sketch_info = "Sketch Information:\n";
+    sketch_info += "Sketch Size: " + String(ESP.getSketchSize()) + " bytes\n";
+    sketch_info += "Sketch MD5: " + String(ESP.getSketchMD5()) + "\n";
+    print(sketch_info);
+    return sketch_info;
+}
+
+String Diagnosis::print_last_reset_details()
+{
+    // Retrieve and print last reset details
+    String reset_info = "Last Reset Details:\n";
+    reset_info += "Reset Reason: " + String(ESP.getResetReason()) + "\n";
+    reset_info += "Reset Info: " + String(ESP.getResetInfo()) + "\n";
+    print(reset_info);
+    return reset_info;
+}
+
+String Diagnosis::print_matrix_fps()
+{
+    // Retrieve and print matrix FPS
+    String matrix_fps = "Matrix FPS: " + String(_matrix->get_fps()) + "\n";
+    print(matrix_fps);
+    return matrix_fps;
+}
+
+void Diagnosis::print(const String &s)
+{
+    if (_logger != nullptr)
+    {
+        _logger->log_string(s);
+    }
+    else
+    {
+        Serial.println(s);
+    }
+}

src/connectivity/ntp_client_plus.cpp

@@ -1,658 +0,0 @@
-#include <Arduino.h>
-#include "ntp_client_plus.h"
-
-/**
- * @brief Construct a new NTPClientPlus::NTPClientPlus object
- *
- * @param udp   UDP client
- * @param pool_server_name    time server name
- * @param utcx  UTC offset (in 1h)
- * @param sw_change should summer/winter time be considered
- */
-NTPClientPlus::NTPClientPlus(UDP &udp, const char *pool_server_name, int utcx, bool sw_change)
-{
-    this->_udp = &udp;
-    this->_utcx = utcx;
-    this->_time_offset = this->seconds_per_hour * this->_utcx;
-    this->_pool_server_name = pool_server_name;
-    this->_sw_change = sw_change;
-}
-
-/**
- * @brief Starts the underlying UDP client, get first NTP timestamp and calc date
- *
- */
-void NTPClientPlus::setup_ntp_client()
-{
-    this->_udp->begin(this->_port);
-    this->_udp_setup = true;
-    this->update_ntp();
-    this->calc_date();
-}
-
-/**
- * @brief Get new update from NTP
- *
- * @return NTP_UPDATE_TIMEOUT         timeout after 500 ms
- * @return NTP_UPDATE_SUCCESS         after successful update
- * @return NTP_UPDATE_DIFFTOOHIGH     too much difference to previous received time (try again)
- * @return NTP_UPDATE_TIME_INVALID    time value is invalid
- */
-int NTPClientPlus::update_ntp()
-{
-    // flush any existing packets
-    while (this->_udp->parsePacket() != 0)
-    {
-        this->_udp->flush();
-    }
-
-    this->send_ntp_packet();
-
-    // Wait till data is there or timeout...
-    uint8_t conn_tries = 0;
-    int received_bytes = 0;
-
-    while ((received_bytes == 0) && (conn_tries++ <= MAX_NTP_CONN_TRIES))
-    {
-        received_bytes = this->_udp->parsePacket();
-        wait(NTP_RECEIVE_WAIT_TIME_MS);
-    }
-
-    if (conn_tries >= MAX_NTP_CONN_TRIES)
-    {
-        return NTP_UPDATE_TIMEOUT;
-    }
-
-    this->_udp->read(this->_packet_buffer, NTP_PACKET_SIZE);
-
-    unsigned long high_word = word(this->_packet_buffer[40], this->_packet_buffer[41]);
-    unsigned long low_word = word(this->_packet_buffer[42], this->_packet_buffer[43]);
-    // combine the four bytes (two words) into a long integer
-    // this is NTP time (seconds since Jan 1 1900):
-    unsigned long temp_secs_since_1900 = high_word << 16 | low_word;
-
-    if (temp_secs_since_1900 < UNIX_TIMESTAMP_1900) // NTP time is not valid
-    {
-        return NTP_UPDATE_TIME_INVALID;
-    }
-
-    // check if time off last ntp update is roughly in the same range: 100sec apart (validation check)
-    if (this->_last_secs_since_1900 == 0 || temp_secs_since_1900 - this->_last_secs_since_1900 < 100000)
-    {
-        // Only update time then account for delay in reading the time
-        this->_last_update = (system_get_time() / 1000) - (NTP_RECEIVE_WAIT_TIME_MS * (conn_tries + 1));
-        this->_secs_since_1900 = temp_secs_since_1900;
-        this->_current_epoc = this->_secs_since_1900 - UNIX_TIMESTAMP_1900;
-
-        // Remember time of last update
-        this->_last_secs_since_1900 = temp_secs_since_1900;
-        return NTP_UPDATE_SUCCESS; // return 0 after successful update
-    }
-    else
-    {
-        // Remember time of last update
-        this->_last_secs_since_1900 = temp_secs_since_1900;
-        return NTP_UPDATE_DIFFTOOHIGH;
-    }
-}
-
-/**
- * @brief Stops the underlying UDP client
- *
- */
-void NTPClientPlus::end()
-{
-    this->_udp->stop();
-    this->_udp_setup = false;
-}
-
-/**
- * @brief Setter TimeOffset
- *
- * @param time_offset offset from UTC in seconds
- */
-void NTPClientPlus::set_time_offset(int time_offset)
-{
-    this->_time_offset = time_offset;
-}
-
-long NTPClientPlus::get_time_offset()
-{
-    return this->_time_offset;
-}
-
-/**
- * @brief Set time server name
- *
- * @param pool_server_name
- */
-void NTPClientPlus::set_pool_server_name(const char *pool_server_name)
-{
-    this->_pool_server_name = pool_server_name;
-}
-
-/**
- * @brief Calc seconds since 1. Jan. 1900
- *
- * @return unsigned long seconds since 1. Jan. 1900
- */
-unsigned long NTPClientPlus::get_secs_since_1900() const
-{
-    return this->_time_offset +                                        // User offset
-           this->_secs_since_1900 +                                    // seconds returned by the NTP server
-           (((system_get_time() / 1000) - this->_last_update) / 1000); // Time since last update
-}
-
-/**
- * @brief Get UNIX Epoch time since 1. Jan. 1970
- *
- * @return unsigned long UNIX Epoch time since 1. Jan. 1970 in seconds
- */
-unsigned long NTPClientPlus::get_epoch_time() const
-{
-    return this->get_secs_since_1900() - UNIX_TIMESTAMP_1900;
-}
-
-/**
- * @brief Get current hours in 24h format
- *
- * @return int
- */
-int NTPClientPlus::get_hours_24() const
-{
-    int hours = ((this->get_epoch_time() % 86400L) / 3600);
-    return hours;
-}
-
-/**
- * @brief Get current hours in 12h format
- *
- * @return int
- */
-int NTPClientPlus::get_hours_12() const
-{
-    return this->get_hours_24() % 12;
-}
-
-/**
- * @brief Get current minutes
- *
- * @return int
- */
-int NTPClientPlus::get_minutes() const
-{
-    return ((this->get_epoch_time() % 3600) / 60);
-}
-
-/**
- * @brief Get current seconds
- *
- * @return int
- */
-int NTPClientPlus::get_seconds() const
-{
-    return this->get_epoch_time() % 60;
-}
-
-/**
- * @brief
- *
- * @return String time formatted like `hh:mm:ss`
- */
-String NTPClientPlus::get_formatted_time() const
-{
-    unsigned long raw_time = this->get_epoch_time();
-    unsigned long hours = (raw_time % 86400L) / 3600;
-    String hours_str = hours < 10 ? "0" + String(hours) : String(hours);
-
-    unsigned long minutes = (raw_time % 3600) / 60;
-    String minute_str = minutes < 10 ? "0" + String(minutes) : String(minutes);
-
-    unsigned long seconds = raw_time % 60;
-    String second_str = seconds < 10 ? "0" + String(seconds) : String(seconds);
-
-    return hours_str + ":" + minute_str + ":" + second_str;
-}
-
-/**
- * @brief
- *
- * @return String date formatted like `dd.mm.yyyy`
- */
-String NTPClientPlus::get_formatted_date()
-{
-    this->calc_date();
-    unsigned int dateDay = this->_date_day;
-    unsigned int dateMonth = this->_date_month;
-    unsigned int dateYear = this->_date_year;
-
-    String dayStr = dateDay < 10 ? "0" + String(dateDay) : String(dateDay);
-    String monthStr = dateMonth < 10 ? "0" + String(dateMonth) : String(dateMonth);
-    String yearStr = dateYear < 10 ? "0" + String(dateYear) : String(dateYear);
-
-    return dayStr + "." + monthStr + "." + yearStr;
-}
-
-/**
- * @brief Calc date from seconds since 1900
- *
- */
-void NTPClientPlus::calc_date()
-{
-    // get days since 1900
-    unsigned long days1900 = this->get_secs_since_1900() / seconds_per_day;
-
-    // calc current year
-    this->_date_year = this->get_year();
-
-    // calc how many leap days since 1.Jan 1900
-    int leap_days = 0;
-    for (unsigned int i = 1900; i < this->_date_year; i++)
-    {
-        // check if leap year
-        if (this->is_leap_year(i))
-        {
-            leap_days++;
-        }
-    }
-    leap_days = leap_days - 1;
-
-    // check if current year is leap year
-    if (this->is_leap_year(this->_date_year))
-    {
-        _days_per_month[2] = 29;
-    }
-    else
-    {
-        _days_per_month[2] = 28;
-    }
-
-    unsigned int day_of_year = (days1900 - ((this->_date_year - 1900) * 365) - leap_days);
-
-    // calc current month
-    this->_date_month = this->get_month(day_of_year);
-
-    this->_date_day = 0;
-
-    // calc day of month
-    for (unsigned int i = 0; i < this->_date_month; i++)
-    {
-        this->_date_day = this->_date_day + _days_per_month[i];
-    }
-    this->_date_day = day_of_year - this->_date_day;
-
-    // calc day of week:
-    // Monday = 1, Tuesday = 2, Wednesday = 3, Thursday = 4, Friday = 5, Saturday = 6, Sunday = 7
-    // 1. Januar 1900 was a monday
-    this->_day_of_week = 1;
-
-    for (unsigned int i = 0; i < days1900; i++)
-    {
-        if (this->_day_of_week < 7)
-        {
-            this->_day_of_week = this->_day_of_week + 1;
-        }
-        else
-        {
-            this->_day_of_week = 1;
-        }
-    }
-
-    // End: Calc date (dateDay, dateMonth, dateYear)
-
-    // calc if summer time active
-
-    this->update_sw_change();
-}
-
-/**
- * @brief Getter for day of the week
- *
- * @return unsigned int
- */
-unsigned int NTPClientPlus::get_day_of_week()
-{
-    return this->_day_of_week;
-}
-
-/**
- * @brief Function to calc current year
- *
- * @return unsigned int
- */
-unsigned int NTPClientPlus::get_year()
-{
-    unsigned long secs_since_1900 = this->get_secs_since_1900();
-
-    // NTP starts at 1. Jan 1900
-    unsigned int result = 1900;
-    unsigned int days_in_year = 0;
-    unsigned int days = 0;
-    unsigned int days_since_1900 = 0;
-
-    unsigned int for_i = 0;
-    bool leap_year = false;
-
-    days_since_1900 = secs_since_1900 / this->seconds_per_day;
-
-    for (for_i = 0; for_i < days_since_1900; for_i++)
-    {
-        leap_year = this->is_leap_year(result);
-
-        if (leap_year)
-        {
-            days_in_year = 366;
-        }
-        else
-        {
-            days_in_year = 365;
-        }
-        days++;
-
-        if (days >= days_in_year)
-        {
-            result++;
-            days = 0;
-        }
-    }
-
-    return result;
-}
-
-/**
- * @brief Function to check if given year is leap year
- *
- * @param year
- * @return true
- * @return false
- */
-bool NTPClientPlus::is_leap_year(unsigned int year)
-{
-
-    bool result = false;
-
-    // check for leap year
-    if ((year % 4) == 0)
-    {
-        result = true;
-
-        if ((year % 100) == 0)
-        {
-            result = false;
-
-            if ((year % 400) == 0)
-            {
-                result = true;
-            }
-        }
-    }
-    else
-    {
-        result = false;
-    }
-
-    return result;
-}
-
-/**
- * @brief Get Month of given day of year
- *
- * @param day_of_year
- * @return int
- */
-int NTPClientPlus::get_month(int day_of_year)
-{
-    bool leap_year = this->is_leap_year(this->get_year());
-
-    // Month beginnings
-    int month_min[13] = {0, 1, 32, 60, 91, 121, 152, 182, 213, 244, 274, 305, 335};
-    // Month endings
-    int month_max[13] = {0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334, 365};
-    int month = 0;
-    int y = 0;
-
-    // Calculation of the beginning and end of each month in the leap year
-    if (leap_year == true)
-    {
-        for (y = 3; y < 13; y++)
-        {
-            ++month_min[y];
-        }
-
-        for (y = 2; y < 13; y++)
-        {
-            ++month_max[y];
-        }
-    }
-
-    // January
-    if (day_of_year >= month_min[1] && day_of_year <= month_max[1])
-    {
-        month = 1;
-    }
-
-    // February
-    if (day_of_year >= month_min[2] && day_of_year <= month_max[2])
-    {
-        month = 2;
-    }
-
-    // March
-    if (day_of_year >= month_min[3] && day_of_year <= month_max[3])
-    {
-        month = 3;
-    }
-
-    // April
-    if (day_of_year >= month_min[4] && day_of_year <= month_max[4])
-    {
-        month = 4;
-    }
-
-    // May
-    if (day_of_year >= month_min[5] && day_of_year <= month_max[5])
-    {
-        month = 5;
-    }
-
-    // June
-    if (day_of_year >= month_min[6] && day_of_year <= month_max[6])
-    {
-        month = 6;
-    }
-
-    // July
-    if (day_of_year >= month_min[7] && day_of_year <= month_max[7])
-    {
-        month = 7;
-    }
-
-    // August
-    if (day_of_year >= month_min[8] && day_of_year <= month_max[8])
-    {
-        month = 8;
-    }
-
-    // September
-    if (day_of_year >= month_min[9] && day_of_year <= month_max[9])
-    {
-        month = 9;
-    }
-
-    // October
-    if (day_of_year >= month_min[10] && day_of_year <= month_max[10])
-    {
-        month = 10;
-    }
-
-    // November
-    if (day_of_year >= month_min[11] && day_of_year <= month_max[11])
-    {
-        month = 11;
-    }
-
-    // December
-    if (day_of_year >= month_min[12] && day_of_year <= month_max[12])
-    {
-        month = 12;
-    }
-
-    return month;
-}
-
-/**
- * @brief (private) Send NTP Packet to NTP server
- *
- */
-void NTPClientPlus::send_ntp_packet()
-{
-    // set all bytes in the buffer to 0
-    memset(this->_packet_buffer, 0, NTP_PACKET_SIZE);
-    // Initialize values needed to form NTP request
-    this->_packet_buffer[0] = 0b11100011; // LI, Version, Mode
-    this->_packet_buffer[1] = 0;          // Stratum, or type of clock
-    this->_packet_buffer[2] = 6;          // Polling Interval
-    this->_packet_buffer[3] = 0xEC;       // Peer Clock Precision
-    // 8 bytes of zero for Root Delay & Root Dispersion
-    this->_packet_buffer[12] = 49;
-    this->_packet_buffer[13] = 0x4E;
-    this->_packet_buffer[14] = 49;
-    this->_packet_buffer[15] = 52;
-
-    // all NTP fields have been given values, now
-    // you can send a packet requesting a timestamp:
-    if (this->_pool_server_name)
-    {
-        this->_udp->beginPacket(this->_pool_server_name, 123);
-    }
-    else
-    {
-        this->_udp->beginPacket(this->_pool_server_ip, 123);
-    }
-    this->_udp->write(this->_packet_buffer, NTP_PACKET_SIZE);
-    this->_udp->endPacket();
-}
-
-/**
- * @brief (private) Set time offset accordance to summer time
- *
- * @param summertime
- */
-void NTPClientPlus::set_summertime(bool summertime)
-{
-    if (summertime)
-    {
-        this->_time_offset = this->seconds_per_hour * (this->_utcx + 1);
-    }
-    else
-    {
-        this->_time_offset = this->seconds_per_hour * (this->_utcx);
-    }
-}
-
-/**
- * @brief (private) Update Summer/Winter time change
- *
- * @returns bool summertime active
- */
-bool NTPClientPlus::update_sw_change()
-{
-    unsigned int day_of_week = this->_day_of_week;
-    unsigned int date_day = this->_date_day;
-    unsigned int date_month = this->_date_month;
-
-    bool summertime_active = false;
-
-    if (this->_sw_change)
-    {
-        // Start: Set summer-/ winter time
-        // current month is march
-        if (date_month == 3)
-        {
-
-            // it is last week in march
-            if ((this->_days_per_month[3] - date_day) < 7)
-            {
-                // Example year 2020: March 31 days; Restart March 26, 2020 (Thursday = weekday = 4); 5 days remaining; Last Sunday March 29, 2020
-                // Calculation: 31 - 26 = 5; 5 + 4 = 9;
-                // Result: Last day in March is a Tuesday. There folfalses another Sunday in October => set winter time
-
-                // Example year 2021: March 31 days; Restart March 30, 2021 (Tuesday = weekday = 2); 1 days remaining; Last Sunday March 28, 2021
-                // Calculation: 31 - 30 = 1; 1 + 2 = 3;
-                // Result: Last day in March is a Wednesday. Changeover to summer time already done => set summer time
-
-                // There folfalses within the last week in March one more Sunday => set winter time
-                if (((this->_days_per_month[3] - date_day) + day_of_week) >= 7)
-                {
-                    this->set_summertime(0);
-                    summertime_active = false;
-                }
-                else // last sunday in march already over -> summer time
-                {
-                    this->set_summertime(1);
-                    summertime_active = true;
-                }
-            }
-            else // restart in first three weeks of march -> winter time
-            {
-                this->set_summertime(0);
-                summertime_active = false;
-            }
-        }
-
-        // current month is october
-        else if (date_month == 10)
-        {
-            // restart last week of october
-            if ((this->_days_per_month[10] - date_day) < 7)
-            {
-
-                // Example year 2020: October 31 days; restart October 26, 2020 (Monday = weekday = 1); 5 days remaining; last Sunday October 25, 2020
-                // Calculation: 31 - 26 = 5; 5 + 1 = 6;
-                // Result: Last day in October is a Saturday. Changeover to winter time already done => set winter time
-
-                // Example year 2021: October 31 days; Restart 26. October 2021 (Tuesday = weekday = 2); 5 days remaining; Last Sunday 31. October 2021
-                // Calculation: 31 - 26 = 5; 5 + 2 = 7;
-                // Result: Last day in October is a Sunday. There folfalses another Sunday in October => set summer time
-
-                // There folfalses within the last week in October one more Sunday => summer time
-                if (((this->_days_per_month[10] - date_day) + day_of_week) >= 7)
-                {
-                    this->set_summertime(1);
-                    summertime_active = true;
-                }
-                else // last sunday in october already over -> winter time
-                {
-                    this->set_summertime(0);
-                    summertime_active = false;
-                }
-            }
-            else // restart in first three weeks of october -> summer time
-            {
-                this->set_summertime(1);
-                summertime_active = true;
-            }
-        }
-        else if (date_month > 3 && date_month < 10) // restart in summer time
-        {
-            this->set_summertime(1);
-            summertime_active = true;
-        }
-        else if (date_month < 3 || date_month > 10) // restart in winter time
-        {
-            this->set_summertime(0);
-            summertime_active = false;
-        }
-    }
-
-    return summertime_active;
-}
-
-void wait(unsigned long time_ms)
-{
-    unsigned long start = (system_get_time() / 1000); // in ms
-    while (((system_get_time() / 1000) - start) < time_ms)
-    {
-        yield();
-    };
-}

src/connectivity/time_manager.cpp

@@ -0,0 +1,214 @@
+#include "time_manager.h"
+#include "wordclock_constants.h"
+#include <coredecls.h> // required for settimeofday_cb()
+#include <sntp.h>
+#include <time.h>
+
+extern UDPLogger logger; // logging instance
+
+// ----------------------------------------------------------------------------------
+//                                        Class
+// ----------------------------------------------------------------------------------
+TimeManager::TimeManager(){};
+TimeManager::TimeManager(const char *tz,
+                         const char *ntp_server,
+                         bool (*is_wifi_connected)(void),
+                         uint32 ntp_max_offline_time_s,
+                         UDPLogger *logger)
+{
+    _tz = tz;
+    _ntp_server = ntp_server;
+    _is_wifi_connected = is_wifi_connected;
+    _ntp_max_offline_time_s = ntp_max_offline_time_s;
+    _logger = logger;
+}
+
+void TimeManager::init()
+{
+    // Set up NTP server once
+    _set_up_ntp();
+
+    if (_tm_state == TM_INITIAL_SYNC)
+    {
+        // force sntp reinit now
+        sntp_stop();
+        sntp_init();
+    }
+    settimeofday_cb([&]() { time_set_cb(); });
+}
+
+bool TimeManager::ntp_sync_successful(void) const
+{
+    return (_now > 1716913300); // UTC timestamp in the past (28.05.2024)
+}
+
+bool TimeManager::ntp_sync_overdue(void)
+{
+    if (!ntp_sync_successful())
+    {
+        return false;
+    }
+
+    bool retval = false;
+    // after the ntp sync update delay has been reached six times, the sync is considered overdue
+    if (_now >= (_ntp_sync_timestamp_s + (time_t)(6 * (sntp_update_delay_MS_rfc_not_less_than_15000() / 1000))))
+    {
+        _tm_state = TM_SYNC_OVERDUE;
+        retval = true;
+    }
+    return retval;
+}
+
+bool TimeManager::ntp_sync_timeout(void)
+{
+    if (!ntp_sync_successful())
+    {
+        return false;
+    }
+
+    bool retval = false;
+    // after the maxmimum offline time has been reached, the sync is considered timed out
+    if (_now >= (_ntp_sync_timestamp_s + (time_t)_ntp_max_offline_time_s))
+    {
+        _tm_state = TM_SYNC_TIMEOUT;
+        retval = true;
+    }
+    return retval;
+}
+
+/**
+ * @brief NTP time update, should be called in loop().
+ *
+ * @retval true if last update was successful
+ */
+TimeUpdateState TimeManager::get_time()
+{
+    TimeUpdateState retval = TIME_UPDATE_PENDING; // NTP time update
+    uint32 timestamp_us = system_get_time();      // NTP update start time
+    struct tm time_info;                          // local NTP time info
+
+    do
+    {
+        _now = time(nullptr);                 // update time
+        (void)localtime_r(&_now, &time_info); // convert time
+        yield();                              // since this loop could take up to NTP_MAX_UPDATE_TIME_US
+
+    } while (((system_get_time() - timestamp_us) <= NTP_MAX_UPDATE_TIME_US) && (time_info.tm_year < (NTP_MINIMUM_RX_YEAR - NTP_START_YEAR)));
+
+    retval = (time_info.tm_year <= (NTP_MINIMUM_RX_YEAR - NTP_START_YEAR)) ? TIME_UPDATE_FAILED : TIME_UPDATE_OK; // sanity check
+
+    if (retval == TIME_UPDATE_OK)
+    {
+        _time_info = time_info; // take over time_info to member variable
+        _tm_state = TM_NORMAL;
+    }
+    else
+    {
+        logger.log_string("NTP-Update was not successful. Retrying in " + String(sntp_update_delay_MS_rfc_not_less_than_15000()) + "ms.");
+    }
+
+    return retval;
+}
+
+bool TimeManager::isdst(void) const
+{
+    return (_time_info.tm_isdst > 0);
+}
+
+int TimeManager::day(void) const
+{
+    return (_time_info.tm_mday + 1); // add 1 to get actual day
+}
+
+int TimeManager::hour(void) const
+{
+    return _time_info.tm_hour;
+}
+
+int TimeManager::minute(void) const
+{
+    return _time_info.tm_min;
+}
+
+int TimeManager::month(void) const
+{
+    return (_time_info.tm_mon + 1); // add 1 to get actual month
+}
+
+int TimeManager::year(void) const
+{
+    return _time_info.tm_year + NTP_START_YEAR; // add start year to get actual year
+}
+
+struct tm TimeManager::time_info(void) const
+{
+    return _time_info;
+}
+
+TimeManagerState TimeManager::tm_state(void) const
+{
+    return _tm_state;
+}
+
+/**
+ * @brief Log time_info as string.
+ *
+ * @param local_time
+ */
+void TimeManager::log_time() const
+{
+    log_time(_time_info);
+}
+
+/**
+ * @brief Log time_info as string.
+ *
+ * @param local_time
+ */
+void TimeManager::log_time(struct tm time_info) const
+{
+    char strftime_buf[64]; // Time string buffer
+    strftime(strftime_buf, sizeof(strftime_buf), "%c", &time_info);
+    logger.log_string(String(strftime_buf));
+}
+
+/**
+ * @brief Sets up the NTP server config
+ *
+ */
+void TimeManager::_set_up_ntp(void)
+{
+    if ((_tz != nullptr) && (_ntp_server != nullptr))
+    {
+        // set up NTP server and timezone at init
+        configTime(_tz, _ntp_server);
+        _tm_state = TM_INITIAL_SYNC;
+        logger.log_string(String("NTP configuration was set up successfully!"));
+    }
+    else
+    {
+        _tm_state = TM_SETUP_FAILED;
+        logger.log_string(String("ERROR: Timezone and/or NTP-Server were not given correctly!"));
+    }
+}
+
+void TimeManager::time_set_cb(void)
+{
+    if ((*_is_wifi_connected)())
+    {
+        if (get_time() == TIME_UPDATE_OK)
+        {
+            _ntp_sync_timestamp_s = _now;
+            logger.log_string("NTP successfully synced at...");
+            log_time();
+        }
+        else
+        {
+            logger.log_string("NTP sync failed!");
+        }
+    }
+    else
+    {
+        logger.log_string("NTP sync failed, WiFi is not connected!");
+    }
+}

src/connectivity/udp_logger.cpp

@@ -1,8 +1,6 @@
 #include "udp_logger.h"
 
-UDPLogger::UDPLogger()
-{
-}
+UDPLogger::UDPLogger() {}
 
 UDPLogger::UDPLogger(IPAddress interface_addr, IPAddress multicast_addr, int port, String name)
 {
@@ -20,13 +18,16 @@ void UDPLogger::set_name(String name)
 
 void UDPLogger::log_string(String message)
 {
-    // wait 5 milliseconds if last send was less than 5 milliseconds before
-    if ((system_get_time() / 1000) < (_lastSend + 5))
+    // wait 5 milliseconds if last send was less than 10 milliseconds before
+    if ((system_get_time() / 1000) < (_lastSend + 10))
     {
-        delay(5);
+        delay(10);
     }
     message = _name + ": " + message;
+#ifdef SERIAL_DEBUG
     Serial.println(message);
+    delay(10);
+#endif /* SERIAL_DEBUG */
     _udp.beginPacketMulticast(_multicastAddr, _port, _interfaceAddr);
     message.toCharArray(_packetBuffer, 100);
     _udp.print(_packetBuffer);

src/games/snake.cpp

@@ -26,7 +26,7 @@ Snake::Snake()
  * @param myledmatrix pointer to LEDMatrix object, need to provide gridAddPixel(x, y, col), gridFlush()
  * @param mylogger pointer to UDPLogger object, need to provide a function logString(message)
  */
-Snake::Snake(LEDMatrix * matrix, UDPLogger * logger)
+Snake::Snake(LEDMatrix *matrix, UDPLogger *logger)
 {
     _logger = logger;
     _ledmatrix = matrix;
@@ -58,7 +58,7 @@ void Snake::loopCycle()
  */
 void Snake::ctrlUp()
 {
-    if ((system_get_time() / 1000) > _lastButtonClick + DEBOUNCE_TIME && _gameState == GAME_STATE_RUNNING)
+    if (((system_get_time() / 1000) > (_lastButtonClick + DEBOUNCE_TIME)) && (_gameState == GAME_STATE_RUNNING) && (_userDirection != DIRECTION_UP))
     {
         _logger->log_string("Snake: UP");
         _userDirection = DIRECTION_DOWN; // need to swap direction as field is rotated 180deg
@@ -72,7 +72,7 @@ void Snake::ctrlUp()
  */
 void Snake::ctrlDown()
 {
-    if ((system_get_time() / 1000) > _lastButtonClick + DEBOUNCE_TIME && _gameState == GAME_STATE_RUNNING)
+    if (((system_get_time() / 1000) > (_lastButtonClick + DEBOUNCE_TIME)) && (_gameState == GAME_STATE_RUNNING) && (_userDirection != DIRECTION_DOWN))
     {
         _logger->log_string("Snake: DOWN");
         _userDirection = DIRECTION_UP; // need to swap direction as field is rotated 180deg
@@ -86,7 +86,7 @@ void Snake::ctrlDown()
  */
 void Snake::ctrlRight()
 {
-    if ((system_get_time() / 1000) > _lastButtonClick + DEBOUNCE_TIME && _gameState == GAME_STATE_RUNNING)
+    if (((system_get_time() / 1000) > (_lastButtonClick + DEBOUNCE_TIME)) && (_gameState == GAME_STATE_RUNNING) && (_userDirection != DIRECTION_RIGHT))
     {
         _logger->log_string("Snake: RIGHT");
         _userDirection = DIRECTION_LEFT; // need to swap direction as field is rotated 180deg
@@ -100,7 +100,7 @@ void Snake::ctrlRight()
  */
 void Snake::ctrlLeft()
 {
-    if ((system_get_time() / 1000) > _lastButtonClick + DEBOUNCE_TIME && _gameState == GAME_STATE_RUNNING)
+    if (((system_get_time() / 1000) > (_lastButtonClick + DEBOUNCE_TIME)) && (_gameState == GAME_STATE_RUNNING) && (_userDirection != DIRECTION_LEFT))
     {
         _logger->log_string("Snake: LEFT");
         _userDirection = DIRECTION_RIGHT; // need to swap direction as field is rotated 180deg

src/matrix/animation_functions.cpp

@@ -10,6 +10,8 @@ extern LEDMatrix led_matrix;
 const int8_t dx[] = {1, -1, 0, 0};
 const int8_t dy[] = {0, 0, -1, 1};
 
+bool spiral_direction = false;  // Direction of sprial animation
+
 /**
  * @brief Function to draw a spiral step (from center)
  *
@@ -305,6 +307,8 @@ int random_tetris(bool init)
     if (init || gameover)
     {
         logger.log_string("Init Tetris: init=" + String(init) + ", gameover=" + String(gameover));
+        delay(10);
+
         // clear local game screen
         for (int h = 0; h < MATRIX_HEIGHT + 3; h++)
         {
@@ -364,6 +368,8 @@ int random_tetris(bool init)
         {
             // no more moving blocks -> check if game over or spawn new block
             logger.log_string("Tetris: No more Mover");
+            delay(10);
+
             gameover = false;
             // check if game was lost -> one pixel active in 4rd row (top row on the led grid)
             for (int s = 0; s < MATRIX_WIDTH; s++)

src/matrix/led_matrix.cpp

@@ -21,7 +21,7 @@ const uint32_t colors_24bit[NUM_COLORS] = {
  * @param mybrightness the initial brightness of the leds
  * @param mylogger pointer to the UDPLogger object
  */
-LEDMatrix::LEDMatrix(Adafruit_NeoMatrix *matrix, uint8_t brightness, UDPLogger *logger)
+LEDMatrix::LEDMatrix(FastLED_NeoMatrix *matrix, uint8_t brightness, UDPLogger *logger)
 {
     _neomatrix = matrix;
     _brightness = brightness;
@@ -181,6 +181,11 @@ void LEDMatrix::flush(void)
     _target_minute_indicators[3] = 0;
 }
 
+uint16_t LEDMatrix::get_fps(void)
+{
+    return FastLED.getFPS();
+}
+
 /**
  * @brief Write target pixels directly to leds
  *

src/matrix/render_functions.cpp

@@ -133,7 +133,7 @@ String time_to_string(uint8_t hours, uint8_t minutes)
     }
     else if (minutes >= 20 && minutes < 25)
     {
-        message += "ZEHN VOR HALB ";
+        message += "ZWANZIG NACH ";
     }
     else if (minutes >= 25 && minutes < 30)
     {
@@ -149,7 +149,7 @@ String time_to_string(uint8_t hours, uint8_t minutes)
     }
     else if (minutes >= 40 && minutes < 45)
     {
-        message += "ZEHN NACH HALB ";
+        message += "ZWANZIG VOR ";
     }
     else if (minutes >= 45 && minutes < 50)
     {
@@ -164,12 +164,14 @@ String time_to_string(uint8_t hours, uint8_t minutes)
         message += "FUNF VOR ";
     }
 
-    // convert hours to 12h format
-    if (minutes >= 20)
+    // increment hour when 25 minutes of an hour have passed
+    if (minutes >= 25)
     {
         hours++;
     }
-    if (hours > 12)  // 24h -> 12h, except 12h
+
+    // convert hours to 12h format
+    if (hours > 12) // 24h -> 12h, except 12h
     {
         hours -= 12;
     }
@@ -177,58 +179,79 @@ String time_to_string(uint8_t hours, uint8_t minutes)
     // show hours
     switch (hours)
     {
-    case 0:
-        if (minutes >= 0 && minutes < 5)
+        case 0:
         {
-            message += "MITTERNACHT ";
+            if (minutes >= 0 && minutes < 5)
+            {
+                message += "MITTERNACHT ";
+            }
+            else
+            {
+                message += "ZWOLF ";
+            }
+            break;
         }
-        else
+        case 1:
+        {
+            message += "EIN";
+            message += (minutes > 4) ? "S " : " ";  // add "S" if needed
+            break;
+        }
+        case 2:
+        {
+            message += "ZWEI ";
+            break;
+        }
+        case 3:
+        {
+            message += "DREI ";
+            break;
+        }
+        case 4:
+        {
+            message += "VIER ";
+            break;
+        }
+        case 5:
+        {
+            message += "FUNF ";
+            break;
+        }
+        case 6:
+        {
+            message += "SECHS ";
+            break;
+        }
+        case 7:
+        {
+            message += "SIEBEN ";
+            break;
+        }
+        case 8:
+        {
+            message += "ACHT ";
+            break;
+        }
+        case 9:
+        {
+            message += "NEUN ";
+            break;
+        }
+        case 10:
+        {
+            message += "ZEHN ";
+            break;
+        }
+        case 11:
+        {
+            message += "ELF ";
+            break;
+        }
+        case 12:
         {
             message += "ZWOLF ";
+            break;
         }
-        break;
-    case 1:
-        message += "EIN";
-        // EIN(S)
-        if (minutes > 4)
-        {
-            message += "S";
-        }
-        message += " ";
-        break;
-    case 2:
-        message += "ZWEI ";
-        break;
-    case 3:
-        message += "DREI ";
-        break;
-    case 4:
-        message += "VIER ";
-        break;
-    case 5:
-        message += "FUNF ";
-        break;
-    case 6:
-        message += "SECHS ";
-        break;
-    case 7:
-        message += "SIEBEN ";
-        break;
-    case 8:
-        message += "ACHT ";
-        break;
-    case 9:
-        message += "NEUN ";
-        break;
-    case 10:
-        message += "ZEHN ";
-        break;
-    case 11:
-        message += "ELF ";
-        break;
-    case 12:
-        message += "ZWOLF ";
-        break;
     }
     if ((minutes < 5) && (hours != 0))
     {
@@ -251,6 +274,7 @@ String split(String s, char parser, int index)
     String rs = "";
     int parser_cnt = 0;
     int r_from_index = 0, r_to_index = -1;
+
     while (index >= parser_cnt)
     {
         r_from_index = r_to_index + 1;
@@ -264,7 +288,9 @@ String split(String s, char parser, int index)
             return s.substring(r_from_index, r_to_index);
         }
         else
+        {
             parser_cnt++;
+        }
     }
     return rs;
 }

src/wrapper/base64_wrapper.cpp

@@ -1,143 +0,0 @@
-/*
-Copyright (C) 2016 Arturo Guadalupi. All right reserved.
-
-This library is free software; you can redistribute it and/or modify it under the terms of the GNU Lesser General Public License as published by the Free Software Foundation; either version 2.1 of the License, or (at your option) any later version.
-
-This library is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License for more details.
-*/
-
-#include <Arduino.h>
-#include "base64_wrapper.h"
-#if (defined(__AVR__))
-#include <avr/pgmspace.h>
-#else
-#include <pgmspace.h>
-#endif
-
-const char PROGMEM _Base64AlphabetTable[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
-		"abcdefghijklmnopqrstuvwxyz"
-		"0123456789+/";
-
-int Base64Class::encode(char *output, char *input, int inputLength) {
-	int i = 0, j = 0;
-	int encodedLength = 0;
-	unsigned char A3[3];
-	unsigned char A4[4];
-
-	while(inputLength--) {
-		A3[i++] = *(input++);
-		if(i == 3) {
-			fromA3ToA4(A4, A3);
-
-			for(i = 0; i < 4; i++) {
-				output[encodedLength++] = pgm_read_byte(&_Base64AlphabetTable[A4[i]]);
-			}
-
-			i = 0;
-		}
-	}
-
-	if(i) {
-		for(j = i; j < 3; j++) {
-			A3[j] = '\0';
-		}
-
-		fromA3ToA4(A4, A3);
-
-		for(j = 0; j < i + 1; j++) {
-			output[encodedLength++] = pgm_read_byte(&_Base64AlphabetTable[A4[j]]);
-		}
-
-		while((i++ < 3)) {
-			output[encodedLength++] = '=';
-		}
-	}
-	output[encodedLength] = '\0';
-	return encodedLength;
-}
-
-int Base64Class::decode(char * output, char * input, int inputLength) {
-	int i = 0, j = 0;
-	int decodedLength = 0;
-	unsigned char A3[3];
-	unsigned char A4[4];
-
-
-	while (inputLength--) {
-		if(*input == '=') {
-			break;
-		}
-
-		A4[i++] = *(input++);
-		if (i == 4) {
-			for (i = 0; i <4; i++) {
-				A4[i] = lookupTable(A4[i]);
-			}
-
-			fromA4ToA3(A3,A4);
-
-			for (i = 0; i < 3; i++) {
-				output[decodedLength++] = A3[i];
-			}
-			i = 0;
-		}
-	}
-
-	if (i) {
-		for (j = i; j < 4; j++) {
-			A4[j] = '\0';
-		}
-
-		for (j = 0; j <4; j++) {
-			A4[j] = lookupTable(A4[j]);
-		}
-
-		fromA4ToA3(A3,A4);
-
-		for (j = 0; j < i - 1; j++) {
-			output[decodedLength++] = A3[j];
-		}
-	}
-	output[decodedLength] = '\0';
-	return decodedLength;
-}
-
-int Base64Class::encodedLength(int plainLength) {
-	int n = plainLength;
-	return (n + 2 - ((n + 2) % 3)) / 3 * 4;
-}
-
-int Base64Class::decodedLength(char * input, int inputLength) {
-	int i = 0;
-	int numEq = 0;
-	for(i = inputLength - 1; input[i] == '='; i--) {
-		numEq++;
-	}
-
-	return ((6 * inputLength) / 8) - numEq;
-}
-
-//Private utility functions
-inline void Base64Class::fromA3ToA4(unsigned char * A4, unsigned char * A3) {
-	A4[0] = (A3[0] & 0xfc) >> 2;
-	A4[1] = ((A3[0] & 0x03) << 4) + ((A3[1] & 0xf0) >> 4);
-	A4[2] = ((A3[1] & 0x0f) << 2) + ((A3[2] & 0xc0) >> 6);
-	A4[3] = (A3[2] & 0x3f);
-}
-
-inline void Base64Class::fromA4ToA3(unsigned char * A3, unsigned char * A4) {
-	A3[0] = (A4[0] << 2) + ((A4[1] & 0x30) >> 4);
-	A3[1] = ((A4[1] & 0xf) << 4) + ((A4[2] & 0x3c) >> 2);
-	A3[2] = ((A4[2] & 0x3) << 6) + A4[3];
-}
-
-inline unsigned char Base64Class::lookupTable(char c) {
-	if(c >='A' && c <='Z') return c - 'A';
-	if(c >='a' && c <='z') return c - 71;
-	if(c >='0' && c <='9') return c + 4;
-	if(c == '+') return 62;
-	if(c == '/') return 63;
-	return -1;
-}
-
-Base64Class Base64;