Record where the YOLO baseline weights actually live (#51) - #92
Closed
jonfroehlich wants to merge 1 commit into
Closed
Record where the YOLO baseline weights actually live (#51)#92jonfroehlich wants to merge 1 commit into
jonfroehlich wants to merge 1 commit into
Conversation
The record's "Where the weights live" section still carried a TODO to stage them somewhere durable. They have been durable since 2026-07-29: /gscratch/makelab/ jonf/rampnet_yolo_baseline_51 on klone lab storage, outside the ~21-day /gscratch/scrubbed purge this record exists to survive, with a MANIFEST.md carrying sha256 for all 10 .pt files verified against source at copy time. A record this careful about provenance should not point at a TODO for the one artifact that cannot be regenerated. Two caveats included rather than left implicit: it is a snapshot taken while the arms were still training (y11l_pano has since gone 13 -> 15 epochs, y26_pano 18 -> 20), so it is not a live mirror and must be re-taken before anything is called final; and the y26_tiles_l40s fork postdates it and is missing entirely. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Member
Author
|
Closing as redundant — PR #87 already fixes this TODO, and more thoroughly than this branch does. I flagged it as an open gap after reading Two small deltas from this branch that #87 does not currently carry, noted here so they are not lost:
Happy to add both to #87 as a commit if wanted. |
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
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.
One-section fix to
scripts/model_comparison/yolo_baseline/README.md.The problem
That record is meticulous about provenance — toolchain versions, resolved hyperparameters
(including
optimizer=autosilently resolving toMuSGD(lr=0.01)and overriding the passedlr0), the Slurm job↔config map, dataset prep decisions, and two documented self-corrections.Its "Where the weights live" section was still a TODO, for the one artifact in the whole
record that cannot be regenerated.
The fix
They have been durable since 2026-07-29 and nothing pointed at them:
/gscratch/makelab/jonf/rampnet_yolo_baseline_51— klone lab storage, outside the~21-day
/gscratch/scrubbedpurge that this record exists to survive. 1.8 GB; each armdirectory holds
best.pt,last.pt,args.yaml,results.csv. AMANIFEST.mdtherealready carries the snapshot time, the job↔config map, per-arm epoch/mAP state, and
sha256 for all 10
.ptfiles verified against source at copy time — it simply was notreferenced from the record.
Two caveats included rather than left implicit
still training. They have advanced since —
y11l_pano13 → 15 epochs,y26_pano18 → 20as of 2026-07-30 — so it must be re-taken before anything is quoted as final, and
/gscratch/scrubbed/jfroehli/yolo_runs/<arm>/results.csvstays the live source until then.y26_tiles_l40sis missing entirely. That arm postdates the snapshot (Slurm 37889646,forked to a dedicated
gpu-l40snode on 2026-07-29 to escape the ckpt preemption ceiling)and needs adding on the next pass.
Staging to Hugging Face is still worth doing for external reproducibility, and the section
says so — it is just no longer the only thing standing between these weights and the purge.
Related: #51 · #91 (Tillicum onboarding, which re-snapshots as part of the migration)
🤖 Generated with Claude Code (claude-opus-5[1m])