Commit 3b9541c
committed
refactor: move NTP config to separate NonuniformTPConfig class
- Create NonuniformTPConfig dataclass in nonuniform_tp.py
- Remove NTP fields from DistributedDataParallelConfig (non-intrusive)
- Update all NTP functions/classes to use NonuniformTPConfig
- Update all tests to use NonuniformTPConfig
- Update CLAUDE.md documentation
This makes the NTP implementation completely self-contained with
zero modifications to core Megatron files.1 parent 3853619 commit 3b9541c
File tree
4 files changed
+132
-107
lines changed- megatron/core/distributed
- tests/unit_tests/distributed
4 files changed
+132
-107
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
25 | 28 | | |
26 | | - | |
27 | | - | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
| 29 | + | |
34 | 30 | | |
35 | 31 | | |
36 | 32 | | |
| |||
44 | 40 | | |
45 | 41 | | |
46 | 42 | | |
47 | | - | |
| 43 | + | |
48 | 44 | | |
49 | 45 | | |
50 | | - | |
| 46 | + | |
51 | 47 | | |
52 | 48 | | |
53 | 49 | | |
| |||
Lines changed: 0 additions & 22 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
157 | 157 | | |
158 | 158 | | |
159 | 159 | | |
160 | | - | |
161 | | - | |
162 | | - | |
163 | | - | |
164 | | - | |
165 | | - | |
166 | | - | |
167 | | - | |
168 | | - | |
169 | | - | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | | - | |
175 | | - | |
176 | | - | |
177 | | - | |
178 | | - | |
179 | | - | |
180 | | - | |
181 | | - | |
182 | 160 | | |
183 | 161 | | |
184 | 162 | | |
| |||
0 commit comments