Skip to content

Commit c0da5bd

Browse files
nnshah1claude
andcommitted
fix: always set kv_port in configure_ports to prevent TypeError
The overwrite_args function always uses config.kv_port regardless of prefix caching settings, so configure_ports must always set this value to prevent "NoneType and int" TypeError. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent ac7e774 commit c0da5bd

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

examples/multimodal/utils/args.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,8 +139,8 @@ def ensure_side_channel_host():
139139
def configure_ports(config: Config):
140140
"""Configure port settings from dedicated environment overrides."""
141141

142-
if config.engine_args.enable_prefix_caching:
143-
config.kv_port = get_kv_port()
142+
# Always set kv_port as it's used by overwrite_args regardless of prefix caching
143+
config.kv_port = get_kv_port()
144144

145145
ensure_side_channel_host()
146146

0 commit comments

Comments
 (0)