Skip to content

Commit e2d8c27

Browse files
authored
[BugFix] Fix pipeline parallel (#24621)
Signed-off-by: Nick Hill <nhill@redhat.com>
1 parent 29799dd commit e2d8c27

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

vllm/executor/uniproc_executor.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,10 @@ def reinitialize_distributed(
7171
self.shutdown()
7272
return
7373

74+
def shutdown(self) -> None:
75+
if worker := self.driver_worker:
76+
worker.shutdown()
77+
7478

7579
UniProcExecutorAsync = UniProcExecutor
7680

vllm/v1/worker/gpu_model_runner.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2070,7 +2070,6 @@ def execute_model(
20702070
sampler_output = self._sample(logits, spec_decode_metadata)
20712071

20722072
with record_function_or_nullcontext("Bookkeep"):
2073-
assert isinstance(hidden_states, torch.Tensor)
20742073
(
20752074
num_nans_in_logits,
20762075
logprobs_lists,

vllm/v1/worker/kv_connector_model_runner_mixin.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ def maybe_setup_kv_connector(scheduler_output: "SchedulerOutput"):
4545

4646
@staticmethod
4747
def ensure_kv_transfer_shutdown() -> None:
48-
if has_kv_transfer_group():
48+
# has_kv_transfer_group can be None during interpreter shutdown.
49+
if has_kv_transfer_group and has_kv_transfer_group():
4950
ensure_kv_transfer_shutdown()
5051

5152
@staticmethod

0 commit comments

Comments
 (0)