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
PNE bug #38
Comments
And for consistency, please name the constraint analogously in the |
There is indeed a bug, but I believe in the else part. In the Indicator Constraint section The else part is a bug, which wasn't debugged since by default indicator constraints are on. I guess probably it would be more intuitive to have something like:
And then |
However, we should check if the other side of the clause is enforced for the first part. |
We need not mind that. We are minimizing the sum of pure_bin[count]. So if it can possibly take a value 0, it will take a value 0. |
src/Games.cpp
Game::EPEC::make_pure_LCP
The PNE constraints are wrong in both the cases -
if
part as well aselse
part.In the if part, it should read
this->lcpmodel->addGenConstrIndicator( pure_bin[count], 0, ...
as opposed to that argument being 1.In the else part we need
GRB_LESS_EQUAL
.Please check. Let me know, if I am misunderstanding.
The text was updated successfully, but these errors were encountered: