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

implement expression-limited fluxes (E-Flux like algorihm) #777

Merged
merged 2 commits into from
May 3, 2023
Merged

Conversation

exaexa
Copy link
Collaborator

@exaexa exaexa commented May 3, 2023

Closes #552

I'll make a separate port to the 2.0 branch later.

@exaexa exaexa modified the milestones: v2.0, v1.5 May 3, 2023
@exaexa
Copy link
Collaborator Author

exaexa commented May 3, 2023

PS let this be the final feature of the 1.x branch. I'll release 1.5, the rest of the 1.5 milestone will be applied as bugfixes to 1.5.x or better to 2.x directly.

@exaexa exaexa requested a review from stelmo May 3, 2023 08:54
@codecov
Copy link

codecov bot commented May 3, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.05 🎉

Comparison is base (28ddfbd) 91.34% compared to head (eed8be5) 91.39%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #777      +/-   ##
==========================================
+ Coverage   91.34%   91.39%   +0.05%     
==========================================
  Files          82       86       +4     
  Lines        2196     2209      +13     
==========================================
+ Hits         2006     2019      +13     
  Misses        190      190              
Impacted Files Coverage Δ
src/analysis/expressions.jl 100.00% <100.00%> (ø)
src/base/types/wrappers/ExpressionLimitedModel.jl 100.00% <100.00%> (ø)
src/base/utils/eflux.jl 100.00% <100.00%> (ø)
src/reconstruction/expresisons.jl 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@stelmo stelmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, 🚀

Comment on lines +12 to +16
E-Flux algorithm is closer described by: *Colijn, Caroline, Aaron Brandes,
Jeremy Zucker, Desmond S. Lun, Brian Weiner, Maha R. Farhat, Tan-Yun Cheng, D.
Branch Moody, Megan Murray, and James E. Galagan. "Interpreting expression data
with metabolic flux models: predicting Mycobacterium tuberculosis mycolic acid
production." PLoS computational biology 5, no. 8 (2009): e1000489*.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

People love making questionable cobra methods :/

Comment on lines +5 to +7
Create E-Flux-like bounds for a reaction that requires gene products given by
`grr`, with expression "choke" data given by relative probabilities (between 0
and 1) in `relative_expression`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

interesting

Comment on lines +13 to +15
isnothing(grr) && return 1.0
lup(g) = get(relative_expression, g, 1.0)
1 - prod(1 - prod(lup.(gr)) for gr in grr)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I thought this would have some stochasticity here

@exaexa exaexa merged commit 2443452 into master May 3, 2023
@exaexa exaexa deleted the mk-eflux branch May 3, 2023 14:48
@exaexa exaexa restored the mk-eflux branch May 3, 2023 14:52
@exaexa exaexa deleted the mk-eflux branch May 3, 2023 15:01
exaexa added a commit that referenced this pull request May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Model type overlay that describes the genetic manipulation
2 participants