Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Parse seed for vLLM #602

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

eldarkurtic
Copy link
Contributor

When seed is set through model_args, for example:

"pretrained=deepseek-ai/DeepSeek-R1-Distill-Qwen-1.5B,seed=123,dtype=bfloat16,max_model_length=38768,gpu_memory_utilization=0.8,tensor_parallel_size=1"

it is parsed as a string attribute in VLLMModelConfig, leading to an error during LLM initialization:

model = LLM(**self.model_args)

This PR ensures that seed is correctly cast to an integer before passing it to the model, preventing initialization errors.

When seed is set through model_args, for example:

```bash
"pretrained=deepseek-ai/DeepSeek-R1-Distill-Qwen-1.5B,seed=123,dtype=bfloat16,max_model_length=38768,gpu_memory_utilization=0.8,tensor_parallel_size=1"
``

it is parsed as a string attribute in `VLLMModelConfig`, leading to an error during LLM initialization:

```python
model = LLM(**self.model_args)
```

This PR ensures that seed is correctly cast to an integer before passing it to the model, preventing initialization errors.
@LHB-kk
Copy link

LHB-kk commented Mar 12, 2025

Same problem!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants