Skip to content

Commit 0988a7d

Browse files
authored
fix: Fix error message in VllmGenerationWorker. (#633)
Signed-off-by: Felipe Vieira Frujeri <ffrujeri@nvidia.com>
1 parent 233cc07 commit 0988a7d

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

nemo_rl/models/generation/vllm.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,8 @@ def _patch_vllm_init_workers_ray():
286286
raise ImportError(
287287
"vLLM is not installed. Please check that the py_executable in the runtime_env of VllmGenerationWorker "
288288
"covers the vllm dependency. You may have to update nemo_rl/distributed/ray_actor_environment_registry.py. "
289-
"If you are working interactively, you can install by running `uv sync --extra vllm` anywhere in the repo."
289+
"This error can also happen if the venv creation was aborted or errored out in the middle. In that case, "
290+
"please run at least once with the environment variable NRL_FORCE_REBUILD_VENVS=true set to force the rebuild of the environment."
290291
)
291292
vllm_kwargs: dict[str, Any] = copy.deepcopy(self.cfg.get("vllm_kwargs", {}))
292293

nemo_rl/models/generation/vllm_backend.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@
2323
raise ImportError(
2424
"vLLM is not installed. Please check that the py_executable in the runtime_env of VllmGenerationWorker "
2525
"covers the vllm dependency. You may have to update nemo_rl/distributed/ray_actor_environment_registry.py. "
26-
"If you are working interactively, you can install by running `uv sync --extra vllm` anywhere in the repo."
26+
"This error can also happen if the venv creation was aborted or errored out in the middle. In that case, "
27+
"please run at least once with the environment variable NRL_FORCE_REBUILD_VENVS=true set to force the rebuild of the environment."
2728
)
2829

2930

0 commit comments

Comments
 (0)