Skip to content

feat: distributed, deterministic, seed CLI flags for pretrain#397

Merged
noahgift merged 1 commit intomainfrom
batch-10-cli-flags
Mar 3, 2026
Merged

feat: distributed, deterministic, seed CLI flags for pretrain#397
noahgift merged 1 commit intomainfrom
batch-10-cli-flags

Conversation

@noahgift
Copy link
Contributor

@noahgift noahgift commented Mar 3, 2026

Summary

  • Add --distributed, --world-size, --rank, --coordinator-addr CLI flags to apr train apply for DDP pretraining
  • Add --deterministic and --seed CLI flags for bitwise reproducible training (C-DETERM-001)
  • Extract build_distributed_yaml(), patch_yaml_config(), print_pretrain_header() helpers to reduce cognitive complexity
  • All flags inject into YAML config via temp file before passing to entrenar

Test plan

  • cargo build --release -p apr-cli passes
  • apr train apply --help shows all new flags
  • pmat pre-commit complexity gates pass

🤖 Generated with Claude Code

… (Refs #393)

Wire --distributed, --world-size, --rank, --coordinator-addr through Apply
variant to run_apply_pretrain() with YAML injection. Add --deterministic
and --seed for bitwise reproducible training (C-DETERM-001). Extract
build_distributed_yaml(), patch_yaml_config(), print_pretrain_header()
helpers to reduce cognitive complexity.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@noahgift noahgift force-pushed the batch-10-cli-flags branch from dd7f8c5 to cd9e3c0 Compare March 3, 2026 22:18
@noahgift noahgift merged commit 70204d0 into main Mar 3, 2026
2 of 11 checks passed
@noahgift noahgift deleted the batch-10-cli-flags branch March 3, 2026 22:40
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