-
Notifications
You must be signed in to change notification settings - Fork 1
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
Implement functionality to work with multiple databases #43
Comments
See #36. Rough changes are already implemented but this can surely be extended. |
As a suggestion, the enhancement of the |
The function get_model_reacs_or_metabs now takes a new parameter. If the model's metabolites/reactions are not from the BiGG namespace the parameter col_name can now be changed to a user-specific string. To not break the original code, col_name is by default set to 'bigg_id'.
As the main aim of this issue, namely downloading and maintaining the different databases used within refineGEMs in one place, is already implemented, this issue can be closed. Furthermore, enhancing refineGEMs through the usage of more databases is already handled or mentioned in more detail in issue #36, and the storage question of these databases is already mentioned in more detail in issue #49. Thus, I will close this issue as completed. |
Is your feature request related to a problem? Please describe.
Multiple databases like KEGG and BiGG are referenced and used within refineGEMs but sometimes functions overlap or duplicate and the databases are not updated.
Describe the solution you'd like
Something triggered by the user which leads to a one-time download (the MCC tool does that) which can be reused or updated.
Maybe this can be bundled into a submodule
databases.py
which takes care of downloading and then parsing as well as formatting so that these databases can be used by a refineGEMs user and within the scripts provided by refineGEMs. (see #31, #28)The text was updated successfully, but these errors were encountered: