Skip to content
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

Weird warning in birkhoff example #96

Closed
wants to merge 2 commits into from
Closed

Weird warning in birkhoff example #96

wants to merge 2 commits into from

Conversation

dhendryc
Copy link
Collaborator

See issue #94

It looks like a Bonobo warning. But I added a bunch of prints to locate there exactly it is thrown and it seems to be in the blended_pariwise_conditional_gradient.

There is no such warning in our fw_callback and, as far as I have seen, also not in the Bonobo source code.

Could this be from SCIP?

@dhendryc dhendryc linked an issue Aug 31, 2022 that may be closed by this pull request
@matbesancon
Copy link
Member

blended_pariwise_conditional_gradient has no such warning I think?

@dhendryc
Copy link
Collaborator Author

Yes, I think this stems from SCIP since neither we nor Bonobo have such a warning in the source code.

@matbesancon
Copy link
Member

Yes indeed:

scip-dev/scip/src/scip/primal.c
666:         SCIPmessagePrintWarning(messagehdlr, "Dual bound %g is larger than the objective of the primal solution %g. The solution might not be optimal.\n",

https://github.com/scipopt/scip/blob/master/src/scip/primal.c#L666

@dhendryc
Copy link
Collaborator Author

Okay, then we at least know where the warning is thrown.

Do you have a clue as to why this happens? Could it be a numerical issue?

@matbesancon
Copy link
Member

Ideally we should try to extract which problem is solved at that point that generates this issue, to report it to SCIP but that's it

@dhendryc
Copy link
Collaborator Author

dhendryc commented Sep 1, 2022

This is a SCIP issue. Seems like some parameters are not reset in between solves. Will be transferred to SCIP.

@dhendryc dhendryc closed this Sep 1, 2022
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.

Another issue with the Birkhoff example
2 participants