Skip to content

fix(monocular): bootstrap fresh-worktree private follow-up#34

Draft
helionaut wants to merge 2 commits intomainfrom
eugeniy/hel-79-execute-the-next-follow-up-from-the-final-validation-report
Draft

fix(monocular): bootstrap fresh-worktree private follow-up#34
helionaut wants to merge 2 commits intomainfrom
eugeniy/hel-79-execute-the-next-follow-up-from-the-final-validation-report

Conversation

@helionaut
Copy link
Copy Markdown
Owner

Summary

  • bootstrap the HEL-79 private follow-up wrapper's missing repo-local media and native prerequisites in fresh worktrees
  • add a make monocular-prereqs verification phase before the aggressive HEL-57 replay starts
  • cover the phase planner with focused regression tests and record that the guarded HEL-79 execution is still running for runtime evidence

Validation

  • make test
  • make build
  • make smoke
  • make check

Follow-ups

  • the guarded HEL-79 execution is still active from this branch and has not reached the private save-path runtime yet because the nested mono_tum_vi rebuild is still in progress
  • live runtime evidence is currently in .symphony/progress/HEL-79.json, .symphony/progress/HEL-79-private-run.json, logs/out/hel-79_private_save_comparison_followup.log, and logs/out/hel-79_private_monocular_followup.log
  • once the build clears, this branch still needs the runtime/save-path artifacts, final validation evidence, green PR checks, and then merge/deploy proof before the Linear issue can move forward

helionaut and others added 2 commits March 21, 2026 13:31
Summary:
- teach the private monocular follow-up wrapper to bootstrap missing
  repo-local media and native prerequisites in a fresh worktree
- add a prerequisite verification phase before the aggressive HEL-57
  replay starts
- cover the new phase planner with focused tests

Rationale:
- the first HEL-79 rerun failed before the save-path diagnostics because
  the wrapper assumed ffmpeg and the native ORB-SLAM3 dependency lane
  were already present from an older workspace
- codifying those prerequisites in the wrapper leaves a reproducible
  repo-owned lane instead of depending on hidden host state

Tests:
- PYTHONPATH=src python3 -m unittest tests.test_run_private_monocular_followup tests.test_run_private_save_comparison_followup tests.test_run_monocular_baseline tests.test_patch_orbslam3_baseline
- python3 -m py_compile scripts/run_private_monocular_followup.py scripts/run_private_save_comparison_followup.py scripts/run_monocular_baseline.py scripts/patch_orbslam3_baseline.py
- make check

Co-authored-by: Codex <codex@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant