Ship bundled penprobe lookup map + wire it as the demo default#7
Merged
Conversation
…demo default The penprobe lookup demo now works on a fresh checkout with zero setup: open the page and hit Run, and the spine chases the nearest reachable point to the live 0B penprobe. - Commit the 2493-point workspace lookup map (compact JSON, ~1.3 MB) to a tracked location OUTSIDE the gitignored data/experiments/ tree: continuum_robot/demo/maps/two_segment_workspace_lookup_map.json. Built from the big 20260526_235950 servo_only collect-pose dataset (allow_lower_trust); tagged bundled_demo_default_map. - Wire it as the demo default: TwoSegmentPenprobeLookupDemoConfig.map_path now defaults to that path (dataclass + from_dict), resolved against project_root at runtime so it works on any checkout. An ABSENT map_path uses the bundled map; an explicitly-empty map_path still refuses (operator intent preserved). Tests: add test_demo_uses_bundled_default_map_when_none_specified (stages the real committed map under the runner project_root, runs with no map_path -> loads the 2493-point map, success); update the GUI page test (default now pre-fills the bundled path) and the empty-map-path refusal test. two-segment suite: 274 pass. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
data/experiments/tree:continuum_robot/demo/maps/two_segment_workspace_lookup_map.json. Built from the big20260526_235950servo_only collect-pose dataset.TwoSegmentPenprobeLookupDemoConfig.map_pathnow defaults to that path (resolved againstproject_rootat runtime). An absentmap_path→ bundled map; an explicitly-emptymap_pathstill refuses (operator intent preserved). The GUI map field pre-fills the bundled path on open; "Use Latest Built Map" / "Browse…" still override it.Test plan
test_demo_uses_bundled_default_map_when_none_specified— stages the real committed map under the runner project_root, runs with nomap_path, loads the 2493-point map, succeeds🤖 Generated with Claude Code