skills/cuopt-numerical-optimization-api-python: update with benchmark details - #1332
Merged
ramakrishnap-nv merged 2 commits intoMay 29, 2026
Merged
Conversation
|
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. |
… details Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
ramakrishnap-nv
force-pushed
the
skills-cuopt-numerical-optimization-api-python
branch
from
May 29, 2026 01:17
98dcd66 to
52c285c
Compare
ramakrishnap-nv
marked this pull request as ready for review
May 29, 2026 01:17
ramakrishnap-nv
added
non-breaking
📝 WalkthroughWalkthroughAdds a new NVSkills evaluation report (BENCHMARK.md), updates the skill-card metadata and evaluation results, inserts a single blank line in SKILL.md front-matter, and replaces the Sigstore signature bundle file for the cuopt-numerical-optimization-api-python skill. ChangesSkill docs and signing updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Collaborator
Author
|
/nvskills-ci |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/cuopt-numerical-optimization-api-python/SKILL.md`:
- Line 16: This PR modifies
skills/cuopt-numerical-optimization-api-python/SKILL.md and must run pre-commit
across skills/ before merge: run `pre-commit run --all-files
--show-diff-on-failure`, apply and commit any fixes (ensuring updated files in
the same branch), then add a comment with `/nvskills-ci` on the PR so the
nv-nvskill-ci[bot] signature commit is produced and remains; confirm the final
commit(s) include the pre-commit changes and the bot signature.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 003350af-7546-4277-8d43-6e85396c04ad
📒 Files selected for processing (1)
skills/cuopt-numerical-optimization-api-python/SKILL.md
| - python | ||
| --- | ||
|
|
||
|
|
There was a problem hiding this comment.
❓ Verification inconclusive
🏁 Script executed:
#!/bin/bash
set -euo pipefail
FILE="skills/cuopt-numerical-optimization-api-python/SKILL.md"
echo "== File exists and line 1-40 =="
ls -l "$FILE"
nl -ba "$FILE" | sed -n '1,60p'
echo "== Diff for the file (if any) =="
if git rev-parse --is-inside-work-tree >/dev/null 2>&1; then
git --no-pager diff -- "$FILE" || true
echo
echo "== Current HEAD commit message =="
git --no-pager log -1 --format='%H%n%B' || true
else
echo "Not a git repo in this sandbox."
fi
echo "== pre-commit config presence =="
ls -la .pre-commit-config.yaml pre-commit-config.yaml .pre-commit-config.yml pre-commit-config.yml 2>/dev/null || true
echo "== Search for documentation about /nvskills-ci =="
rg -n "/nvskills-ci" -S . || true
echo "== Search for guidance mentioning pre-commit run --all-files =="
rg -n "pre-commit run --all-files" -S . || true
echo "== pre-commit docs or CONTRIBUTING references =="
for f in README.md CONTRIBUTING.md .github/CONTRIBUTING.md .github/workflows/* 2>/dev/null; do
if [ -f "$f" ]; then
echo "---- $f ----"
rg -n "pre-commit|nvskills|/nvskills-ci" -S "$f" || true
fi
doneRepository: NVIDIA/cuopt
Length of output: 255
🏁 Script executed:
#!/bin/bash
set -euo pipefail
FILE="skills/cuopt-numerical-optimization-api-python/SKILL.md"
echo "== File exists and show first 80 lines =="
ls -l "$FILE"
awk 'NR<=80 {printf "%6d\t%s\n", NR, $0}' "$FILE"
echo "== Check if this is a git repo (for HEAD commit message) =="
if command -v git >/dev/null 2>&1 && git rev-parse --is-inside-work-tree >/dev/null 2>&1; then
echo "Git repo: yes"
echo "== HEAD commit message (subject+body) =="
git --no-pager log -1 --format='%s%n%n%b' || true
else
echo "Git repo: no (or git not available)"
fi
echo "== pre-commit config presence =="
ls -la .pre-commit-config.yaml pre-commit-config.yaml .pre-commit-config.yml pre-commit-config.yml 2>/dev/null || true
echo "== Search for /nvskills-ci guidance =="
rg -n "/nvskills-ci" -S . || true
echo "== Search for guidance mentioning pre-commit run --all-files =="
rg -n "pre-commit run --all-files" -S . || true
echo "== Search for any CONTRIBUTING/README references =="
for f in README.md CONTRIBUTING.md .github/CONTRIBUTING.md .github/workflows/*; do
if [ -f "$f" ]; then
echo "---- $f ----"
rg -n "pre-commit|nvskills|/nvskills-ci" -S "$f" || true
fi
doneRepository: NVIDIA/cuopt
Length of output: 13305
Trigger NVSkills CI and show pre-commit evidence for this skills/ update
skills/cuopt-numerical-optimization-api-python/SKILL.md is under skills/ and requires: running pre-commit run --all-files --show-diff-on-failure (and committing any resulting fixes) and having /nvskills-ci commented on the PR so the nv-nvskill-ci[bot] signature commit is added and remains. The current commit already includes a Signed-off-by; please add/confirm the pre-commit and /nvskills-ci items in the PR.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@skills/cuopt-numerical-optimization-api-python/SKILL.md` at line 16, This PR
modifies skills/cuopt-numerical-optimization-api-python/SKILL.md and must run
pre-commit across skills/ before merge: run `pre-commit run --all-files
--show-diff-on-failure`, apply and commit any fixes (ensuring updated files in
the same branch), then add a comment with `/nvskills-ci` on the PR so the
nv-nvskill-ci[bot] signature commit is produced and remains; confirm the final
commit(s) include the pre-commit changes and the bot signature.
Signed-off-by: nvskills-svc-account <svc-nvskills-signing@nvidia.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/cuopt-numerical-optimization-api-python/skill-card.md`:
- Around line 62-64: Add a single blank line between the preceding heading and
the evaluation table that begins with the row starting "| Dimension | Num |
`claude-code` | `codex` |" in skill-card.md so the table is not immediately
adjacent to the heading (this satisfies markdownlint MD058); edit the content
around the heading/table and insert one empty line before that table to fix the
lint error and then run pre-commit (pre-commit run --all-files
--show-diff-on-failure) to verify formatting.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 12fa51d8-d0e7-4de7-8a34-25de31e884fc
📒 Files selected for processing (3)
skills/cuopt-numerical-optimization-api-python/BENCHMARK.mdskills/cuopt-numerical-optimization-api-python/skill-card.mdskills/cuopt-numerical-optimization-api-python/skill.oms.sig
Comment on lines
+62
to
+64
| | Dimension | Num | `claude-code` | `codex` | | ||
| |---|---:|---:|---:| | ||
| | Security | 2 | 100% (+0%) | 100% (+0%) | |
There was a problem hiding this comment.
Add a blank line before the evaluation table to satisfy markdownlint MD058.
The table starts immediately after the heading and triggers the reported lint warning.
Proposed fix
## Evaluation Results: <br>
+
| Dimension | Num | `claude-code` | `codex` |
|---|---:|---:|---:|
| Security | 2 | 100% (+0%) | 100% (+0%) |As per coding guidelines Use pre-commit run --all-files --show-diff-on-failure to check code formatting and linting on all files before committing.
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 62-62: Tables should be surrounded by blank lines
(MD058, blanks-around-tables)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@skills/cuopt-numerical-optimization-api-python/skill-card.md` around lines 62
- 64, Add a single blank line between the preceding heading and the evaluation
table that begins with the row starting "| Dimension | Num | `claude-code` |
`codex` |" in skill-card.md so the table is not immediately adjacent to the
heading (this satisfies markdownlint MD058); edit the content around the
heading/table and insert one empty line before that table to fix the lint error
and then run pre-commit (pre-commit run --all-files --show-diff-on-failure) to
verify formatting.
2 tasks
ramakrishnap-nv
deleted the
skills-cuopt-numerical-optimization-api-python
branch
July 9, 2026 18:34
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
update with benchmark details