Skip to content

Commit

Permalink
fix: use clone instead of deepcopy for params
Browse files Browse the repository at this point in the history
Signed-off-by: Travis Johnson <[email protected]>
  • Loading branch information
tjohnson31415 committed Dec 20, 2024
1 parent d021f0f commit 66e4127
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions vllm/sequence.py
Original file line number Diff line number Diff line change
Expand Up @@ -1366,9 +1366,9 @@ def maybe_assemble_group(
class ParallelSampleSequenceGroup(SequenceGroupBase):

@staticmethod
def add_request(request_id: str, engine, params, **kwargs):
def add_request(request_id: str, engine, params: Union[SamplingParams, PoolingParams], **kwargs):

Check failure on line 1369 in vllm/sequence.py

View workflow job for this annotation

GitHub Actions / ruff (3.12)

Ruff (E501)

vllm/sequence.py:1369:81: E501 Line too long (101 > 80)
original_params = params
params = copy.deepcopy(original_params)
params = original_params.clone()
params.n = 1
group = ParallelSampleSequenceGroup(request_id)
seqs = []
Expand Down

0 comments on commit 66e4127

Please sign in to comment.