Skip to content

[DYNAMO] pass renderer transport to verifiers#2420

Draft
AmeenP wants to merge 7 commits into
feat/dynamo-admin-apifrom
feat/dynamo-renderer-transport-config
Draft

[DYNAMO] pass renderer transport to verifiers#2420
AmeenP wants to merge 7 commits into
feat/dynamo-admin-apifrom
feat/dynamo-renderer-transport-config

Conversation

@AmeenP
Copy link
Copy Markdown
Contributor

@AmeenP AmeenP commented May 5, 2026

Summary

Passes renderer_transport="dynamo" to verifiers renderer clients when client.backend = "dynamo", while preserving vllm for the default vLLM path.

Preserves backend when ElasticInferencePool rebuilds URL-scoped client configs, so elastic Dynamo renderer clients keep the same transport selection.

Updates the shared Dynamo renderer stack contract:

Stacked on #2398.

Companion PRs:

Validation

Local:

  • uv lock --check
  • uvx ruff==0.13.0 check --config=pyproject.toml src/prime_rl/utils/client.py tests/unit/utils/test_client.py
  • uvx ruff==0.13.0 format --check --config=pyproject.toml src/prime_rl/utils/client.py tests/unit/utils/test_client.py

GitHub Actions:

  • Ruff: pass
  • Slim install (prime-rl-configs only): pass
  • Unit tests: pass

Local test note:

  • uv run pytest tests/unit/utils/test_client.py is blocked on macOS by the repo's Linux-only lockfile; uv run --no-sync pytest tests/unit/utils/test_client.py exited 139 locally.

@AmeenP AmeenP force-pushed the feat/dynamo-admin-api branch from 52aac5c to f63aab5 Compare May 8, 2026 10:03
@AmeenP AmeenP force-pushed the feat/dynamo-renderer-transport-config branch from 9cad934 to 360f0c1 Compare May 8, 2026 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant