Skip to content

ci: nudge multi-objective skill to trigger eval #1392

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rapids-bot merged 2 commits into from
Jun 4, 2026
+171 −0
Merged

ci: nudge multi-objective skill to trigger eval #1392

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f0ad3b7
ci: nudge multi-objective skill to trigger eval
rgsl888prabhu Jun 4, 2026
d6eccab
Attach NVSkills validation signatures
svc-nvskills-signing Jun 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions skills/cuopt-multi-objective-exploration/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ metadata:

# Multi-Objective Exploration


@coderabbitai coderabbitai Bot Jun 4, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== Check DCO sign-off trailers on PR commits =="
git log --format='%H%n%B%n----' | awk '
BEGIN{ok=1; RS="----\n"}
{
  if ($0 ~ /^[0-9a-f]{40}\n/ && $0 !~ /Signed-off-by:[[:space:]]+/) {
    split($0, a, "\n"); print "Missing Signed-off-by on commit: " a[1]; ok=0
  }
}
END{ if (ok) print "All commits include Signed-off-by trailer." ; else exit 1 }'

echo
echo "== Check PR comments for /nvskills-ci trigger =="
# Requires gh auth in the verification environment
gh pr view 1392 --repo NVIDIA/cuopt --comments --json comments \
  --jq '.comments[].body' | rg -n '/nvskills-ci' || {
    echo "No /nvskills-ci comment found on PR `#1392`"; exit 1;
  }

Repository: NVIDIA/cuopt

Length of output: 178

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== Check PR comments for /nvskills-ci trigger =="
# Use gh to check PR comments for NVSkills CI validation
gh pr view 1392 --repo NVIDIA/cuopt --json comments --jq '.comments[].body' | grep -c '/nvskills-ci' > /dev/null && {
  echo "/nvskills-ci validation comment found"
} || {
  echo "No /nvskills-ci validation comment found on PR"
}

Repository: NVIDIA/cuopt

Length of output: 143

Add DCO sign-off to commits before merging.

Commit f0ad3b7 is missing the required Signed-off-by trailer. Amend all commits with git commit --amend -s and force-push to satisfy DCO policy for **/* files.

🤖 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-multi-objective-exploration/SKILL.md` at line 20, The commit
f0ad3b7218abb88f84b882da7fcd016e76775994 is missing the Signed-off-by trailer;
amend that commit (and any other commits in this branch affecting **/*) by
running git commit --amend -s for the most recent commit or use git rebase -i to
edit older commits, add the Signed-off-by line, then force-push the branch (git
push --force) to update the PR.

cuOpt optimizes **one** objective per solve. Many real problems have several objectives that pull against each other — cost vs. service level, return vs. risk, makespan vs. overtime, distance vs. vehicle count. A single solve answers "what's optimal *for one particular weighting*," but it hides the tradeoff the user actually needs to see.

This skill turns a sequence of single-objective cuOpt solves into a **Pareto frontier** — the set of solutions where you can't improve one objective without giving up another — and gives the discipline to read it. It adds no solver features; it orchestrates the LP / MILP / QP solves already covered by the formulation and API skills.
Expand Down
Loading