Skip to content

Conversation

@rapids-bot
Copy link
Contributor

@rapids-bot rapids-bot bot commented Nov 18, 2025

Forward-merge triggered by push to release/25.12 that creates a PR to keep main up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge. See forward-merger docs for more info.

<!--

Thank you for contributing to cuOpt :)

Here are some guidelines to help the review process go smoothly.

Many thanks in advance for your cooperation!

Note: The pull request title will be included in the CHANGELOG.
-->


## Description
This adds an option to run PRs in lean mode where it runs only for
amd64, one python version and a cuda version.

This is meant for EPIC PRs which need to do minimal CI runs to make sure
there are not major breaking issues.

## Issue
<!-- Add closes #ISSUE_NUMBER here, this would close the issue once PR
is merged, if there is no issue, please feel free to remove this section
-->

## Checklist

- [x] I am familiar with the [Contributing
Guidelines](https://github.com/NVIDIA/cuopt/blob/HEAD/CONTRIBUTING.md).
- Testing
   - [ ] New or existing tests cover these changes
   - [ ] Added tests
   - [ ] Created an issue to follow-up
   - [x] NA
- Documentation
   - [ ] The documentation is up to date with these changes
   - [ ] Added new documentation
   - [x] NA


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Chores**
* Added CI jobs to detect "lean" PRs, compute selection filters, and
block merges when lean CI is enabled.
* Centralized computation of matrix filters so downstream build and test
jobs dynamically select lean or full variants.
* Rewired job ordering and dependencies so builds, tests, wheel and
cuopt steps respect the computed filters.

* **Style**
* Updated include paths in several C++ benchmark, test, and utility
files to use unified RMM headers (no behavior changes).
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@rapids-bot rapids-bot bot requested a review from a team as a code owner November 18, 2025 22:56
@rapids-bot rapids-bot bot requested a review from gforsyth November 18, 2025 22:56
@rapids-bot
Copy link
Contributor Author

rapids-bot bot commented Nov 18, 2025

FAILURE - Unable to forward-merge due to an error, manual merge is necessary. Do not use the Resolve conflicts option in this PR, follow these instructions https://docs.rapids.ai/maintainers/forward-merger/

IMPORTANT: When merging this PR, do not use the auto-merger (i.e. the /merge comment). Instead, an admin must manually merge by changing the merging strategy to Create a Merge Commit. Otherwise, history will be lost and the branches become incompatible.

@coderabbitai
Copy link

coderabbitai bot commented Nov 18, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


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

…in review. (#572)

## Checklist

- [x] I am familiar with the [Contributing
Guidelines](https://github.com/NVIDIA/cuopt/blob/HEAD/CONTRIBUTING.md).
- Testing
   - [ ] New or existing tests cover these changes
   - [ ] Added tests
   - [ ] Created an issue to follow-up
   - [x] NA
- Documentation
   - [ ] The documentation is up to date with these changes
   - [x] Added new documentation
   - [ ] NA


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Chores**
* Updated review configuration with new flags controlling review
behaviors and a knowledge-base opt-out.
* **Documentation**
* Added path-specific review guidance for documentation and public C++
API headers.
* Introduced comprehensive code review guidelines covering reviewer
roles, issue severities, review protocol, checklists, and
project-specific considerations.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@rgsl888prabhu rgsl888prabhu mentioned this pull request Nov 19, 2025
8 tasks
@rockhowse rockhowse merged commit bf0b5bc into main Nov 19, 2025
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.

3 participants