Skip to content

SDG data designer providers addition#197

Open
rapaul-nv wants to merge 3 commits into
NVIDIA-NeMo:romeyn/agenticfrom
rkalaniNV:rapaul/sdg-data-designer-providers
Open

SDG data designer providers addition#197
rapaul-nv wants to merge 3 commits into
NVIDIA-NeMo:romeyn/agenticfrom
rkalaniNV:rapaul/sdg-data-designer-providers

Conversation

@rapaul-nv
Copy link
Copy Markdown
Contributor

Summary

  • Add optional providers: support to sdg/data_designer so configs can register custom Data Designer model providers.
  • Allow models[].provider to target either a built-in provider such as nvidia or a custom provider declared in YAML.
  • Add a commented OpenAI-compatible endpoint example in customer_support_tools.yaml.
  • Add static tests for provider shape, model-provider references, and API key handling.

Notes

providers[].api_key should be an environment variable name such as OPENAI_API_KEY, not ${oc.env:OPENAI_API_KEY}, so Data Designer resolves the secret at request time instead of materializing it in the compiled config.

Validation

  • uv run pytest tests/steps/sdg/test_data_designer.py
  • uv run ruff check src/nemotron/steps/sdg/data_designer/step.py tests/steps/sdg/test_data_designer.py
  • Verified provider wiring locally: providers[].name is registered and models[].provider selects that provider.

rapaul-nv added 3 commits May 11, 2026 11:48
Signed-off-by: Rakesh Paul <rapaul@nvidia.com>
Signed-off-by: Rakesh Paul <rapaul@nvidia.com>
Signed-off-by: Rakesh Paul <rapaul@nvidia.com>
@rapaul-nv rapaul-nv force-pushed the rapaul/sdg-data-designer-providers branch from 58465ce to 321f71c Compare May 11, 2026 06:19
@rapaul-nv rapaul-nv changed the title Rapaul/sdg data designer providers SDG data designer providers addition May 11, 2026
@mikemckiernan
Copy link
Copy Markdown
Contributor

Thank you!

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.

2 participants