Skip to content

feat(nemo-platform): onboard NeMo Platform skills - #120

Merged
mosheabr merged 4 commits into
NVIDIA:mainfrom
ngoncharenko:ngoncharenko/nemo-platform-onboarding
May 30, 2026
Merged

feat(nemo-platform): onboard NeMo Platform skills#120
mosheabr merged 4 commits into
NVIDIA:mainfrom
ngoncharenko:ngoncharenko/nemo-platform-onboarding

Conversation

@ngoncharenko

@ngoncharenko ngoncharenko commented May 28, 2026

Copy link
Copy Markdown
Contributor

Onboarding type

  • New product onboarding (new components.d/<slug>.yml file)
  • Other (catalog change, README fix, infrastructure, etc.)

For new product onboarding — author affirmations

By submitting this PR, I confirm on behalf of my team:

  • Skills cleared for open source release per NVIDIA's internal IP review process (six-question check, all answers affirmative)
  • License selected: Apache 2.0 / CC-BY 4.0 / Dual (Apache 2.0 + CC-BY 4.0). Specify: Apache 2.0
  • No new license or new third-party component introduced beyond what the source repo already carries
  • Source repo is public and under an NVIDIA-owned GitHub org
  • .agents/skills/ or skills/ path used for new entries (or existing path retained for legacy entries per components.d/<slug>.yml)

NVIDIA contributors: see the internal onboarding guide for the IP review process details and license selection.

Reviewer checklist (OSS Skills PIC)

  • Author confirmations above are checked
  • components.d/<slug>.yml entry valid (required fields, unique catalog_dir, path exists in source repo, filename slug matches name)
  • SKILL.md frontmatter spec-compliant (at least one sampled)
  • No new license or third-party dependency requiring OSRB filing

All PRs

  • All commits signed off with DCO (git commit -s).
    If you forgot, run git rebase --signoff origin/main && git push --force-with-lease to retroactively sign all commits in your branch.

Other context (for non-onboarding PRs)

Dependency

@calliente

calliente commented May 30, 2026

Copy link
Copy Markdown
Collaborator

eval/evals.json are in the PR -- awaiting fixes then will push and merge. NVIDIA-NeMo/nemo-platform#116

The onboarding config looks fine to me

Signed-off-by: Nick Goncharenko <ngoncharenko@nvidia.com>
Signed-off-by: Nick Goncharenko <ngoncharenko@nvidia.com>
Evaluator and data-designer entries now use skills/nemo-*-plugin/
symlinks in nemo-platform instead of deep plugin package paths.

Signed-off-by: Nick Goncharenko <ngoncharenko@nvidia.com>
Signed-off-by: Nick Goncharenko <ngoncharenko@nvidia.com>
@ngoncharenko
ngoncharenko force-pushed the ngoncharenko/nemo-platform-onboarding branch from 3bc2645 to 661c2ed Compare May 30, 2026 02:44

@mosheabr mosheabr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving. Both skills now carry the full 5-artifact set on source repo (SKILL.md + skill-card.md + skill.oms.sig + evals/ + BENCHMARK.md). Flat-layout entry, paths exist, DCO + Verify Authors pass. nemo-setup correctly removed.

@mosheabr
mosheabr merged commit b7d99cf into NVIDIA:main May 30, 2026
2 checks passed
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.

3 participants