Skip to content

Rename and split up cpp/src directories#846

Merged
rapids-bot[bot] merged 9 commits intoNVIDIA:mainfrom
chris-maes:move_files
Feb 12, 2026
Merged

Rename and split up cpp/src directories#846
rapids-bot[bot] merged 9 commits intoNVIDIA:mainfrom
chris-maes:move_files

Conversation

@chris-maes
Copy link
Copy Markdown
Contributor

@chris-maes chris-maes commented Feb 10, 2026

This PR makes the following changes:

  • The dual_simplex directory is split into the following directories: barrier, branch_and_bound, cuts
  • The linear_programming directory is renamed to pdlp
  • The mip directory is renamed to mip_heurisitics

This better reflects the structure of the code.

This PR has no other code changes. It only alters the #include paths

This PR also includes a fix for C++20 in the cuOpt test builds

@chris-maes chris-maes requested review from a team as code owners February 10, 2026 22:12
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Feb 10, 2026

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.

@chris-maes chris-maes self-assigned this Feb 10, 2026
@chris-maes chris-maes added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Feb 10, 2026
@chris-maes chris-maes changed the base branch from main to release/26.02 February 10, 2026 22:14
@chris-maes
Copy link
Copy Markdown
Contributor Author

/ok to test 1346552

Comment thread cpp/src/branch_and_bound/CMakeLists.txt Outdated
Comment thread cpp/src/cuts/CMakeLists.txt Outdated
@chris-maes chris-maes changed the base branch from release/26.02 to main February 11, 2026 16:05
@chris-maes
Copy link
Copy Markdown
Contributor Author

/ok to test 93561ed

Comment thread cpp/src/barrier/CMakeLists.txt Outdated
Comment thread cpp/src/branch_and_bound/CMakeLists.txt Outdated
@chris-maes
Copy link
Copy Markdown
Contributor Author

/ok to test 2de545b

Copy link
Copy Markdown
Collaborator

@rgsl888prabhu rgsl888prabhu left a comment

Choose a reason for hiding this comment

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

We will follow-up on any lingering reviews, thank you @chris-maes

@chris-maes
Copy link
Copy Markdown
Contributor Author

/ok to test 8db47ed

@chris-maes
Copy link
Copy Markdown
Contributor Author

/ok to test fb1966e

@chris-maes
Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit b3bb21e into NVIDIA:main Feb 12, 2026
183 of 185 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.

3 participants