Skip to content

[DYNAMO] pass renderer transport to verifiers#2420

Draft
AmeenP wants to merge 5 commits intofeat/dynamo-admin-apifrom
feat/dynamo-renderer-transport-config
Draft

[DYNAMO] pass renderer transport to verifiers#2420
AmeenP wants to merge 5 commits intofeat/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_chat_nvext" to verifiers renderer clients when client.backend = "dynamo", while preserving prime_vllm_generate for the default vLLM path.

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

Pins prime-rl to the rebased verifiers renderer-transport PR head (1c7c35f0) and adds a direct standalone renderers source pin (PrimeIntellect-ai/renderers@7ca1ab3, PR #11). This removes the old verifiers/packages/renderers path from the lockfile.

Stacked on #2398.

Companion PRs:

Validation

  • uv lock --check
  • uvx ruff==0.13.0 check src/prime_rl/utils/client.py src/prime_rl/utils/elastic.py packages/prime-rl-configs/src/prime_rl/configs/shared.py tests/unit/utils/test_client.py tests/unit/utils/test_elastic.py
  • uvx ruff==0.13.0 format --check src/prime_rl/utils/client.py src/prime_rl/utils/elastic.py packages/prime-rl-configs/src/prime_rl/configs/shared.py tests/unit/utils/test_client.py tests/unit/utils/test_elastic.py
  • python -m py_compile src/prime_rl/utils/client.py src/prime_rl/utils/elastic.py packages/prime-rl-configs/src/prime_rl/configs/shared.py tests/unit/utils/test_client.py tests/unit/utils/test_elastic.py

Full pytest is left to Linux CI for this stack because the local checkout is macOS while the lockfile targets Linux environments.

@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