Skip to content

Commit

Permalink
Merge pull request #273 from LCSB-BioCore/mk-fix-screening
Browse files Browse the repository at this point in the history
fix a problem in screening
  • Loading branch information
stelmo committed May 23, 2021
2 parents 6b66951 + cf53296 commit 01ba93d
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/analysis/screening.jl
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ function screen(

map(fetch, save_at.(workers, :cobrexa_screen_variants_model, Ref(model)))
map(fetch, save_at.(workers, :cobrexa_screen_variants_analysis_fn, Ref(analysis)))
map(fetch, get_from.(workers, Ref(:(precache!(cobrexa_screen_model)))))
map(fetch, get_from.(workers, Ref(:(precache!(cobrexa_screen_variants_model)))))

if isnothing(variants)
if isnothing(args)
Expand Down
9 changes: 8 additions & 1 deletion test/analysis/screening.jl
Original file line number Diff line number Diff line change
Expand Up @@ -34,16 +34,23 @@
return mm
end

ws = addprocs(2)
@everywhere using COBREXA
@everywhere using Tulip

@test screen_variants(
m,
[[quad_rxn(i)] for i = 1:3],
m -> flux_balance_analysis_vec(m, Tulip.Optimizer),
m -> flux_balance_analysis_vec(m, Tulip.Optimizer);
workers = ws,
) == [
[250.0, -250.0, -1000.0, 250.0, 1000.0, 250.0, 250.0],
[500.0, 500.0, 1000.0, 500.0, -1000.0, 500.0, 500.0],
[500.0, 500.0, 1000.0, -500.0, 1000.0, 500.0, 500.0],
]

rmprocs(ws)

# test solver modifications
@test screen(
m;
Expand Down

0 comments on commit 01ba93d

Please sign in to comment.