robert/PedalPebble

Watch

Fork 0

BLE CSC wheel sensor client and wheel-circumference calibration #18

New issue

Closed

opened 2026-08-31 17:14:40 +02:00 by robert · 1 comment

robert commented

2026-08-31 17:14:40 +02:00

(Migrated from git.butzei.de)

Goal

Optional wheel sensor for instant, tunnel-proof speed.

Acceptance criteria

Scan for and connect to the standard BLE Cycling Speed and Cadence service 0x1816
CSC Measurement characteristic 0x2A5B parsed: cumulative wheel revolutions and last wheel event time
16-bit event-time and 32-bit revolution wraparound handled correctly
Wheel circumference configurable, with common presets
Auto-reconnect when the sensor wakes
Sensor battery level read where the peripheral exposes it

Files

companion/.../sensors/CscClient.kt

## Goal Optional wheel sensor for instant, tunnel-proof speed. ## Acceptance criteria - [ ] Scan for and connect to the standard BLE Cycling Speed and Cadence service `0x1816` - [ ] `CSC Measurement` characteristic `0x2A5B` parsed: cumulative wheel revolutions and last wheel event time - [ ] 16-bit event-time and 32-bit revolution wraparound handled correctly - [ ] Wheel circumference configurable, with common presets - [ ] Auto-reconnect when the sensor wakes - [ ] Sensor battery level read where the peripheral exposes it ## Files - `companion/.../sensors/CscClient.kt`

robert added a new dependency

2026-09-03 19:01:17 +02:00

#14 Android project skeleton and Gradle setup

robert added a new dependency

2026-09-03 19:01:17 +02:00

#19 Speed-source arbitration and GPS quality reporting

robert added a new dependency

2026-09-03 19:01:24 +02:00

#49 Cadence from the CSC sensor

robert added a new dependency

2026-09-03 19:01:24 +02:00

#50 BLE Cycling Power meter support

robert added a new dependency

2026-09-03 19:01:25 +02:00

#54 Sensor and battery dropout alerts

robert referenced this issue from a commit

2026-09-04 19:38:13 +02:00

Implement BLE CSC wheel sensor client and wheel-circumference calibration (#18)

robert referenced this issue from a pull request that will close it,

2026-09-04 19:38:38 +02:00

BLE CSC wheel sensor client and wheel-circumference calibration (#18) #99

robert closed this issue

2026-09-04 19:39:45 +02:00

robert referenced this issue from a commit

2026-09-04 19:39:46 +02:00

Merge pull request 'BLE CSC wheel sensor client and wheel-circumference calibration (#18)' (#99) from area/ble-csc-sensor into main

robert commented

2026-09-04 19:39:58 +02:00

Owner

Closed by PR #99 (merged): CSC UUIDs verified live against the current Bluetooth SIG spec (0x1816/0x2A5B/0x2A5C), wraparound-safe uint32/uint16 delta math via Math.floorMod (JVM's always-non-negative modulo, not %'s sign-follows-dividend), 23 host tests including fixtures that actually cross real uint16 and uint32 rollover boundaries. WheelSpeedTracker wired directly against #44's WheelCircumference settings type. Sensor state modeled as a proper sealed class (NotConnected/Scanning/Connecting/Reconnecting/Connected) so "no sensor" and "sensor stationary" (NoNewRevolution) can never collapse into a fabricated 0 speed \u2014 sets #19 (speed-source arbitration) up to consume this honestly later.\n\nReal bug caught by actually compiling against this project's real compileSdk 37: API 37 deprecates the old connectGatt(Context, boolean, callback, transport) overload in favor of connectGatt(BluetoothGattConnectionSettings, Executor, callback) \u2014 confirmed via the installed SDK's api-versions.xml, not assumed. AndroidCscClient branches on SDK_INT since minSdk is 31.\n\nVerified: :companion:core:test 23/23, :companion:sensors:assembleDebug/:companion:assembleDebug both clean. No Bluetooth hardware in this sandbox \u2014 the scan/connect/notify path against a real sensor was compiled, not exercised.

Closed by PR #99 (merged): CSC UUIDs verified live against the current Bluetooth SIG spec (0x1816/0x2A5B/0x2A5C), wraparound-safe uint32/uint16 delta math via Math.floorMod (JVM's always-non-negative modulo, not %'s sign-follows-dividend), 23 host tests including fixtures that actually cross real uint16 and uint32 rollover boundaries. WheelSpeedTracker wired directly against #44's WheelCircumference settings type. Sensor state modeled as a proper sealed class (NotConnected/Scanning/Connecting/Reconnecting/Connected) so \"no sensor\" and \"sensor stationary\" (NoNewRevolution) can never collapse into a fabricated 0 speed \u2014 sets #19 (speed-source arbitration) up to consume this honestly later.\n\n**Real bug caught by actually compiling against this project's real compileSdk 37**: API 37 deprecates the old connectGatt(Context, boolean, callback, transport) overload in favor of connectGatt(BluetoothGattConnectionSettings, Executor, callback) \u2014 confirmed via the installed SDK's api-versions.xml, not assumed. AndroidCscClient branches on SDK_INT since minSdk is 31.\n\n**Verified**: `:companion:core:test` 23/23, `:companion:sensors:assembleDebug`/`:companion:assembleDebug` both clean. No Bluetooth hardware in this sandbox \u2014 the scan/connect/notify path against a real sensor was compiled, not exercised.

robert referenced this issue

2026-09-04 19:55:09 +02:00

Auto-pause: one definition of stopped, in Phase 2 #69

robert referenced this issue

2026-09-04 20:30:12 +02:00

Audit issue #22 speed-pipeline test coverage, fill the real GPX-replay distance gap #103

robert referenced this issue

2026-09-04 20:42:21 +02:00

Derived metric producers: MAX_SPEED, AVG_HR, HR_ZONE, AVG_CADENCE, NORM_POWER (#65) #104

robert referenced this issue

2026-09-04 20:43:23 +02:00

Derived metric producers: max speed, average HR, average cadence, normalised power #65