biking_home/climb/detection.go, branch mainUnnamed repository; edit this file 'description' to name the repository.
https://git.kagamino.dev/biking_home/atom?h=main2026-08-27T10:07:11Zclenaup2026-08-27T10:07:11ZMartin Kagamino Lehouxmartin@lehoux.net2026-08-27T10:07:11Zurn:sha1:522512197de372aa9b9c5584711619a76a5546d8Tame the deadcode gate and remove dead web helpers2026-08-27T09:32:31ZMartin Kagamino Lehouxmartin@lehoux.net2026-08-27T09:32:31Zurn:sha1:1c5fc2e4364063884f33b78f45f351201705fae4
Run deadcode with -test and without the vendored node_modules package so the
check reports only genuinely unreachable code instead of test-only helpers and
vendored functions. Remove the three web helpers it still flagged (formatBikeType,
formatRideType, dataQualityURL) that had no callers. Apply the go fix
modernizations the check gate performs (range-over-int loops and embedded-field
composite literals) that the gate had rewritten in the working tree.
Extract climb detection into a dedicated climb package2026-08-27T09:21:17ZMartin Kagamino Lehouxmartin@lehoux.net2026-08-27T09:21:17Zurn:sha1:825cf388b55a6c8441e02149749bb6e2968477c0
Move climb detection, slope, and Cotacol scoring out of ride into a new
climb package that operates on a geometry-only Route (distances, elevations,
coords). Ride embeds climb.Route and keeps its timing and telemetry columns.
- climb.Detect returns []DetectedClimb (Climb + ride-absolute Start).
- Climb owns its sliced geometry; indices are local, with Start reported at
the detection boundary for absolute position.
- Detected bounds use core.Interval; freeRegions claim suppression fixed.
- ride no longer builds slices: ClimbFromDist/ClimbFromIndexes/climbFrom and
ride.Climb/Interval/Range are removed; rideanalysis, mountain_pass MatchClimb,
and web views are rewired to climb types.
- New internal/routetest builds climb.Route from the ride DSL; ridetest reuses
it and adds timestamps and telemetry.
- Tests moved to climb (synthetic detection and difficulty) and kept in ride
for GPX-example integration; ride.comparison detection coverage retained.
- mountain_pass.nearestCrossing now reuses geo.Nearest.