-
Notifications
You must be signed in to change notification settings - Fork 10
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 add_vars #52
Comments
I actually think this is the incorrect implementation. blend = mining.addVars(years, name="Blend") Notice how it uses the singular 'name' parameter. Therefore I'm drawn to conclude that the intention of add_var vs add_vars are inconsistent with the official Gurobi API. Here are the associated docs of Gurobi: I'm pretty novice in rust, so probably won't be able to solve this :') |
I don't think the difference between one vs many names is the cause of the segfault. The Python API internally converts the single name into many names by appending Of course, we could discuss whether you should have a single-name version of the function, which then does the same logic to generate names, but that's a separate question from this issue. |
I got a segmentation fault when running
add_vars
. There's something wrong with it. It works when callingadd_var
in a loop.The text was updated successfully, but these errors were encountered: