Release 4.1.0 — LED strip + assignable status LEDs, BLE download rework, RPM fixes, menu escapes

.github/workflows/beta.yml

@@ -9,7 +9,7 @@ name: beta
 # builds you flash from your phone via DovesDataViewer's beta channel while
 # wrenching at the track.
 #
-# Three things differ from release.yml:
+# Four things differ from release.yml:
 #   1. The image self-reports a beta version. The build injects
 #      -DFIRMWARE_VERSION_OVERRIDE=<base>-beta.<gitsha> so a flashed unit
 #      reports its exact commit over BLE (DIS) — you can read the version off
@@ -23,6 +23,14 @@ name: beta
 #      from master/release, which default the flag to 0. See project.h for
 #      the full flag list — anything else defaulted off stays off here too
 #      (BIRDSEYE_ENABLE_ONBOARD_CHARGING in particular).
+#   4. Beta is also the loop-profiling channel: -DBIRDSEYE_ENABLE_PROFILING=1
+#      turns on the LOOP PROFILE page and the pin-30 scope output (plan
+#      0011). That flag TAKES PIN 30 FROM THE NEOPIXEL BOOST EN LINE, so a
+#      beta image can no longer switch the 5 V rail — including at sleep,
+#      where the level is retained. A beta unit left asleep on a battery
+#      with LEDs wired will drain it. That is the accepted cost of getting
+#      real per-subsystem CPU numbers off the hardware; prod images are
+#      unaffected. See project.h and BirdsEye/profiling.h.
 #
 # NOTE: the build steps intentionally mirror release.yml's build job. If the
 # BSP version, library list, or asset layout changes there, mirror it here.
@@ -93,7 +101,8 @@ jobs:
             "Adafruit SSD1306" \
             "SparkFun u-blox GNSS v3" \
             "Seeed Arduino LSM6DS3" \
-            "ArxTypeTraits"
+            "ArxTypeTraits" \
+            "Adafruit NeoPixel"
           # Beta builds track the lap-timer library's own BETA branch so the
           # two beta channels move together (master/release pin a tag).
           arduino-cli lib install --git-url https://github.com/TheAngryRaven/DovesLapTimer.git#BETA
@@ -103,7 +112,7 @@ jobs:
         run: |
           arduino-cli compile \
             --fqbn "${{ matrix.board.fqbn }}" \
-            --build-property "compiler.cpp.extra_flags=-D${{ matrix.board.define }} -DFIRMWARE_VERSION_OVERRIDE=${{ steps.ver.outputs.betaver }} -DSERIAL_BUFFER_SIZE=256 -DDOVES_DISABLE_DEBUG -DBIRDSEYE_ENABLE_SENSOREGG=1" \
+            --build-property "compiler.cpp.extra_flags=-D${{ matrix.board.define }} -DFIRMWARE_VERSION_OVERRIDE=${{ steps.ver.outputs.betaver }} -DSERIAL_BUFFER_SIZE=256 -DDOVES_DISABLE_DEBUG -DBIRDSEYE_ENABLE_SENSOREGG=1 -DBIRDSEYE_ENABLE_NEOPIXEL=1 -DBIRDSEYE_ENABLE_PROFILING=1" \
             --output-dir dist \
             --warnings none \
             BirdsEye

.github/workflows/clang-tidy.yml

@@ -41,4 +41,13 @@ jobs:
             BirdsEye/lap_format.cpp \
             BirdsEye/tach_filter.cpp \
             BirdsEye/camera_fsm.cpp \
-            BirdsEye/insta360_protocol.cpp
+            BirdsEye/insta360_protocol.cpp \
+            BirdsEye/led_frame.cpp \
+            BirdsEye/led_modes.cpp \
+            BirdsEye/led_status.cpp \
+            BirdsEye/led_animations.cpp \
+            BirdsEye/sector_purple.cpp \
+            BirdsEye/local_time.cpp \
+            BirdsEye/setting_parse.cpp \
+            BirdsEye/ble_stream.cpp \
+            BirdsEye/loop_profile.cpp

.github/workflows/compile-sketch.yml

@@ -34,10 +34,15 @@ jobs:
       # release tag.
       LAPTIMER_REF: ${{ (github.base_ref == 'BETA' || github.head_ref == 'BETA' || github.ref_name == 'BETA') && 'BETA' || 'v4.3.0' }}
       # Feature flags for this build (see project.h). BETA enables the
-      # SensorEgg POC exactly as beta.yml does, so the flag-on build is
-      # compile-checked on the PR rather than first failing on the publish
-      # workflow. Everything else builds the master/release defaults (off).
-      FEATURE_FLAGS: ${{ (github.base_ref == 'BETA' || github.head_ref == 'BETA' || github.ref_name == 'BETA') && '-DBIRDSEYE_ENABLE_SENSOREGG=1' || '' }}
+      # SensorEgg POC, the NeoPixel strip and loop profiling exactly as
+      # beta.yml does, so the flag-on build is compile-checked on the PR
+      # rather than first failing on the publish workflow. Everything else
+      # builds the project.h defaults — which since 4.1.0 means the
+      # NeoPixel strip is ON (it is a core feature now; the flag defaults
+      # to 1) and only the SensorEgg POC and profiling are off. So the
+      # difference between the two arms is SensorEgg, profiling, and the
+      # DovesLapTimer ref.
+      FEATURE_FLAGS: ${{ (github.base_ref == 'BETA' || github.head_ref == 'BETA' || github.ref_name == 'BETA') && '-DBIRDSEYE_ENABLE_SENSOREGG=1 -DBIRDSEYE_ENABLE_NEOPIXEL=1 -DBIRDSEYE_ENABLE_PROFILING=1' || '' }}
     # Build both XIAO nRF52840 variants. The Sense board has the onboard
     # LSM6DS3 IMU; the plain board does not (accelerometer logging degrades
     # gracefully). Same MCU/BLE/bootloader otherwise.
@@ -77,6 +82,7 @@ jobs:
             - name: SparkFun u-blox GNSS v3
             - name: Seeed Arduino LSM6DS3
             - name: ArxTypeTraits
+            - name: Adafruit NeoPixel
             - source-url: https://github.com/TheAngryRaven/DovesLapTimer.git
               version: ${{ env.LAPTIMER_REF }}
           sketch-paths: |
@@ -193,3 +199,55 @@ jobs:
           name: size-deltas-${{ matrix.board.name }}
           path: sketches-reports
           if-no-files-found: ignore
+
+  # Flag-OFF compile, for BETA-targeted work only.
+  #
+  # The `compile` job above picks ONE flag set per PR: BETA-targeted work
+  # builds with SensorEgg + profiling ON, everything else with them off.
+  # That means code inside a `#else` arm of one of those flags reaches
+  # master having never been through a compiler — and plan 0013 added
+  # several (the DOVEX row/header shapes and the settings defaults now
+  # differ by channel). This job closes that hole from the BETA side.
+  #
+  # One board is enough: the two arms here differ only in feature flags,
+  # and the sense/non-sense split is already covered above.
+  # LAPTIMER_REF deliberately stays BETA so the flags are the ONLY
+  # variable — pinning the release tag as well would fail on BETA-era
+  # library APIs and tell us nothing about the flags.
+  compile-stock-arm:
+    name: xiao-nrf52840-sense (flags off)
+    if: >-
+      github.base_ref == 'BETA' || github.head_ref == 'BETA' ||
+      github.ref_name == 'BETA'
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Install adafruit-nrfutil
+        run: pip install --user adafruit-nrfutil
+
+      - name: Compile sketch with the feature flags off
+        uses: arduino/compile-sketches@v1
+        with:
+          fqbn: Seeeduino:nrf52:xiaonRF52840Sense
+          platforms: |
+            - name: Seeeduino:nrf52
+              source-url: https://files.seeedstudio.com/arduino/package_seeeduino_boards_index.json
+          libraries: |
+            - name: ArduinoJson
+            - name: SdFat - Adafruit Fork
+            - name: Adafruit GFX Library
+            - name: Adafruit SH110X
+            - name: Adafruit SSD1306
+            - name: SparkFun u-blox GNSS v3
+            - name: Seeed Arduino LSM6DS3
+            - name: ArxTypeTraits
+            - name: Adafruit NeoPixel
+            - source-url: https://github.com/TheAngryRaven/DovesLapTimer.git
+              version: BETA
+          sketch-paths: |
+            - BirdsEye
+          cli-compile-flags: |
+            - --build-property
+            - compiler.cpp.extra_flags=-DBIRDSEYE_BOARD_SENSE -DSERIAL_BUFFER_SIZE=256 -DDOVES_DISABLE_DEBUG

.github/workflows/release.yml

@@ -69,7 +69,8 @@ jobs:
             "Adafruit SSD1306" \
             "SparkFun u-blox GNSS v3" \
             "Seeed Arduino LSM6DS3" \
-            "ArxTypeTraits"
+            "ArxTypeTraits" \
+            "Adafruit NeoPixel"
           # Release builds pin the lap-timer library to a known-good tag —
           # bump deliberately. (Beta builds track its BETA branch instead.)
           arduino-cli lib install --git-url https://github.com/TheAngryRaven/DovesLapTimer.git#v4.3.0

.gitignore

@@ -2,6 +2,8 @@
 # Host unit-test build (also covered by tests/.gitignore)
 build/
 tests/build/
+# Sim wasm build (emcmake; CI uploads its dist/ as the artifact)
+build-wasm/
 
 # Coverage artifacts generated by the coverage workflow / local runs
 public-badges/