-
Notifications
You must be signed in to change notification settings - Fork 27
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
Integration with eQuilibrator #36
Comments
Good day @carrascomj ! I am happy to see you are interested in pyTFA. This is definitely a functionality we have considered, and it would considerably expand the usability of pyTFA. However, we don't really have anyone that could work on it at the moment, so your effort is very welcome ! For the implementation, if you decide to write a standalone module we will gladly link to it, and if instead you would like to contribute directly to the code we can work with pull requests :) Very cool notebook also, thank you for sharing it. And indeed, yes, pH and Ionic strength are extremely important for the models to be accurate. I am happy to see you could reproduce our results ! It's been a while since I have used eQuilibrator, so maybe things changed, but also it is important to make sure we recover the ΔGr'°, and not the ΔGr'm. |
Nice! I will fork the project and will try to implement it in the package. Just a quick question: as the eQuilibrator-API provides calculation of Confidence intervals, would it be meaningful to use them as the error for ΔGr'? |
Yes, we use the errors as bounds for the ΔGr'°, so we can directly integrate them! |
Hello! After this PR is merged to equilibrator_api, I should be able to open mine for pytfa. Meanwhile, I have stumbled upon some decisions to take:
pip install pytfa[equilibrator]
|
Good day Jorge,
Thank your for being so on top of this |
Hi! After talking about it on the equilibrator_api side, I will try to implement it by reconstructing the However, using raw formation energies from eQuilibrator may yield very high uncertainties. If they turned to be too high to be informative, I will need to go back to the proposed alternative |
Good day Jorge, |
Hello Pierre, As explained in the PR, eQuilibrator uses the Component Contribution method instead of group contributions. Thus, major profound changes would be needed in the package to accommodate this information from eQuilibrator. The solution from the eQuilibrator prepared tmodel varies from but it is more similar to the cobra model than using the I have used the total summation of fluxes to quickly compare them: cobrapy -> 143.13836864239119
thermo_data from equilibrator -> 188.8536839871894
thermo_data from .thermodb -> 1190.6526819344149 |
As an user, I would like to use the eQuilibrator-API as the thermodynamic database. Thus, eQuilibrator would parse the reactions, calculate ΔGr for each reaction and use the data to prepare and convert a
cobra.Model
into apyTFA.ThermoModel
. Is there any reason that hampers this integration?I would be happy to help with the implementation, maybe as an extra dependency. I have used the tutorial in
tutorials/figure_paper.py
to try to reproduce the results just by adding reaction data, but the results differ (at the end of this notebook).EDIT: after adding the pH and ionic strength information about each metabolite (without running the built-in
.prepare()
method), the results are exactly reproduced.The text was updated successfully, but these errors were encountered: