lars/giant
1
0
Fork 0
Code Issues 4 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

Make default checkpoint out_dir name reflect only non-default hyperparams #19

Merged
lars merged 1 commits from feature/smart-outdir-naming into master 2026-07-29 13:38:16 +02:00
Conversation 0 Commits 1 Files Changed 3
+198 -11
lars commented 2026-07-29 11:39:53 +02:00
Owner
Copy Link
Copy Source

Previously the same fixed 7 fields (mode/hidden_dim/n_blocks/emb_dim/
conditioning/lr/batch_size) were always baked into the name, even for a
vanilla run, and router config wasn't represented at all. Now
default_out_dir_name only includes fields that differ from
DEFAULT_CONFIG, adds router/seed/epochs as candidates, and caps at 6
shown fields with a hashed overflow suffix for heavily-swept configs.

Previously the same fixed 7 fields (mode/hidden_dim/n_blocks/emb_dim/ conditioning/lr/batch_size) were always baked into the name, even for a vanilla run, and router config wasn't represented at all. Now default_out_dir_name only includes fields that differ from DEFAULT_CONFIG, adds router/seed/epochs as candidates, and caps at 6 shown fields with a hashed overflow suffix for heavily-swept configs.
lars added 1 commit 2026-07-29 11:39:53 +02:00
Make default checkpoint out_dir name reflect only non-default hyperparams
CI / Format (ruff format) (push) Successful in 27s
Details
CI / Lint (ruff check) (push) Successful in 28s
Details
CI / Sync project version with tag (push) Has been skipped
Details
CI / Type check (ty) (push) Successful in 23s
Details
CI / Tests (push) Successful in 53s
Details
CI / Format (ruff format) (pull_request) Successful in 28s
Details
CI / Lint (ruff check) (pull_request) Successful in 28s
Details
CI / Sync project version with tag (pull_request) Has been skipped
Details
CI / Type check (ty) (pull_request) Successful in 22s
Details
CI / Tests (pull_request) Successful in 52s
Details
1115451c8e 
Previously the same fixed 7 fields (mode/hidden_dim/n_blocks/emb_dim/
conditioning/lr/batch_size) were always baked into the name, even for a
vanilla run, and router config wasn't represented at all. Now
default_out_dir_name only includes fields that differ from
DEFAULT_CONFIG, adds router/seed/epochs as candidates, and caps at 6
shown fields with a hashed overflow suffix for heavily-swept configs.
lars marked the pull request as ready for review 2026-07-29 11:48:47 +02:00
lars merged commit e331148afa into master 2026-07-29 13:38:16 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
architecture
experiment
investigation
performance
phase2
validation
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
lars (Lars Bogner)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: lars/giant#19
Delete Branch "feature/smart-outdir-naming"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?