Repository navigation
feat: add six PVA SDK skills - #784
Merged
Merged
Conversation
Signed-off-by: Venkat Balan <venkatramanb@nvidia.com>
nv-venkatramanb
requested review from
jasonnvidia,
mosheabr,
sayalinvidia and
siqinvidia
as code owners
October 28, 2026 17:41
The six PVA SDK skills are imported directly from an internal source repository, like the Physical AI skills, so they have no components.d entry. Register them in manual-components.yml so the sync keeps their directories, the README lists them under PVA SDK, and benchmark aggregation assigns them to that product. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Moshe Abramovitch <moshea@nvidia.com>
Collaborator
|
/nvskills-ci |
Signed-off-by: nvskills-svc-account <svc-nvskills-signing@nvidia.com>
mosheabr
approved these changes
Oct 28, 2026
mosheabr
left a comment
Collaborator
There was a problem hiding this comment.
Approved. Six PVA SDK skills imported directly from the internal source repo and registered as a manual component (same model as Physical AI). NVSkills CI passed on the final head: Tiers 1–3 PASS for all six skills, gate passed, and fresh signatures attached (e551ccb), each chaining to the NVIDIA Agent Capabilities root. OSRB review closed (bug 6882140). benchmarks.json / versions.json will be regenerated after merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds six PVA SDK skills to the catalog for setup, diagnostics, specification, operator generation, building/testing, and debugging:
skills/pva-build-test/skills/pva-debug/skills/pva-doctor/skills/pva-opgen/skills/pva-setup/skills/pva-spec/All 90 files are imported unchanged from
pva-agents/pva-sdk-skillsmain ata34b75386f77c600b3eb57a21da4fdc988151390. Each folder already contains its README with public installation instructions, supporting resources, evaluation data, benchmark report, skill card, and signature.The diff contains only these six folders.
benchmarks.jsonandversions.jsonare intentionally left for post-merge regeneration so version provenance reflects the final merged commit IDs. Catalog registration and workflow files are unchanged.Validation
SKILL.mdnames/descriptions pass frontmatter checks.--check --no-aiexits successfully, with enrichment deferred.git diff --checkpasses.Maintainer follow-up
The current PR workflow still checks that
benchmarks.jsonandversions.jsonare up to date; both read-only checks report drift with the generated files intentionally excluded. Please handle generated-index publication after merge and the corresponding PR-check requirements.These folders are not registered in
components.d/,manual-components.yml, or catalog exceptions. The catalog sync can prune unregistered folders, so retention needs maintainer handling separately from this six-folder import. NVSkills CI also requires a maintainer/admin to trigger/nvskills-ci; the signing account already has write access to the fork.Onboarding and author confirmations
skills/<skill-name>/paths.The source remains internal; acceptance of direct catalog maintenance requires maintainer review.