Skip to content

Update Model-Optimizer skill count from 4 to 5#20

Merged
mosheabr merged 1 commit intoNVIDIA:mainfrom
sayalinvidia:fix/model-optimizer-count
Apr 16, 2026
Merged

Update Model-Optimizer skill count from 4 to 5#20
mosheabr merged 1 commit intoNVIDIA:mainfrom
sayalinvidia:fix/model-optimizer-count

Conversation

@sayalinvidia
Copy link
Copy Markdown
Collaborator

@sayalinvidia sayalinvidia commented Apr 16, 2026

Summary

  • Model-Optimizer repo added a 5th skill
  • Updates the count in both README.md and ci.yml to unblock all open PRs

Test plan

  • ci.yml check passes on this PR

Model-Optimizer repo added a 5th skill, causing ci.yml drift check
to fail on all open PRs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sayalinvidia sayalinvidia requested a review from mosheabr as a code owner April 16, 2026 01:02
@sayalinvidia sayalinvidia self-assigned this Apr 16, 2026
@mosheabr
Copy link
Copy Markdown
Collaborator

LGTM — verified Model-Optimizer source repo has exactly 5 skill directories (common, debug, deployment, evaluation, ptq). Count update is correct in both README.md and ci.yml.

Note: CI failure appears unrelated to this change — likely a PAT/fork permission issue or drift in other products. This fix is needed to unblock PR #21 (automated sync).

Copy link
Copy Markdown
Collaborator

@mosheabr mosheabr left a comment

Choose a reason for hiding this comment

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

LGTM — verified Model-Optimizer source repo has exactly 5 skill directories (common, debug, deployment, evaluation, ptq). Count update is correct in both README.md and ci.yml. Note: CI failure appears unrelated to this change — likely a PAT/fork permission issue or drift in other products. This fix is needed to unblock PR #21 (automated sync).

@mosheabr mosheabr merged commit acc0d87 into NVIDIA:main Apr 16, 2026
1 of 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