robert/PedalPebble

Watch

Fork 0

CueSheetEnricher interface and canonical direction enum #26

New issue

Closed

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

robert commented

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

(Migrated from git.butzei.de)

Goal

One vocabulary for turns, so every backend and the watch icon set agree.

Acceptance criteria

Canonical direction enum adopted from the komoot BLE Connect spec: straight, forks, turns, roundabout exits, start, finish, off-route
CueSheetEnricher interface: takes a polyline, returns an ordered cue sheet
Cue = (polyline index, direction enum, street name, distance along route)
Backends tried in order with graceful fallback; which one succeeded is recorded
Mapping from each backend's vocabulary lives in exactly one place

Files

companion/.../route/enrich/CueSheetEnricher.kt
companion/.../route/enrich/Direction.kt

Notes

Spec: https://github.com/palto42/BLEConnect - the enum is well designed and publicly documented, which is why it is worth adopting even though we never talk to komoot.

Update — 2026-09-01: four tiers, and cue confidence

The chain gains a tier 0 and loses GraphHopper (D26):

Tier	Backend	Needs	Quality
0	`GpxCueEnricher` (#63) — cues already in the file	nothing	exact where present
1	`BRouterEnricher` (#27) via AIDL	BRouter + segments	good, subject to D25
2	`MatchingEnricher` (#28) — Valhalla only	network, consent	good
3	`GeometryEnricher` (#29) — heading delta	nothing	crude, no street names

Interface admits four implementations tried in order, with tier 0 first
Each cue carries which tier produced it, surfaced to the watch as NAV_CUE_CONFIDENCE so a
geometric guess is not presented with the authority of a matched turn
The canonical enum mapping layer covers the GPX vocabularies from #63 too, in the same place

## Goal One vocabulary for turns, so every backend and the watch icon set agree. ## Acceptance criteria - [ ] Canonical direction enum adopted from the komoot BLE Connect spec: straight, forks, turns, roundabout exits, start, finish, off-route - [ ] `CueSheetEnricher` interface: takes a polyline, returns an ordered cue sheet - [ ] Cue = (polyline index, direction enum, street name, distance along route) - [ ] Backends tried in order with graceful fallback; which one succeeded is recorded - [ ] Mapping from each backend's vocabulary lives in exactly one place ## Files - `companion/.../route/enrich/CueSheetEnricher.kt` - `companion/.../route/enrich/Direction.kt` ## Notes Spec: https://github.com/palto42/BLEConnect - the enum is well designed and publicly documented, which is why it is worth adopting even though we never talk to komoot. ## Update — 2026-09-01: four tiers, and cue confidence The chain gains a tier 0 and loses GraphHopper (D26): | Tier | Backend | Needs | Quality | |---|---|---|---| | 0 | `GpxCueEnricher` (#63) — cues already in the file | nothing | exact where present | | 1 | `BRouterEnricher` (#27) via AIDL | BRouter + segments | good, subject to D25 | | 2 | `MatchingEnricher` (#28) — **Valhalla only** | network, consent | good | | 3 | `GeometryEnricher` (#29) — heading delta | nothing | crude, no street names | - [ ] Interface admits four implementations tried in order, with tier 0 first - [ ] **Each cue carries which tier produced it**, surfaced to the watch as `NAV_CUE_CONFIDENCE` so a geometric guess is not presented with the authority of a matched turn - [ ] The canonical enum mapping layer covers the GPX vocabularies from #63 too, in the same place

robert added a new dependency

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

#24 GPX share-sheet import intent, parser and simplification

robert added a new dependency

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

#27 BRouterEnricher via IBRouterService AIDL

robert added a new dependency

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

#28 MatchingEnricher via Valhalla map matching

robert added a new dependency

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

#29 GeometryEnricher heading-delta fallback

robert added a new dependency

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

#30 Cue-sheet caching so each route is enriched once

robert added a new dependency

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

#33 Next turn, then-turn, remaining distance and ETA

robert added a new dependency

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

#34 Turn arrow icon set

robert added a new dependency

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

#38 Enrichment regression fixtures from real komoot GPX exports

robert added a new dependency

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

#63 GpxCueEnricher: tier-0 cues from the GPX itself

robert referenced this issue

2026-09-03 19:31:48 +02:00

GpxCueEnricher: tier-0 cues from the GPX itself #63

robert referenced this issue from a commit

2026-09-04 00:11:58 +02:00

Add CueSheetEnricher interface, canonical Direction enum, and tier-0 GpxCueEnricher (#26)

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

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

CueSheetEnricher interface, canonical Direction enum, and tier-0 GpxCueEnricher (#26) #79

robert referenced this issue from a commit

2026-09-04 00:13:29 +02:00

Merge pull request 'CueSheetEnricher interface, canonical Direction enum, and tier-0 GpxCueEnricher (#26)' (#79) from area/cue-sheet-enricher-interface into main

robert commented

2026-09-04 00:14:48 +02:00

Owner

Closed by PR #79 (merged): Direction enum (BLE Connect vocabulary), CueSheetEnricher interface, Cue/CueSheetResult/CueEnrichmentSource/EnrichmentTier types, TieredCueSheetEnricher four-slot orchestrator enforcing tier-0-first by construction. A real tier-0 GpxCueEnricher implementation came along with it (see #63 for its remaining scope) as tiers 1-3 remain unimplemented slots pending #27/#28/#29.

robert closed this issue

2026-09-04 00:14:48 +02:00

robert referenced this issue from a commit

2026-09-04 10:47:43 +02:00

Route library: storage, list UI, distance and elevation profile (#25)

robert referenced this issue

2026-09-04 10:48:17 +02:00

Route library: storage, list UI, distance and elevation profile (#25) #88

robert referenced this issue from a commit

2026-09-04 14:03:17 +02:00

Cache cue sheets so each route is enriched once (#30)

robert referenced this issue