Skip to content

Conversation

@akifcorduk
Copy link
Contributor

@akifcorduk akifcorduk commented Aug 6, 2025

This PR adds a new local search operator: FP with objective cutoff.

We were using FP as a means of solution generation algorithm. Now we are also using it as a local search by adding a cutting plane at the current fesible point perpendicular to the objective. We rerun FJ at this point to have a different restart point and continue the FP search with the cutting plane added.

This on brings down the average optimality gap by 1-2%.

@akifcorduk akifcorduk added this to the 25.10 milestone Aug 6, 2025
@akifcorduk akifcorduk added the non-breaking Introduces a non-breaking change label Aug 6, 2025
@akifcorduk akifcorduk requested a review from a team as a code owner August 6, 2025 09:50
@akifcorduk akifcorduk added the improvement Improves an existing functionality label Aug 6, 2025
Copy link
Contributor

@aliceb-nv aliceb-nv left a comment

Choose a reason for hiding this comment

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

Just a few comments :) Thanks a lot for the great work Akif! Awesome to see further quality improvements
Additionally: Shouldn't this PR be merged to the branch-25.10 branch instead?
I'll approve once this is confirmed

@rgsl888prabhu rgsl888prabhu changed the base branch from branch-25.08 to branch-25.10 August 6, 2025 14:25
Copy link
Contributor

@aliceb-nv aliceb-nv left a comment

Choose a reason for hiding this comment

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

Approving, thanks for the great work Akif!

@anandhkb
Copy link
Contributor

anandhkb commented Aug 8, 2025

GREAT work @akifcorduk
@afender for viz

@akifcorduk akifcorduk marked this pull request as draft August 15, 2025 15:52
@copy-pr-bot
Copy link

copy-pr-bot bot commented Aug 15, 2025

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.

@akifcorduk akifcorduk marked this pull request as ready for review August 18, 2025 11:47
@akifcorduk
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit c76c186 into NVIDIA:branch-25.10 Aug 21, 2025
142 of 144 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