Skip to content

Commit

Permalink
Fix changed behavior of pipe_parallel
Browse files Browse the repository at this point in the history
  • Loading branch information
yang committed May 16, 2024
1 parent 49cd41f commit a57bf6d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion megatron/neox_arguments/arguments.py
Original file line number Diff line number Diff line change
Expand Up @@ -1035,7 +1035,7 @@ def calculate_derived(self):
# the sequential model without the PipelineModule wrapper to avoid the overhead it incurs
self.update_value(
"is_pipe_parallel",
self.pipe_parallel_size > 1 and self.moe_num_experts == 1,
self.pipe_parallel_size >= 1 and self.moe_num_experts == 1,
)
if self.moe_num_experts > 1:
assert not (
Expand Down
4 changes: 2 additions & 2 deletions requirements/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
git+https://github.com/EleutherAI/DeeperSpeed.git@02e2ebf7dee6aaab3d89094ed470a4609763c742#egg=deepspeed
deepspeed@git+https://github.com/EleutherAI/DeeperSpeed.git@02e2ebf7dee6aaab3d89094ed470a4609763c742#egg=deepspeed
ftfy>=6.0.1
git+https://github.com/EleutherAI/lm_dataformat.git@4eec05349977071bf67fc072290b95e31c8dd836
lm_dataformat@git+https://github.com/EleutherAI/lm_dataformat.git@4eec05349977071bf67fc072290b95e31c8dd836
huggingface_hub>=0.11.0
jinja2==3.1.4
lm_eval>=0.4.0,<=0.4.1
Expand Down

0 comments on commit a57bf6d

Please sign in to comment.