fix: move Gardiner corridor onto land#5
Conversation
The Gardiner Expressway corridor in scripts/generate.ts placed its waypoints at latitudes 43.633-43.640, at or south of the downtown harbour shoreline, and its east end at -79.30 (about 2 km offshore off the Beaches). Haulix 02's interpolated GPS pings therefore rendered in Lake Ontario. Nudge the four waypoints north onto the real Gardiner alignment (just south of the CN Tower, ~43.638-43.646 along the waterfront) and pull the east end back to the actual DVP junction (~-79.358). Reviewed all five corridors; the other four (401, 400, QEW, DVP) are inland or north of the lakeshore and were left unchanged. Regenerated truth.json and haulix.json; vehicle registry (VINs) and per-vehicle event counts are unchanged since corridor geometry does not affect the RNG or speed profile.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe Gardiner Expressway corridor waypoints in Gardiner corridor reroute and hx-02 data regeneration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
The Gardiner Expressway corridor in the Phase 2 generator placed its waypoints south of the downtown shoreline, so Haulix 02's interpolated GPS pings rendered in Lake Ontario. Nudges the four Gardiner waypoints north onto the real road alignment and pulls the east end back to the DVP junction. Reviewed all five corridors; only the Gardiner drifted offshore. Regenerated truth.json and haulix.json; VINs and event counts unchanged. All tests and the build pass.
Summary by CodeRabbit