Skip to content

Commit 06436ad

Browse files
committed
Rename mode to send_mode in WPI weight transfer engine and example
1 parent e3e0ad8 commit 06436ad

2 files changed

Lines changed: 13 additions & 13 deletions

File tree

examples/rl/rlhf_http_wpi.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ def main():
167167
print("Propagating weights via WPI...")
168168
param_iter = ((n, p) for n, p in train_model.named_parameters())
169169
args = WPITrainerSendWeightsArgs(
170-
mode="http",
170+
send_mode="http",
171171
url=BASE_URL,
172172
trainer_ctx=ctx,
173173
)

vllm/distributed/weight_transfer/wpi_engine.py

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ class WPITrainerContext:
6868
class WPITrainerSendWeightsArgs:
6969
"""Arguments for WPI trainer_send_weights method."""
7070

71-
mode: str
71+
send_mode: str
7272
"""Transport mode: 'http' or 'ray'."""
7373

7474
buffer_id: str = "vllm-weights"
@@ -98,18 +98,18 @@ class WPITrainerSendWeightsArgs:
9898

9999
# Transport-specific fields
100100
llm_handle: Any = None
101-
"""Ray ObjectRef to LLM handle (required for 'ray' mode)."""
101+
"""Ray ObjectRef to LLM handle (required for 'ray' send_mode)."""
102102

103103
url: str | None = None
104-
"""Base URL for HTTP endpoint (required for 'http' mode)."""
104+
"""Base URL for HTTP endpoint (required for 'http' send_mode)."""
105105

106106
def __post_init__(self):
107-
if self.mode == "ray" and self.llm_handle is None:
108-
raise ValueError("llm_handle is required for 'ray' mode")
109-
if self.mode == "http" and self.url is None:
110-
raise ValueError("url is required for 'http' mode")
111-
if self.mode not in ("ray", "http"):
112-
raise ValueError(f"mode must be 'ray' or 'http', got {self.mode}")
107+
if self.send_mode == "ray" and self.llm_handle is None:
108+
raise ValueError("llm_handle is required for 'ray' send_mode")
109+
if self.send_mode == "http" and self.url is None:
110+
raise ValueError("url is required for 'http' send_mode")
111+
if self.send_mode not in ("ray", "http"):
112+
raise ValueError(f"send_mode must be 'ray' or 'http', got {self.send_mode}")
113113

114114

115115
@dataclass
@@ -484,7 +484,7 @@ def trainer_send_weights(
484484
>>> # Per-step weight sync
485485
>>> param_iter = ((n, p) for n, p in model.named_parameters())
486486
>>> args = WPITrainerSendWeightsArgs(
487-
... mode="http",
487+
... send_mode="http",
488488
... url="http://vllm-server:8000",
489489
... trainer_ctx=ctx,
490490
... )
@@ -570,14 +570,14 @@ def trainer_send_weights(
570570
total_bytes=total_bytes,
571571
))
572572

573-
if args.mode == "ray":
573+
if args.send_mode == "ray":
574574
import ray
575575
ray.get(
576576
args.llm_handle.update_weights.remote(
577577
dict(update_info=update_info)
578578
)
579579
)
580-
elif args.mode == "http":
580+
elif args.send_mode == "http":
581581
import requests
582582
url = f"{args.url}/update_weights"
583583
payload = {"update_info": update_info}

0 commit comments

Comments
 (0)