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

Branch and Price generates already active columns #312

Closed
guilhemdupuis opened this issue Apr 17, 2020 · 3 comments · Fixed by #319
Closed

Branch and Price generates already active columns #312

guilhemdupuis opened this issue Apr 17, 2020 · 3 comments · Fixed by #319
Labels
bug Something isn't working

Comments

@guilhemdupuis
Copy link
Contributor

Hello to all,

With the generalized assignment problem demo provided on GitHub, some columns appears to be generated multiple times. I get the following warning for most of the new columns, except during the resolution of the first node:
┌ Warning: Column already exists as MC_4000663 and is already active.
└ @ Coluna.Algorithm ~/.julia/packages/Coluna/OqFqT/src/Algorithm/colgen.jl:193

I used the model provided with the demos on GitHub, on the instance gapC-5-100. Model and dataset can be found at
https://covid19.framadrop.org/r/6vFxjdD226#Hqayehtwz0hbT0vTiCDmkh9vyLyWC6f1ouBURYgRJBM=

Environment:
Julia => 1.3.1
"Coluna" => v"0.3.0"
"GLPK" => v"0.12.1"
"BlockDecomposition" => v"1.2.0"
"MathOptInterface" => v"0.9.13"
"JuMP" => v"0.21.2"
OS: MacOs

Best regards,

Guilhem

@guimarqu
Copy link
Contributor

Hi Guilhem,

It's because how we check if a column has negative reduced cost.
The reduced cost must be less than -1e-8. It's too small and some columns may be generated several times because of floating point approximations.

We should let the user decide the threshold.

@RodolpheGriset
Copy link

Thanks for your quick answer Guillaume, That's what we thought but we've prefered to be sur

@guilhemdupuis
Copy link
Contributor Author

Thanks for the explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants