-
Notifications
You must be signed in to change notification settings - Fork 44
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
Unbalance reactions checked in the new content of Human-GEM #814
Comments
@Devlin-Moyer I noticed that you have added |
Really nice work @JHL-452b, thank you for this! |
If by "remove H+(MAM02039m):1" you mean "add one H+ ( |
Yeah, it's a mistake in my statement, which has been corrected. |
In Human-GEM version 1.18, the above reactions were modified to make the model consistent, but the additional unbalanced reactions in the current develop branch still make the model inconsistent. Thus, I try to find and resolve the following imbalance reactions so that the model regains consistency.
|
Am I missing something or does PR #815 not mention |
ah I see you meant #805; both of your proposed changes to fix the mass balance issues with those reactions look good to me |
Yeah, it's a wrong link and I corrected it. So, all these reactions has been modified and I will go head. |
Current behavior:
With the curation of Human-GEM, there are some unbalance rxns here that did not exist in the previous model.
MAR01444
MAR20005
MAR20006
MAR20007
MAR20008
MAR20021
MAR20022
MAR20023
MAR20024
MAR20025
MAR20026
MAR20027
MAR20028
MAR20029
MAR20030
MAR20046
MAR20047
MAR20049
MAR20050
MAR20064
MAR20065
MAR20098
MAR20111
MAR20112
Proposed changes:
All these changes ensure that the current reaction is balanced without affecting other reactions.
The text was updated successfully, but these errors were encountered: