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

🚸Explain FVA better #345

Closed
stelmo opened this issue Jun 17, 2021 · 1 comment · Fixed by #424
Closed

🚸Explain FVA better #345

stelmo opened this issue Jun 17, 2021 · 1 comment · Fixed by #424
Assignees
@stelmo
Labels
documentation Improvements or additions to documentation minor Easy fix with low priority, likely just cosmetic

Comments

@stelmo
Copy link
Collaborator

stelmo commented Jun 17, 2021

My undergrad is struggling with FVA, especially all the options. I think we should explain it more carefully 🚸 in the docs
@stelmo stelmo added the documentation Improvements or additions to documentation label Jun 17, 2021
@stelmo stelmo self-assigned this Jun 17, 2021
@stelmo stelmo changed the title Explain FVA better 🚸Explain FVA better Jun 17, 2021
@exaexa
Copy link
Collaborator

exaexa commented Jun 18, 2021

I already bumped this a bit in #318 , looks like not sufficiently. Any more precise feedback (what failed) would really help.

Making some of the options work right (esp. the functionalish ones) might require a bit of Julia experience, perhaps we can refer to Julia docs somewhere?

@stelmo stelmo added this to the Release - v1.0.4 milestone Jul 21, 2021
@exaexa exaexa added the minor Easy fix with low priority, likely just cosmetic label Aug 4, 2021
@exaexa exaexa removed this from the Release - v1.0.4 milestone Aug 4, 2021
@stelmo stelmo mentioned this issue Aug 9, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stelmo stelmo

Labels
documentation Improvements or additions to documentation minor Easy fix with low priority, likely just cosmetic
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add some more explanations for FVA and add flux_summary usage to notebook LCSB-BioCore/COBREXA.jl
2 participants
@exaexa @stelmo