-
Notifications
You must be signed in to change notification settings - Fork 527
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(pt): use user seed in DpLoaderSet
#4015
Conversation
WalkthroughWalkthroughThe recent changes enhance randomness control in the training process by adding a Changes
Sequence Diagram(s)sequenceDiagram
participant Trainer as Trainer
participant DataLoader as DataLoader
participant Model as Model
Trainer->>DataLoader: Prepare training input with seed
DataLoader->>Model: Set up data with provided seed
Model-->>Trainer: Return prepared data
Recent review detailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## devel #4015 +/- ##
==========================================
- Coverage 82.92% 82.92% -0.01%
==========================================
Files 522 522
Lines 51010 51009 -1
Branches 3023 3023
==========================================
- Hits 42301 42300 -1
+ Misses 7767 7765 -2
- Partials 942 944 +2 ☔ View full report in Codecov by Sentry. |
<!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced a `seed` parameter for enhanced control over randomness in the training process. - Improved randomness handling for distributed training setups to ensure unique batch sequences. - **Bug Fixes** - Enhanced robustness in seed setup during initialization to prevent unnecessary function calls with a `None` value. - **Chores** - Streamlined the random seed management by removing redundant seed-setting logic in training processes. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Summary by CodeRabbit
New Features
seed
parameter for enhanced control over randomness in the training process.Bug Fixes
None
value.Chores