Skip to content

skills(cuopt-numerical-optimization-formulation): update for mathematical_optimization namespace refactor#1479

Merged
ramakrishnap-nv merged 2 commits into
mainfrom
skills/update-cuopt-numerical-optimization-formulation-math-opt-namespace
Jun 26, 2026
Merged

skills(cuopt-numerical-optimization-formulation): update for mathematical_optimization namespace refactor#1479
ramakrishnap-nv merged 2 commits into
mainfrom
skills/update-cuopt-numerical-optimization-formulation-math-opt-namespace

Conversation

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator

Update cuopt-numerical-optimization-formulation skill content for the mathematical_optimization namespace refactor from #1446.

Summary

NVSkills CI

Comment /nvskills-ci to trigger evaluation and signing.

…ical_optimization namespace refactor

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ramakrishnap-nv ramakrishnap-nv requested a review from a team as a code owner June 26, 2026 16:17
@ramakrishnap-nv ramakrishnap-nv requested a review from Iroy30 June 26, 2026 16:17
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Updates the cuOpt numerical optimization skill metadata, benchmark dates and metrics, and the signed skill bundle. One Markdown file also keeps a blank line adjustment.

Changes

Skill metadata and benchmark refresh

Layer / File(s) Summary
Documentation and benchmark updates
skills/cuopt-numerical-optimization-formulation/SKILL.md, skills/cuopt-numerical-optimization-formulation/BENCHMARK.md, skills/cuopt-numerical-optimization-formulation/skill-card.md
The skill description, requirements, references, output text, evaluation wording, benchmark date, and benchmark result values were updated.
Signed bundle refresh
skills/cuopt-numerical-optimization-formulation/skill.oms.sig
The embedded DSSE/in-toto payload metadata and the DSSE signature value were replaced.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • NVIDIA/cuopt#1398: Updates the same cuopt numerical optimization skill artifacts, including benchmark and signature data.
  • NVIDIA/cuopt#1453: Also changes the same skill package files and refreshed signature payload content.

Suggested reviewers

  • Iroy30
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the main change: updating the cuOpt skill for the mathematical_optimization namespace refactor.
Description check ✅ Passed The description is relevant to the changeset and explains the skill update, porting, and NVSkills evaluation trigger.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch skills/update-cuopt-numerical-optimization-formulation-math-opt-namespace

Comment @coderabbitai help to get the list of available commands.

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/nvskills-ci

@ramakrishnap-nv ramakrishnap-nv self-assigned this Jun 26, 2026
@ramakrishnap-nv ramakrishnap-nv added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels Jun 26, 2026
Signed-off-by: nvskills-svc-account <svc-nvskills-signing@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jun 26, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/ok to test 471b35a

@github-actions

Copy link
Copy Markdown

CI Test Summary

✅ All 0 test job(s) passed.

@ramakrishnap-nv ramakrishnap-nv merged commit 1221b24 into main Jun 26, 2026
22 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants