Repository navigation
Conversation
Both SFT and OSFT backends default to beta2=0.95 instead of PyTorch's standard 0.999. This non-obvious difference affects training dynamics and could confuse users porting hyperparameters from other frameworks. Source: training_hub PR #108 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: multica-agent <github@multica.ai>
|
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 (1)
📝 WalkthroughWalkthroughAdds one bullet to ChangesAGENTS.md AdamW documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Adds 1 new learning to the "Working With This Codebase Specifically" section of AGENTS.md:
beta2=0.95, not PyTorch's standard0.999. This non-standard default affects training dynamics and can confuse users porting hyperparameters from other frameworks. All four AdamW parameters (beta1,beta2,eps,weight_decay) are now exposed as first-class kwargs onsft()andosft().Sources
Relationship to open PRs
Independent — no overlap with open PRs #96, #97, #99, #100, #101, #102, or #105. Those cover NVIDIA driver compat, Unsloth import order, datasets/pyarrow, trl 1.x packing, and other topics. This PR covers the AdamW beta2 default, which none of the open PRs address.
Analysis
22 merged PRs analyzed across 3 repos since last merged AGENTS.md update (PR #84, June 3). One new non-obvious learning identified:
Other recent merges (PR #104 trl packing, PR #110 datasets pin, PR #83 ITS rollout, PR #106 validation loss docs) are either already covered by open PR #105 or are code/docs-derivable facts that don't meet the bar for AGENTS.md.
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit