Skip to content

chore: publish Megatron-Core mcore skills#166

Merged
sayalinvidia merged 2 commits into
NVIDIA:mainfrom
Phlip79:chore/megatron-core-flat-skills
May 29, 2026
Merged

chore: publish Megatron-Core mcore skills#166
sayalinvidia merged 2 commits into
NVIDIA:mainfrom
Phlip79:chore/megatron-core-flat-skills

Conversation

@Phlip79
Copy link
Copy Markdown
Member

@Phlip79 Phlip79 commented May 29, 2026

Onboarding type

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

This converts the existing Megatron-Core component (components.d/megatron-core.yml) from the bulk-sync pattern (single path: skills/ + catalog_dir: Megatron-Core) to the flat, explicit one-entry-per-skill pattern used by components.d/nemo-mbridge.yml and components.d/nemo-automodel.yml. It registers the signed mcore-* skills from NVIDIA/Megatron-LM for the automated catalog sync without manually adding catalog skill content in this PR.

The component slug and display name are unchanged (Megatron-Core) — there is no rebrand, and the mcore-* skills already belong to this component. This is the Megatron-LM analog of #151 (which performed the same bulk → flat conversion for Megatron-Bridge, additionally renaming to NeMo MBridge).

Reviewer checklist (OSS Skills PIC)

  • components.d/megatron-core.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).

Other context (for non-onboarding PRs)

Changes:

  • Replace the single bulk skills: entry in components.d/megatron-core.yml with explicit entries, one per signed skill, each using path: skills/<catalog_dir>/ and a flat catalog_dir.
  • Keep name: Megatron-Core, repo: NVIDIA/Megatron-LM, and links.security: false unchanged.

Skills registered (from NVIDIA/Megatron-LM#5066, assumed merged):

  • mcore-create-issue
  • mcore-linting-and-formatting
  • mcore-run-on-slurm
  • mcore-split-pr
  • mcore-testing

This PR intentionally does not add files under skills/ or update README.md; those are generated by the sync workflow after the component entry merges.

Validation run locally:

  • Parsed components.d/megatron-core.yml and verified 5 entries.
  • Verified every entry uses path: skills/<catalog_dir>/ and all catalog_dir values are unique.

🤖 Generated with Claude Code

Replace the bulk Megatron-Core component entry (single `path: skills/`
+ `catalog_dir: Megatron-Core`) with the flat, one-entry-per-skill
pattern used by components.d/nemo-mbridge.yml and nemo-automodel.yml.

Registers the 9 signed mcore skills from NVIDIA/Megatron-LM#5066 for the
automated catalog sync:
- mcore-build-and-dependency
- mcore-bump-base-image
- mcore-cicd
- mcore-create-issue
- mcore-linting-and-formatting
- mcore-onboard-gb200-1node-tests
- mcore-run-on-slurm
- mcore-split-pr
- mcore-testing

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
Remove mcore-build-and-dependency, mcore-bump-base-image, mcore-cicd,
and mcore-onboard-gb200-1node-tests, leaving 5 registered skills.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Philip Petrakian <ppetrakian@nvidia.com>
Copy link
Copy Markdown
Collaborator

@sayalinvidia sayalinvidia left a comment

Choose a reason for hiding this comment

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

LGTM!

@sayalinvidia sayalinvidia merged commit 09caf15 into NVIDIA:main May 29, 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.

2 participants