docs(openspec): phase6 tracker cleanup — check off done T7 tasks + corrections - #252
Conversation
…3000ep budget Tracker cleanup after the C3 assembly / metric / re-measure PRs (#249/#250/#251): - Check off the three tasks completed but left unmarked: T7.prep.connectome_continuous (Stage 2 — connectome climbed C1->C2->C3, integrated config committed, re-measured ~52% mid-pack), T7.prep.continuous_behaviours (parent) and its s3_generic_and_assemble sub-task (LSTM/CfC behaviours + the 6 assembled C3 configs). - Add a Stage-2-complete summary sub-bullet for the connectome. - Add the missing T7.prep.convergence_metric task (the level-agnostic ranked-metric fix, #250) — a hard prerequisite for the n>=8 ranking that had no tracker entry. - Add a Stage-6 readiness note: configs committed + recipes frozen, the 3000ep budget, the level-agnostic metric, and the revised converged ranking pending the n>=8 runs. No task content invalidated; remaining [ ] tasks (n>=8 cells, ablations, SHOULD/MAY arms, validation, analysis, logbook) are genuinely outstanding. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…d post-Stage-6) Mark the one remaining 'before the runs' prep task as explicitly non-gating for the MUST n>=8 ranking: the 6 MUST arms don't need it; it gates only optional SHOULD/MAY arm inclusion, decided after Stage 6 (revisit if we add any; quantum already settled negative at T4). Keeps it from reading as a forgotten prerequisite. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Branch review caught a self-contradiction: the converged-ranking line listed 'connectome 52 > LSTM (~59 tuned)', placing tuned-LSTM (59) last despite 59 > 52. The baseline 2400ep re-measure had LSTM last (45); the 3000ep re-tune lifts it to ~59, re-ranking it into the CfC/connectome mid-cluster (~52-59) whose internal order is unsettled pending the uniform-budget n>=8. Fix the tracker (both the #251 checkpoint and the Stage-6 readiness note) and the 5 config headers' ranking line to state this consistently. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughInline ranking comments in five ChangesPhase 6 T7 Ranking Comment and Tracker Sync
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
What
Tracker hygiene pass on
phase6-tracking/tasks.mdafter the C3-assembly / metric / re-measure PRs (#249/#250/#251), bringing the T7 checkbox state and notes in line with what's actually done.Changes
Check off completed-but-unmarked tasks:
T7.prep.connectome_continuous(Stage 2) — connectome climbed C1→C2→C3, integrated config committed, re-measured ~52% mid-pack (added a Stage-2-complete summary sub-bullet).T7.prep.continuous_behaviours(parent) +s3_generic_and_assemble— LSTM/CfC behaviours + the 6 assembled C3 configs.Add what was missing:
T7.prep.convergence_metric— the level-agnostic ranked-metric fix (feat(benchmark): level-agnostic plateau convergence detection #250) had no tracker entry despite being a hard prerequisite for the ranking. Added marked-done with the why.Clarity + correctness:
T7.prep.should_may_continuousas non-gating for the MUST ranking (it gates only optional SHOULD/MAY arm inclusion, deferred to after Stage 6) so it doesn't read as a forgotten prerequisite.connectome 52 > LSTM (~59 tuned), placing tuned-LSTM last despite 59 > 52. Corrected in both tracker occurrences and the 5 config headers to state the 2400ep baseline order (LSTM 45 last) and that the re-tune lifts LSTM to ~59 into the CfC/connectome/LSTM mid-cluster (~52–59), internal order pending the uniform-budget n≥8.Audit result
Every prerequisite for the MUST n≥8 ranking is now checked off; the prep section is complete for the 6 MUST arms. Remaining
[ ]tasks are genuinely outstanding (the n≥8 cells themselves, ablations, SHOULD/MAY arms, separation experiments, real-worm validation, analysis, logbook) — nothing blocks the final runs.Validation
openspec validate phase6-tracking --strict✓; all 5 configs parse; pre-commit (yaml + mdformat + markdownlint) ✓; no tmp/ or home-path leaks.🤖 Generated with Claude Code
Summary by CodeRabbit