-
Notifications
You must be signed in to change notification settings - Fork 42
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
All reactions (not just reversible ones) have now forward_variable an…
…d reverse_variable. Also the following major changes have been introduced: - reversible_encoding has been eliminated. Everything needs to use v_fwd - v_rev now to model the flux of a reaction. - reaction.variable is deprecated. Start using reactions.forward_variable All test miraculously pass ... which is weird. I anticipated flux_variability_analysis would fail because it is not using v_fwd - v_rev explicitily. It doesn't fail though because the objective is set with reactions and SolverBasedModel.objective = reaction will use v_fwd - v_rev
- Loading branch information
1 parent
e490648
commit 5136cb6
Showing
7 changed files
with
221 additions
and
166 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.