-
Notifications
You must be signed in to change notification settings - Fork 556
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
[WIP] 2D parallelism for training #2318
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/torchtune/2318
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New Failures, 2 Cancelled JobsAs of commit cfd31cd with merge base d4465c8 ( NEW FAILURES - The following jobs have failed:
CANCELLED JOBS - The following jobs were cancelled. Please retry:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2318 +/- ##
==========================================
- Coverage 23.87% 23.64% -0.24%
==========================================
Files 359 361 +2
Lines 21260 21488 +228
==========================================
+ Hits 5076 5080 +4
- Misses 16184 16408 +224 ☔ View full report in Codecov by Sentry. |
# Distributed variables | ||
self.world_size, self.rank = utils.get_world_size_and_rank() | ||
self._is_rank_zero = self.rank == 0 | ||
self.parallelize_plan = config.instantiate(cfg.get("parallelize_plan")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI the default parallelize plan we have didn't work when I was using it with the smaller Llama3 models with tied embeddings.
No description provided.