Skip to content

TAO Toolkit: add 2 missing skills to catalog (paidf-anomalygen, tao-run-on-docker)#318

Open
christinayyw wants to merge 1 commit into
NVIDIA:mainfrom
christinayyw:chore/tao-add-2-missing-skills
Open

TAO Toolkit: add 2 missing skills to catalog (paidf-anomalygen, tao-run-on-docker)#318
christinayyw wants to merge 1 commit into
NVIDIA:mainfrom
christinayyw:chore/tao-add-2-missing-skills

Conversation

@christinayyw

Copy link
Copy Markdown

Two TAO skills exist in the source repo (NVIDIA-TAO/tao-skills-bank) but were never listed in components.d/tao-toolkit.yml, so the catalog sync never picked them up:

  • skills/data/paidf-anomalygen/paidf-anomalygen
  • skills/platform/tao-run-on-docker/tao-run-on-docker

Both are present, signed (skill.oms.sig), and carded (skill-card.md) in the source. This adds their catalog entries so they sync like the other 56 TAO Toolkit skills.

🤖 Generated with Claude Code

@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.

Thanks @christinayyw — good catch on the two un-listed skills. The yml structure is correct (right paths, correct sections, no catalog_dir collisions). paidf-anomalygen is fully ready at source (SKILL.md + skill.oms.sig + skill-card.md + evals). Two things to fix before merge:

  1. DCO sign-off is failing. Commit 63430864 has no Signed-off-by: trailer. Fix:

    git rebase --signoff origin/main && git push --force-with-lease
    
  2. tao-run-on-docker isn't fully signed at source yet. In NVIDIA-TAO/tao-skills-bank, skills/platform/tao-run-on-docker/ currently has only SKILL.md + evals/ — it's missing skill.oms.sig, skill-card.md, and BENCHMARK.md (compare tao-run-on-brev, which has all of them). Without the signature, the sync's enforcement will drop it on every run. Please run /nvskills-ci on that skill in the source repo to generate the sig + card, then it's good.

    Heads-up: the content-integrity CI check here is green, but that's expected — it only validates skills already present in the catalog, not source artifacts for a skill that hasn't synced yet. So it won't catch this gap.

paidf-anomalygen is ready to go — once DCO is signed and tao-run-on-docker is signed/carded at source, this is good to merge. (If you'd rather land paidf-anomalygen now and do tao-run-on-docker in a follow-up, I'm happy to split it — just say the word.)

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