Skip to content

add engine_kwargs in vllm_rollout_spmd to set params in vllm engine #1442

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 22, 2025

Conversation

zhangyongxin121
Copy link
Contributor

@zhangyongxin121 zhangyongxin121 commented May 8, 2025

Checklist Before Starting

  • Search for similar PR(s).

What does this PR do?

add engine_kwargs in vllm_rollout_spmd to set swap_space in vllm engine.

Specific Changes

add engine_kwargs in vllm_rollout_spmd, which can be set in config file. Same changes has been made in vllm_rollout. As the version of vllm is update to 0.8, the default vllm_rollout worker becomes vllm_rollout_spmd, which does not have engine_kwargs as in vllm_rollout, so this RP want to complete it.

Usage Example

users can set vllm engine param such as swap_space, seed through the engine_kwargs in config file. For example, if one want to set the swap_space=32 in vllm, he can set the item in config like this

actor_rollout_ref.rollout.engine_kwargs.swap_space=32

@zhangyongxin121 zhangyongxin121 changed the title add engine_kwargs in vllm_rollout_spmd to set swap_space in vllm add engine_kwargs in vllm_rollout_spmd to set params in vllm engine May 8, 2025
@ZihengJiang
Copy link
Collaborator

@eric-haibin-lin @vermouth1992 shall we merge?

@eric-haibin-lin eric-haibin-lin merged commit c07013e into volcengine:main May 22, 2025
26 checks passed
yellowbee686 pushed a commit to yellowbee686/verl that referenced this pull request May 22, 2025
…n vllm engine (volcengine#1442)

### Checklist Before Starting

- [x] Search for similar PR(s).

### What does this PR do?

> add `engine_kwargs` in vllm_rollout_spmd to set swap_space in vllm
engine.

### Specific Changes

> add `engine_kwargs` in vllm_rollout_spmd, which can be set in config
file. Same changes has been made in vllm_rollout. As the version of vllm
is update to 0.8, the default vllm_rollout worker becomes
vllm_rollout_spmd, which does not have `engine_kwargs` as in
vllm_rollout, so this RP want to complete it.

### Usage Example

> users can set vllm engine param such as `swap_space`, `seed` through
the `engine_kwargs` in config file. For example, if one want to set the
swap_space=32 in vllm, he can set the item in config like this
```bash
actor_rollout_ref.rollout.engine_kwargs.swap_space=32
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants