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.

Sign up for GitHub

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

Jump to bottom

Finalizer for Env #24

Closed
joehuchette opened this issue Oct 17, 2014 · 4 comments · Fixed by #346
Closed

Finalizer for Env #24

joehuchette opened this issue Oct 17, 2014 · 4 comments · Fixed by #346
Labels
Wrapper: C API Issue is specific to low-level C API

Comments

@joehuchette
Copy link
Contributor

Eli complained today about memory becoming saturated with a (small) model in a tight loop. I thought this was fixed when we changed the behavior in CPLEX.jl, but it doesn't look like it.

cc @mlubin @IainNZ
@IainNZ
Copy link
Contributor

IainNZ commented Oct 17, 2014

Would be good to have reproducible test case - I wonder if forcing gc() fixes it or if we're really actually messing something up.

@mlubin
Copy link
Member

mlubin commented Oct 17, 2014

We definitely finalize the Gurobi Env now.

@yeesian yeesian mentioned this issue Oct 19, 2014
Closed
@mlubin
Copy link
Member

mlubin commented Jan 18, 2015

Can we close this?

@joehuchette
Copy link
Contributor Author

I don't think I ever got a test case from Eli, but presumably whatever potential issue there was hasn't been fixed.

@odow odow added the Wrapper: C API Issue is specific to low-level C API label Dec 14, 2018
@odow odow mentioned this issue Sep 24, 2020
Merged
@odow odow closed this as completed in #346 Sep 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Wrapper: C API Issue is specific to low-level C API
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Rewrite C wrapper using Clang jump-dev/Gurobi.jl
4 participants
@IainNZ @joehuchette @mlubin @odow