These are chat archives for opencobra/cobrapy

10th
Feb 2017
Henning Redestig
@hredestig
Feb 10 2017 10:07
short notice but since everyone seem to be available, let's try meeting today at 16:00 (copenhagen time) on https://meet.jit.si/cobrapy
Moritz E. Beber
@Midnighter
Feb 10 2017 14:13
@cdiener: can we make the following part of your FVA port a utility function somewhere?
legacy = False
if solver is None:
    solver = cobra_model.solver
elif "optlang-" in solver:
    solver = sutil.interface_to_str(solver)
    solver = sutil.solvers[solver]
else:
    legacy = True
solver = solver_dict[solver]
I imagine it would return the tuple (legacy, solver).
Christian Diener
@cdiener
Feb 10 2017 14:17
Sure like legacy, solver = choose_solver(model, solver_str)?
I can put it in util.solver
Ah also needs an argument for capabilities (QP, MIP)
Peter St. John
@pstjohn
Feb 10 2017 14:51
so just to make sure I’ve got the time right, we’re meeting in ~ 10 minutes?
Peter St. John
@pstjohn
Feb 10 2017 15:03
Perhaps not :smile: ?
Kristian Jensen
@KristianJensen
Feb 10 2017 15:04
Yes
Henning Redestig
@hredestig
Feb 10 2017 15:09
Meeting is on, @mmundy42 you also joining?
Peter St. John
@pstjohn
Feb 10 2017 15:22
status on this meeting?
just want to make sure I’m not connected wrong, the jitsi page doesn’t look active
Christian Diener
@cdiener
Feb 10 2017 15:23
its running. we are here
Peter St. John
@pstjohn
Feb 10 2017 16:29
Are you guys familiar with CVXPY? Its another package that seems to have similar goals to optlang. Just wondering what the efficiency of their setter/getter methods was and if we could steal some inspiration
Christian Diener
@cdiener
Feb 10 2017 19:03

@hredestig Here is the comparison script.

https://gist.github.com/cdiener/f326c33f331b370c6596fcf83d9d4bb4

You will need to run with py.test --benchmark-storage=.bench --benchmark-save=bla, than compare with ./compare.py .bench/0001_bla.json .bench/0002_bla.json

Christian Diener
@cdiener
Feb 10 2017 19:46
@Midnighter put the changes you requested in #375 . After it's pulled I will refactor the single deletion code to use it as well.
Mike Mundy
@mmundy42
Feb 10 2017 23:24
Sorry, had a bunch of stuff going on today and missed that there was a meeting this morning