Skip to content

Add example batch runner and shared config - #349

Closed
ak495867 wants to merge 1 commit into
NVIDIA:mainfrom
ak495867:main
Closed

ak495867 wants to merge 1 commit into
NVIDIA:mainfrom
ak495867:main

Conversation

@ak495867

Copy link
Copy Markdown

This change adds a batch runner for the Cosmos3 NIM examples, centralizes generator defaults in examples/settings.py, and updates the example scripts to use shared request-building helpers, output handling, and CLI overrides. It also adds offline tests and a request snapshot contract to catch drift between scripts and their generated requests, while documenting the full suite and runtime/profile checks in the README.

This change adds a batch runner for the Cosmos3 NIM examples, centralizes generator defaults in examples/settings.py, and updates the example scripts to use shared request-building helpers, output handling, and CLI overrides. It also adds offline tests and a request snapshot contract to catch drift between scripts and their generated requests, while documenting the full suite and runtime/profile checks in the README.
@yaoxu-crypto

Copy link
Copy Markdown
Collaborator

Thank you for this, @ak495867. A shared config, a batch runner and request-snapshot tests are useful ideas for the NIM examples.

We're closing this PR because it can no longer merge here. The NIM cookbook moved to NVIDIA/cosmos-framework (.agents/skills/cookbooks/cosmos3/nim/) and was removed from this repo in #384. Since then it has been updated to NIM 2.1.0 (cosmos-framework#296) and gained new examples (reasoner.py, streaming.py, policy_droid.py, inspect_profile.py), so the example scripts this PR refactors have changed.

If you'd like to continue, please open an issue on NVIDIA/cosmos-framework first, so the NIM cookbook owners can agree on the shape before you rebase about 3,000 lines. Splitting it would also make review much easier: the shared settings and helpers first, then the batch runner, then the tests.

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