-
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
Duplicate Beta-Oxidation Reactions for (4Z,7Z,10Z,13Z,16Z)-docosapentaenoyl-CoA #746
Comments
Really enjoying this clean-up of the beta-oxidation reactions. This looks good to me. Out of curiosity, and maybe I missed it before, are you using a script-based approach to search for these cases where reactions are the same but broken into multiple steps, or are you manually identifying them as you go? |
I used a script to identify all reactions that had metabolites in [m] that were associated with one or more genes that encode exlusively peroxisomal beta-oxidation enzymes in #634, but eventually realized that many of those reactions had other problems, so all of these issues are manual follow ups. For this issue in particular, I identified a whole set (maybe like 15-20) of reactions like MAR03422 that do complete beta-oxidation of something in a single step, but so far, each individual case has a somewhat unique set of problems associated with it, so there are going to be a lot of issues following up on those. (beta-oxidation of linoleoyl- and gamma-linolenoyl-CoA is particularly messy) |
Ok, great work though. |
good to know this - would be nice to include the script that can potentially be turned into a GH action bot at some point |
ah yea I just did something like perox_genes = ['ACOX1', 'ACOX2', 'ACOX3', 'ACAD11', 'EHHADH', 'HSD17B4', 'ACAA1', 'DECR2']
for r in model.reactions:
if any(g in r.gene_name_reaction_rule for g in perox_genes) and all(m.compartment == 'm' for m in r.metabolites):
print(f'{r.id} | {r.build_reaction_string(True)} | {r.gene_name_reaction_rule}') If you wanted to scale that up to a GH action, I suppose you could take the localization info in |
scaling this up to GH action would be a very good idea that worth further consideration |
Current behavior:
(4Z,7Z,10Z,13Z,16Z)-docosapentaenoyl-CoA can be beta-oxidized in a single step in
MAR03422
: (4Z,7Z,10Z,13Z,16Z)-docosapentaenoyl-CoA [m] + 10 CoA [m] + 5 FAD [m] + 10 H2O [m] + 10 NAD+ [m] ⇒ 5 FADH2 [m] + 10 H+ [m] + 10 NADH [m] + 11 acetyl-CoA [m], GPR: ACAA1 or HSD17B10 or ACADVL or HADHA or ACADL or ACADM or ECHS1 or HADHB or HADH or ACAA2 or ACAD9or a lot of steps:
MAR05380
MAR05404
MAR05383
MAR05378
MAR05373
MAR05367
MAR05360
MAR05195
MAR05302
MAR05229
MAR05208
MAR05123
MAR05075
MAR05072
MAR03294
MAR04970
MAR03298
MAR03143
MAR03144
MAR03146
MAR03149
MAR03150
MAR03151
MAR03153
MAR03156
MAR03157
MAR03158
MAR03160
MAR03163
MAR03164
MAR03166
MAR03885
Expected feature/value/output:
Since
MAR05380
-MAR03885
accurately represent each individual step of the beta-oxidation of (4Z,7Z,10Z,13Z,16Z)-docosapentaenoyl-CoA in much more detail thanMAR03422
, I thinkMAR03422
should be removed for being unnecessary.The text was updated successfully, but these errors were encountered: