-
Notifications
You must be signed in to change notification settings - Fork 3k
Description
System Info
I find the _ROLLOUT_REGISTRY in line81 of verl/workers/rollout/base.py only has "vllm" + "async" pair but not "vllm" and "sync".
_ROLLOUT_REGISTRY = { ("vllm", "async"): "verl.workers.rollout.vllm_rollout.vLLMAsyncRollout", ("sglang", "async"): "verl.workers.rollout.sglang_rollout.sglang_rollout.ServerAdapter", }
And the vLLMRollout class in verl/workers/rollout/vllm_rollout/vllm_rollout_spmd.py has beed removed. So is the latest verl does not support the sync version of vllm rollout method?
Information
- The official example scripts
- My own modified scripts
Tasks
- An officially supported task in the
examplesfolder (such as GLUE/SQuAD, ...) - My own task or dataset (give details below)
Reproduction
I find the _ROLLOUT_REGISTRY in line81 of verl/workers/rollout/base.py only has "vllm" + "async" pair but not "vllm" and "sync".
_ROLLOUT_REGISTRY = { ("vllm", "async"): "verl.workers.rollout.vllm_rollout.vLLMAsyncRollout", ("sglang", "async"): "verl.workers.rollout.sglang_rollout.sglang_rollout.ServerAdapter", }
And the vLLMRollout class in verl/workers/rollout/vllm_rollout/vllm_rollout_spmd.py has beed removed. So is the latest verl does not support the sync version of vllm rollout method?
Expected behavior
Hope your help