-
Notifications
You must be signed in to change notification settings - Fork 111
Fix compile and log #450
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.
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
Fix compile and log #450
Conversation
hlinsen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Akif. Do you think we could also disable postsolve logs or change the log level https://github.com/NVIDIA/cuopt/blob/branch-25.10/cpp/src/mip/presolve/third_party_presolve.cpp#L311?
|
@hlinsen IMO, we can keep it. The user would know why we are returning a slightly infeasible solution. But I think we can get @chris-maes opinion too. |
|
@akifcorduk not sure why it is failing or even where it is failing, but this test run has been failing several times, I think it has been run atleast 4 times. Please take a look in earlier run if you can find any clue https://github.com/NVIDIA/cuopt/actions/runs/18288323659/job/52087939557?pr=450 On the top you can see attempts, where you can switch to previous runs. |
|
Some test runs returns a Latest: 3rd attempt: |
This PR fixes the delayed termination of the branch-and-bound algorithm after reaching the time limit. Authors: - Nicolas L. Guidotti (https://github.com/nguidotti) Approvers: - Chris Maes (https://github.com/chris-maes) URL: NVIDIA#451
|
/merge |
This PR fixes a log level and a compilation error that happens on non-CI envs.