Add exchange rxns - #123
Conversation
the "today" command requires the Financial Toolbox to be installed, whereas "clock" is in the base Matlab installation.
|
This PR also demonstrates the use of the new |
|
@JonathanRob this PR looks good. Would it be better to name the files according to the targeted issue? For example, change filename from |
|
@Hao-Chalmers I think that's a good idea. It would make it quite clear which files are associated with which issues/changes. |
|
@Hao-Chalmers Yes, this PR adds more exchange reactions than the 4 discussed in #117, but these are added for a different reason (as explained in the description of this PR). The 4 exchange reactions discussed in #117 are to allow exchange of metabolites that are in the extracellular compartment, but do not currently have exchange reactions to/from the boundary. The other exchange reactions added in this PR are for a separate problem, where metabolites are being transported to/from the boundary from a non-extracellular compartment. Since this was not the topic of #117, I don't think there's a need to discuss these other reaction changes there. |
Main improvements in this PR:
[Addresses Issue #117]
Exchange reactions are added for four metabolites that are present in the extracellular compartment, but do not currently have exchange reactions transporting them to/from the boundary. These changes are implemented with the
curateExchangeRxns2.mscript.The four metabolites for which exchange reactions are added:
In addition, the script addresses the issue of five reactions that involve transport between the boundary compartment and a non-extracellular compartment. The reaction IDs are:
Information on the new reactions added to the model is contained in the
newExchangeRxns.tsvfile.I hereby confirm that I have:
develas a target branch