Skip to content

chore: nemo-gym-pivot-datasets skill: NVSkills integration#1448

Open
ananthsub wants to merge 4 commits into
mainfrom
ci/nvskills-pivot-datasets
Open

chore: nemo-gym-pivot-datasets skill: NVSkills integration#1448
ananthsub wants to merge 4 commits into
mainfrom
ci/nvskills-pivot-datasets

Conversation

@ananthsub
Copy link
Copy Markdown
Contributor

Summary

  • Add license: Apache-2.0 to the frontmatter of .claude/skills/nemo-gym-pivot-datasets/SKILL.md.
  • Add .claude/skills/nemo-gym-pivot-datasets/evals/evals.json with 3 positive trigger cases (convert tool-call trajectories, validate with --gym-repo, chat-completion + config) and 2 negative cases that delegate to sibling skills.

Motivation

Prepares the nemo-gym-pivot-datasets skill for NVSkills CI signing.

Test plan

  • After dependencies merge, comment /nvskills-ci on this PR. Expect a signature commit attaching skill-card.md and skill.oms.sig under .claude/skills/nemo-gym-pivot-datasets/.

@ananthsub ananthsub requested review from a team, cwing-nvidia and jkyi-nvidia May 28, 2026 18:41
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 28, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

## Summary

- Move `.claude/skills/nemo-gym-pivot-datasets/` to top-level
  `skills/nemo-gym-pivot-datasets/` so this PR touches files under
  the central `team-request.yml` trigger allowlist (`skills/`,
  `team-skills/`, `rules/team-rules/`, `plugins/`).
- Replace `.claude/skills/nemo-gym-pivot-datasets/` with a symlink to
  `../../skills/nemo-gym-pivot-datasets` so Claude Code and Cursor
  continue to discover the skill via the conventional
  `.claude/skills/<name>/SKILL.md` path with no tool-side change.
- Add `license: Apache-2.0` to
  `skills/nemo-gym-pivot-datasets/SKILL.md` frontmatter.
- Add `skills/nemo-gym-pivot-datasets/evals/evals.json` with positive
  trigger cases (build pivot from rollouts, single-step tool-use
  configs, agent_ref alignment) and negative cases that delegate to
  sibling skills.

## Motivation

Prepares the `nemo-gym-pivot-datasets` skill for NVSkills CI signing.
Per-skill scope keeps the diff small and lets NVSkills CI evaluate
one skill at a time. Other skills remain at `.claude/skills/<name>/`
until each has its own migration PR.

## Test plan

- [ ] Comment `/nvskills-ci` on this PR. Expect the request workflow
      to dispatch (not skip) and `svc-nvskills-signing` to attach
      `skill-card.md` and `skill.oms.sig` under
      `skills/nemo-gym-pivot-datasets/`.
- [ ] Claude Code discovers `nemo-gym-pivot-datasets` via
      `.claude/skills/nemo-gym-pivot-datasets/SKILL.md` (follows symlink).

Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
@ananthsub ananthsub force-pushed the ci/nvskills-pivot-datasets branch from ef762fb to b78d327 Compare May 28, 2026 23:58
ananthsub added 3 commits May 29, 2026 09:03
Apply the same content fixes validated on add-benchmark:
- Add metadata.author and metadata.tags.
- Tighten the description and add negative triggers (not for general
  reward profiling or debugging runs).
- Add a Purpose section and an Examples section, and rename Core
  Workflow to Instructions.

Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
- Shorten the description to under 150 characters.
- Add Prerequisites, Limitations, and Troubleshooting sections.

Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
Add explicit routing in the invocation check so the skill defers to
nemo-gym-reward-profiling, nemo-gym-debugging, and add-benchmark for
adjacent tasks, reducing misrouted activations in agent evaluation.

Signed-off-by: Ananth Subramaniam <ansubramania@nvidia.com>
@ananthsub ananthsub marked this pull request as ready for review May 30, 2026 04:54
@ananthsub
Copy link
Copy Markdown
Contributor Author

/nvskills-ci

@ananthsub ananthsub changed the title chore: nemo-gym-pivot-datasets) skill: NVSkills integration chore: nemo-gym-pivot-datasets skill: NVSkills integration May 30, 2026
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