annotate COBRAxy/utils/model_utils.py @ 509:5956dcf94277 draft default tip

Uploaded
author francesco_lapi
date Wed, 01 Oct 2025 15:34:21 +0000
parents ca98c149ec61
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
1 """
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
2 Utilities for generating and manipulating COBRA models and related metadata.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
3
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
4 This module includes helpers to:
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
5 - extract rules, reactions, bounds, objective coefficients, and compartments
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
6 - build a COBRA model from a tabular file
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
7 - set objective and medium from dataframes
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
8 - validate a model and convert gene identifiers
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
9 - translate model GPRs using mapping tables
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
10 """
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
11 import os
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
12 import cobra
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
13 import pandas as pd
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
14 import re
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
15 import logging
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
16 from typing import Optional, Tuple, Union, List, Dict, Set
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
17 from collections import defaultdict
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
18 import utils.rule_parsing as rulesUtils
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
19 import utils.reaction_parsing as reactionUtils
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
20 from cobra import Model as cobraModel, Reaction, Metabolite
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
21 import sys
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
22
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
23
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
24 ############################ check_methods ####################################
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
25 def gene_type(l :str, name :str) -> str:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
26 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
27 Determine the type of gene ID.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
28
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
29 Args:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
30 l (str): The gene identifier to check.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
31 name (str): The name of the dataset, used in error messages.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
32
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
33 Returns:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
34 str: The type of gene ID ('hugo_id', 'ensembl_gene_id', 'symbol', or 'entrez_id').
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
35
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
36 Raises:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
37 sys.exit: If the gene ID type is not supported, the execution is aborted.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
38 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
39 if check_hgnc(l):
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
40 return 'hugo_id'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
41 elif check_ensembl(l):
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
42 return 'ensembl_gene_id'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
43 elif check_symbol(l):
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
44 return 'symbol'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
45 elif check_entrez(l):
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
46 return 'entrez_id'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
47 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
48 sys.exit('Execution aborted:\n' +
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
49 'gene ID type in ' + name + ' not supported. Supported ID'+
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
50 'types are: HUGO ID, Ensemble ID, HUGO symbol, Entrez ID\n')
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
51
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
52 def check_hgnc(l :str) -> bool:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
53 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
54 Check if a gene identifier follows the HGNC format.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
55
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
56 Args:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
57 l (str): The gene identifier to check.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
58
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
59 Returns:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
60 bool: True if the gene identifier follows the HGNC format, False otherwise.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
61 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
62 if len(l) > 5:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
63 if (l.upper()).startswith('HGNC:'):
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
64 return l[5:].isdigit()
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
65 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
66 return False
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
67 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
68 return False
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
69
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
70 def check_ensembl(l :str) -> bool:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
71 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
72 Check if a gene identifier follows the Ensembl format.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
73
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
74 Args:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
75 l (str): The gene identifier to check.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
76
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
77 Returns:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
78 bool: True if the gene identifier follows the Ensembl format, False otherwise.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
79 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
80 return l.upper().startswith('ENS')
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
81
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
82
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
83 def check_symbol(l :str) -> bool:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
84 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
85 Check if a gene identifier follows the symbol format.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
86
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
87 Args:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
88 l (str): The gene identifier to check.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
89
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
90 Returns:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
91 bool: True if the gene identifier follows the symbol format, False otherwise.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
92 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
93 if len(l) > 0:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
94 if l[0].isalpha() and l[1:].isalnum():
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
95 return True
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
96 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
97 return False
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
98 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
99 return False
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
100
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
101 def check_entrez(l :str) -> bool:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
102 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
103 Check if a gene identifier follows the Entrez ID format.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
104
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
105 Args:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
106 l (str): The gene identifier to check.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
107
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
108 Returns:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
109 bool: True if the gene identifier follows the Entrez ID format, False otherwise.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
110 """
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
111 if len(l) > 0:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
112 return l.isdigit()
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
113 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
114 return False
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
115
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
116 ################################- DATA GENERATION -################################
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
117 ReactionId = str
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
118 def generate_rules(model: cobraModel, *, asParsed = True) -> Union[Dict[ReactionId, rulesUtils.OpList], Dict[ReactionId, str]]:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
119 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
120 Generate a dictionary mapping reaction IDs to GPR rules from the model.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
121
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
122 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
123 model: COBRA model to derive data from.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
124 asParsed: If True, parse rules into a nested list structure; otherwise keep raw strings.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
125
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
126 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
127 Dict[ReactionId, rulesUtils.OpList]: Parsed rules by reaction ID.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
128 Dict[ReactionId, str]: Raw rules by reaction ID.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
129 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
130 _ruleGetter = lambda reaction : reaction.gene_reaction_rule
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
131 ruleExtractor = (lambda reaction :
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
132 rulesUtils.parseRuleToNestedList(_ruleGetter(reaction))) if asParsed else _ruleGetter
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
133
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
134 return {
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
135 reaction.id : ruleExtractor(reaction)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
136 for reaction in model.reactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
137 if reaction.gene_reaction_rule }
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
138
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
139 def generate_reactions(model :cobraModel, *, asParsed = True) -> Dict[ReactionId, str]:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
140 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
141 Generate a dictionary mapping reaction IDs to reaction formulas from the model.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
142
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
143 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
144 model: COBRA model to derive data from.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
145 asParsed: If True, convert formulas into a parsed representation; otherwise keep raw strings.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
146
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
147 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
148 Dict[ReactionId, str]: Reactions by reaction ID (parsed if requested).
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
149 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
150
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
151 unparsedReactions = {
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
152 reaction.id : reaction.reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
153 for reaction in model.reactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
154 if reaction.reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
155 }
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
156
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
157 if not asParsed: return unparsedReactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
158
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
159 return reactionUtils.create_reaction_dict(unparsedReactions)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
160
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
161 def get_medium(model:cobraModel) -> pd.DataFrame:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
162 """
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
163 Extract the uptake reactions representing the model medium.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
164
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
165 Returns a DataFrame with a single column 'reaction' listing exchange reactions
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
166 with negative lower bound and no positive stoichiometric coefficients (uptake only).
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
167 """
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
168 trueMedium=[]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
169 for r in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
170 positiveCoeff=0
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
171 for m in r.metabolites:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
172 if r.get_coefficient(m.id)>0:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
173 positiveCoeff=1;
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
174 if (positiveCoeff==0 and r.lower_bound<0):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
175 trueMedium.append(r.id)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
176
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
177 df_medium = pd.DataFrame()
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
178 df_medium["reaction"] = trueMedium
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
179 return df_medium
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
180
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
181 def extract_objective_coefficients(model: cobraModel) -> pd.DataFrame:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
182 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
183 Extract objective coefficients for each reaction.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
184
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
185 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
186 model: COBRA model
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
187
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
188 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
189 pd.DataFrame with columns: ReactionID, ObjectiveCoefficient
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
190 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
191 coeffs = []
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
192 # model.objective.expression is a linear expression
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
193 objective_expr = model.objective.expression.as_coefficients_dict()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
194
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
195 for reaction in model.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
196 coeff = objective_expr.get(reaction.forward_variable, 0.0)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
197 coeffs.append({
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
198 "ReactionID": reaction.id,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
199 "ObjectiveCoefficient": coeff
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
200 })
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
201
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
202 return pd.DataFrame(coeffs)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
203
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
204 def generate_bounds(model:cobraModel) -> pd.DataFrame:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
205 """
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
206 Build a DataFrame of lower/upper bounds for all reactions.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
207
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
208 Returns:
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
209 pd.DataFrame indexed by reaction IDs with columns ['lower_bound', 'upper_bound'].
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
210 """
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
211
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
212 rxns = []
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
213 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
214 rxns.append(reaction.id)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
215
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
216 bounds = pd.DataFrame(columns = ["lower_bound", "upper_bound"], index=rxns)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
217
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
218 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
219 bounds.loc[reaction.id] = [reaction.lower_bound, reaction.upper_bound]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
220 return bounds
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
221
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
222
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
223
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
224 def generate_compartments(model: cobraModel) -> pd.DataFrame:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
225 """
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
226 Generates a DataFrame containing pathway information for each reaction.
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
227 Creates columns for each pathway position (Pathway_1, Pathway_2, etc.) only if pathways exist.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
228
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
229 Args:
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
230 model: the COBRA model to extract pathway data from.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
231
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
232 Returns:
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
233 pd.DataFrame: DataFrame with ReactionID and pathway columns (if any pathways exist)
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
234 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
235 pathway_data = []
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
236
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
237 # First pass: determine the maximum number of pathways any reaction has
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
238 max_pathways = 0
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
239 reaction_pathways = {}
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
240 has_any_pathways = False
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
241
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
242 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
243 # Get unique pathways from all metabolites in the reaction
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
244 if 'pathways' in reaction.annotation and reaction.annotation['pathways']:
503
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
245 if type(reaction.annotation['pathways']) == list:
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
246 # Filter out empty/None values
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
247 valid_pathways = [p for p in reaction.annotation['pathways'] if p]
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
248 if valid_pathways:
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
249 reaction_pathways[reaction.id] = valid_pathways
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
250 max_pathways = max(max_pathways, len(valid_pathways))
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
251 has_any_pathways = True
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
252 else:
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
253 reaction_pathways[reaction.id] = []
503
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
254 else:
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
255 # Single pathway value
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
256 if reaction.annotation['pathways']:
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
257 reaction_pathways[reaction.id] = [reaction.annotation['pathways']]
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
258 max_pathways = max(max_pathways, 1)
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
259 has_any_pathways = True
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
260 else:
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
261 reaction_pathways[reaction.id] = []
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
262 else:
503
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
263 # No pathway annotation - use empty list
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
264 reaction_pathways[reaction.id] = []
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
265
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
266 # If no pathways exist in the model, return DataFrame with only ReactionID
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
267 if not has_any_pathways:
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
268 return None
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
269
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
270 # Create column names for pathways only if they exist
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
271 pathway_columns = [f"Pathway_{i+1}" for i in range(max_pathways)]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
272
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
273 # Second pass: create the data with pathway columns
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
274 for reaction_id, pathways in reaction_pathways.items():
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
275 row = {"ReactionID": reaction_id}
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
276
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
277 # Fill pathway columns
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
278 for i in range(max_pathways):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
279 col_name = pathway_columns[i]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
280 if i < len(pathways):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
281 row[col_name] = pathways[i]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
282 else:
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
283 row[col_name] = None
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
284
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
285 pathway_data.append(row)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
286
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
287 return pd.DataFrame(pathway_data)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
288
505
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
289 def set_annotation_pathways_from_data(model: cobraModel, df: pd.DataFrame):
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
290 """Set reaction pathways based on 'Pathway_1', 'Pathway_2', ... columns in the dataframe."""
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
291 pathway_cols = [col for col in df.columns if col.startswith('Pathway_')]
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
292 if not pathway_cols:
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
293 print("No 'Pathway_' columns found, skipping pathway annotation")
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
294 return
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
295
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
296 pathway_data = defaultdict(list)
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
297
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
298 for idx, row in df.iterrows():
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
299 reaction_id = str(row['ReactionID']).strip()
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
300 if reaction_id not in model.reactions:
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
301 continue
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
302
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
303 pathways = []
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
304 for col in pathway_cols:
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
305 if pd.notna(row[col]) and str(row[col]).strip():
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
306 pathways.append(str(row[col]).strip())
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
307
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
308 if pathways:
506
ffc234ec80db Uploaded
francesco_lapi
parents: 505
diff changeset
309
505
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
310 reaction = model.reactions.get_by_id(reaction_id)
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
311 if len(pathways) == 1:
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
312 reaction.annotation['pathways'] = pathways[0]
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
313 else:
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
314 reaction.annotation['pathways'] = pathways
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
315
505
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
316 pathway_data[reaction_id] = pathways
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
317
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
318 print(f"Annotated {len(pathway_data)} reactions with pathways.")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
319
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
320 def build_cobra_model_from_csv(csv_path: str, model_id: str = "new_model") -> cobraModel:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
321 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
322 Build a COBRApy model from a tabular file with reaction data.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
323
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
324 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
325 csv_path: Path to the tab-separated file.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
326 model_id: ID for the newly created model.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
327
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
328 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
329 cobra.Model: The constructed COBRApy model.
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
330 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
331
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
332 # Try to detect separator
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
333 with open(csv_path, 'r') as f:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
334 first_line = f.readline()
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
335 sep = '\t' if '\t' in first_line else ','
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
336
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
337 df = pd.read_csv(csv_path, sep=sep)
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
338
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
339 # Check required columns
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
340 required_cols = ['ReactionID', 'Formula']
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
341 missing_cols = [col for col in required_cols if col not in df.columns]
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
342 if missing_cols:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
343 raise ValueError(f"Missing required columns: {missing_cols}. Available columns: {list(df.columns)}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
344
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
345 model = cobraModel(model_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
346
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
347 metabolites_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
348 compartments_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
349
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
350 print(f"Building model from {len(df)} reactions...")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
351
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
352 for idx, row in df.iterrows():
448
f8b1761eee37 Uploaded
francesco_lapi
parents: 444
diff changeset
353 reaction_formula = str(row['Formula']).strip()
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
354 if not reaction_formula or reaction_formula == 'nan':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
355 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
356
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
357 metabolites = extract_metabolites_from_reaction(reaction_formula)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
358
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
359 for met_id in metabolites:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
360 compartment = extract_compartment_from_metabolite(met_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
361
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
362 if compartment not in compartments_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
363 compartments_dict[compartment] = compartment
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
364
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
365 if met_id not in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
366 metabolites_dict[met_id] = Metabolite(
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
367 id=met_id,
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
368 compartment=compartment,
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
369 name=met_id.replace(f"_{compartment}", "").replace("__", "_")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
370 )
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
371
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
372 model.compartments = compartments_dict
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
373
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
374 model.add_metabolites(list(metabolites_dict.values()))
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
375
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
376 print(f"Added {len(metabolites_dict)} metabolites and {len(compartments_dict)} compartments")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
377
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
378 reactions_added = 0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
379 reactions_skipped = 0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
380
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
381 for idx, row in df.iterrows():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
382
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
383 reaction_id = str(row['ReactionID']).strip()
427
4a385fdb9e58 Uploaded
francesco_lapi
parents: 426
diff changeset
384 reaction_formula = str(row['Formula']).strip()
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
385
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
386 if not reaction_formula or reaction_formula == 'nan':
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
387 raise ValueError(f"Missing reaction formula for {reaction_id}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
388
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
389 reaction = Reaction(reaction_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
390 reaction.name = reaction_id
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
391
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
392 reaction.lower_bound = float(row['lower_bound']) if pd.notna(row['lower_bound']) else -1000.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
393 reaction.upper_bound = float(row['upper_bound']) if pd.notna(row['upper_bound']) else 1000.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
394
427
4a385fdb9e58 Uploaded
francesco_lapi
parents: 426
diff changeset
395 if pd.notna(row['GPR']) and str(row['GPR']).strip():
4a385fdb9e58 Uploaded
francesco_lapi
parents: 426
diff changeset
396 reaction.gene_reaction_rule = str(row['GPR']).strip()
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
397
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
398 try:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
399 parse_reaction_formula(reaction, reaction_formula, metabolites_dict)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
400 except Exception as e:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
401 print(f"Error parsing reaction {reaction_id}: {e}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
402 reactions_skipped += 1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
403 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
404
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
405 model.add_reactions([reaction])
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
406 reactions_added += 1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
407
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
408
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
409 print(f"Added {reactions_added} reactions, skipped {reactions_skipped} reactions")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
410
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
411 # set objective function
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
412 set_objective_from_csv(model, df, obj_col="ObjectiveCoefficient")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
413
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
414 set_medium_from_data(model, df)
505
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
415
96f512dff490 Uploaded
francesco_lapi
parents: 503
diff changeset
416 set_annotation_pathways_from_data(model, df)
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
417
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
418 print(f"Model completed: {len(model.reactions)} reactions, {len(model.metabolites)} metabolites")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
420 return model
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
421
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
422
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
423 # Estrae tutti gli ID metaboliti nella formula (gestisce prefissi numerici + underscore)
499
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
424 #def extract_metabolites_from_reaction(reaction_formula: str) -> Set[str]:
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
425 # """
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
426 # Extract metabolite IDs from a reaction formula.
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
427 # Robust pattern: tokens ending with _<compartment> (e.g., _c, _m, _e),
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
428 # allowing leading digits/underscores.
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
429 # """
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
430 # metabolites = set()
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
431 # # optional coefficient followed by a token ending with _<letters>
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
432 # if reaction_formula[-1] == ']' and reaction_formula[-3] == '[':
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
433 # pattern = r'(?:\d+(?:\.\d+)?\s+)?([A-Za-z0-9_]+[[A-Za-z0-9]]+)'
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
434 # else:
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
435 # pattern = r'(?:\d+(?:\.\d+)?\s+)?([A-Za-z0-9_]+_[A-Za-z0-9]+)'
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
436 # matches = re.findall(pattern, reaction_formula)
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
437 # metabolites.update(matches)
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
438 # return metabolites
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
439
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
440
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
441 def extract_metabolites_from_reaction(reaction_formula: str) -> Set[str]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
442 """
500
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
443 Extract metabolite IDs from a reaction formula.
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
444
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
445 Handles:
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
446 - optional stoichiometric coefficients (integers or decimals)
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
447 - compartment tags at the end of the metabolite, either [c] or _c
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
448
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
449 Returns the IDs including the compartment suffix exactly as written.
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
450 """
499
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
451 pattern = re.compile(
500
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
452 r'(?:^|(?<=\s)|(?<=\+)|(?<=,)|(?<==)|(?<=:))' # left boundary (start, space, +, comma, =, :)
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
453 r'(?:\d+(?:\.\d+)?\s+)?' # optional coefficient (requires space after)
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
454 r'([A-Za-z0-9][A-Za-z0-9_]*(?:\[[A-Za-z0-9]+\]|_[A-Za-z0-9]+))' # metabolite + compartment (can start with number)
499
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
455 )
a2f7a6dd9d0b Uploaded
francesco_lapi
parents: 493
diff changeset
456 return {m.group(1) for m in pattern.finditer(reaction_formula)}
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
457
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
458
500
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
459
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
460 def extract_compartment_from_metabolite(metabolite_id: str) -> str:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
461 """Extract the compartment from a metabolite ID."""
500
4e7e67693ce7 Uploaded
francesco_lapi
parents: 499
diff changeset
462 if '_' == metabolite_id[-2]:
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
463 return metabolite_id.split('_')[-1]
493
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
464 if metabolite_id[-1] == ']' and metabolite_id[-3] == '[':
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
465 return metabolite_id[-2]
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
466 return 'c' # default cytoplasm
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
467
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
468
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
469 def parse_reaction_formula(reaction: Reaction, formula: str, metabolites_dict: Dict[str, Metabolite]):
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
470 """Parse a reaction formula and set metabolites with their coefficients."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
471
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
472 if '<=>' in formula:
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
473 parts = formula.split('<=>')
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
474 reversible = True
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
475 elif '<--' in formula:
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
476 parts = formula.split('<--')
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
477 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
478 elif '-->' in formula:
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
479 parts = formula.split('-->')
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
480 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
481 elif '<-' in formula:
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
482 parts = formula.split('<-')
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
483 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
484 else:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
485 raise ValueError(f"Unrecognized reaction format: {formula}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
486
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
487 # Handle cases where one side might be empty (exchange reactions)
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
488 if len(parts) != 2:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
489 raise ValueError(f"Invalid reaction format, expected 2 parts: {formula}")
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
490
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
491 left, right = parts[0].strip(), parts[1].strip()
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
492
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
493 reactants = parse_metabolites_side(left) if left else {}
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
494 products = parse_metabolites_side(right) if right else {}
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
495
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
496 metabolites_to_add = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
497
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
498 for met_id, coeff in reactants.items():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
499 if met_id in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
500 metabolites_to_add[metabolites_dict[met_id]] = -coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
501
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
502 for met_id, coeff in products.items():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
503 if met_id in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
504 metabolites_to_add[metabolites_dict[met_id]] = coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
505
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
506 reaction.add_metabolites(metabolites_to_add)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
507
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
508
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
509 def parse_metabolites_side(side_str: str) -> Dict[str, float]:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
510 """Parse one side of a reaction and extract metabolites with coefficients."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
511 metabolites = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
512 if not side_str or side_str.strip() == '':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
513 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
514
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
515 terms = side_str.split('+')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
516 for term in terms:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
517 term = term.strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
518 if not term:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
519 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
520
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
521 # First check if term has space-separated coefficient and metabolite
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
522 parts = term.split()
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
523 if len(parts) == 2:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
524 # Two parts: potential coefficient + metabolite
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
525 try:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
526 coeff = float(parts[0])
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
527 met_id = parts[1]
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
528 # Verify the second part looks like a metabolite with compartment
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
529 if re.match(r'[A-Za-z0-9_]+(?:\[[A-Za-z0-9]+\]|_[A-Za-z0-9]+)', met_id):
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
530 metabolites[met_id] = coeff
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
531 continue
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
532 except ValueError:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
533 pass
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
534
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
535 # Single term - check if it's a metabolite (no coefficient)
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
536 # Updated pattern to include metabolites starting with numbers
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
537 if re.match(r'[A-Za-z0-9][A-Za-z0-9_]*(?:\[[A-Za-z0-9]+\]|_[A-Za-z0-9]+)', term):
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
538 metabolites[term] = 1.0
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
539 else:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
540 print(f"Warning: Could not parse metabolite term: '{term}'")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
541
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
542 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
543
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
544
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
545
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
546 def set_objective_from_csv(model: cobra.Model, df: pd.DataFrame, obj_col: str = "ObjectiveCoefficient"):
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
547 """
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
548 Sets the model's objective function based on a column of coefficients in the CSV.
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
549 Can be any reaction(s), not necessarily biomass.
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
550 """
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
551 obj_dict = {}
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
552
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
553 for idx, row in df.iterrows():
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
554 reaction_id = str(row['ReactionID']).strip()
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
555 coeff = float(row[obj_col]) if pd.notna(row[obj_col]) else 0.0
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
556 if coeff != 0:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
557 if reaction_id in model.reactions:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
558 obj_dict[model.reactions.get_by_id(reaction_id)] = coeff
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
559 else:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
560 print(f"Warning: reaction {reaction_id} not found in model, skipping for objective.")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
561
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
562 if not obj_dict:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
563 raise ValueError("No reactions found with non-zero objective coefficient.")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
564
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
565 model.objective = obj_dict
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
566 print(f"Objective set with {len(obj_dict)} reactions.")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
567
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
568
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
569
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
570
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
571 def set_medium_from_data(model: cobraModel, df: pd.DataFrame):
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
572 """Set the medium based on the 'InMedium' column in the dataframe."""
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
573 if 'InMedium' not in df.columns:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
574 print("No 'InMedium' column found, skipping medium setup")
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
575 return
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
576
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
577 medium_reactions = df[df['InMedium'] == True]['ReactionID'].tolist()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
578
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
579 medium_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
580 for rxn_id in medium_reactions:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
581 if rxn_id in [r.id for r in model.reactions]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
582 reaction = model.reactions.get_by_id(rxn_id)
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
583 if reaction.lower_bound < 0:
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
584 medium_dict[rxn_id] = abs(reaction.lower_bound)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
585
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
586 if medium_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
587 model.medium = medium_dict
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
588 print(f"Medium set with {len(medium_dict)} components")
501
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
589 else:
9bfd1ec3ae6f Uploaded
francesco_lapi
parents: 500
diff changeset
590 print("No medium components found")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
591 def validate_model(model: cobraModel) -> Dict[str, any]:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
592 """Validate the model and return basic statistics."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
593 validation = {
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
594 'num_reactions': len(model.reactions),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
595 'num_metabolites': len(model.metabolites),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
596 'num_genes': len(model.genes),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
597 'num_compartments': len(model.compartments),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
598 'objective': str(model.objective),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
599 'medium_size': len(model.medium),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
600 'reversible_reactions': len([r for r in model.reactions if r.reversibility]),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
601 'exchange_reactions': len([r for r in model.reactions if r.id.startswith('EX_')]),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
602 }
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
603
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
604 try:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
605 # Growth test
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
606 solution = model.optimize()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
607 validation['growth_rate'] = solution.objective_value
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
608 validation['status'] = solution.status
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
609 except Exception as e:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
610 validation['growth_rate'] = None
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
611 validation['status'] = f"Error: {e}"
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
612
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
613 return validation
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
614
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
615 def convert_genes(model, annotation):
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
616 """Rename genes using a selected annotation key in gene.notes; returns a model copy."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
617 from cobra.manipulation import rename_genes
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
618 model2=model.copy()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
619 try:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
620 dict_genes={gene.id:gene.notes[annotation] for gene in model2.genes}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
621 except:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
622 print("No annotation in gene dict!")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
623 return -1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
624 rename_genes(model2,dict_genes)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
625
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
626 return model2
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
627
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
628 # ---------- Utility helpers ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
629 def _normalize_colname(col: str) -> str:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
630 return col.strip().lower().replace(' ', '_')
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
631
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
632 def _choose_columns(mapping_df: 'pd.DataFrame') -> Dict[str, str]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
633 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
634 Find useful columns and return a dict {ensg: colname1, hgnc_id: colname2, ...}.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
635 Raise ValueError if no suitable mapping is found.
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
636 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
637 cols = { _normalize_colname(c): c for c in mapping_df.columns }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
638 chosen = {}
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
639 # candidate names for each category
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
640 candidates = {
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
641 'ensg': ['ensg', 'ensembl_gene_id', 'ensembl'],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
642 'hgnc_id': ['hgnc_id', 'hgnc', 'hgnc:'],
444
06564187fba3 Uploaded
francesco_lapi
parents: 430
diff changeset
643 'hgnc_symbol': ['hgnc_symbol', 'hgnc symbol', 'symbol'],
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
644 'entrez_id': ['entrez', 'entrez_id', 'entrezgene'],
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
645 'gene_number': ['gene_number']
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
646 }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
647 for key, names in candidates.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
648 for n in names:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
649 if n in cols:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
650 chosen[key] = cols[n]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
651 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
652 return chosen
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
653
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
654 def _validate_target_uniqueness(mapping_df: 'pd.DataFrame',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
655 source_col: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
656 target_col: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
657 model_source_genes: Optional[Set[str]] = None,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
658 logger: Optional[logging.Logger] = None) -> None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
659 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
660 Check that, within the filtered mapping_df, each target maps to at most one source.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
661 Log examples if duplicates are found.
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
662 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
663 if logger is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
664 logger = logging.getLogger(__name__)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
665
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
666 if mapping_df.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
667 logger.warning("Mapping dataframe is empty for the requested source genes; skipping uniqueness validation.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
668 return
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
669
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
670 # normalize temporary columns for grouping (without altering the original df)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
671 tmp = mapping_df[[source_col, target_col]].copy()
503
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
672 tmp['_src_norm'] = tmp[source_col].astype(str).apply(_normalize_gene_id)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
673 tmp['_tgt_norm'] = tmp[target_col].astype(str).str.strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
674
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
675 # optionally filter to the set of model source genes
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
676 if model_source_genes is not None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
677 tmp = tmp[tmp['_src_norm'].isin(model_source_genes)]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
678
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
679 if tmp.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
680 logger.warning("After filtering to model source genes, mapping table is empty — nothing to validate.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
681 return
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
682
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
683 # build reverse mapping: target -> set(sources)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
684 grouped = tmp.groupby('_tgt_norm')['_src_norm'].agg(lambda s: set(s.dropna()))
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
685 # find targets with more than one source
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
686 problematic = {t: sorted(list(s)) for t, s in grouped.items() if len(s) > 1}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
687
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
688 if problematic:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
689 # prepare warning message with examples (limited subset)
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
690 sample_items = list(problematic.items())
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
691 msg_lines = ["Mapping validation failed: some target IDs are associated with multiple source IDs."]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
692 for tgt, sources in sample_items:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
693 msg_lines.append(f" - target '{tgt}' <- sources: {', '.join(sources)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
694 full_msg = "\n".join(msg_lines)
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
695 # log warning
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
696 logger.warning(full_msg)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
697
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
698 # if everything is fine
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
699 logger.info("Mapping validation passed: no target ID is associated with multiple source IDs (within filtered set).")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
700
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
701
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
702 def _normalize_gene_id(g: str) -> str:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
703 """Normalize a gene ID for use as a key (removes prefixes like 'HGNC:' and strips)."""
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
704 if g is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
705 return ""
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
706 g = str(g).strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
707 # remove common prefixes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
708 g = re.sub(r'^(HGNC:)', '', g, flags=re.IGNORECASE)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
709 g = re.sub(r'^(ENSG:)', '', g, flags=re.IGNORECASE)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
710 return g
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
711
493
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
712 def _is_or_only_expression(expr: str) -> bool:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
713 """
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
714 Check if a GPR expression contains only OR operators (no AND operators).
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
715
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
716 Args:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
717 expr: GPR expression string
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
718
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
719 Returns:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
720 bool: True if expression contains only OR (and parentheses) and has multiple genes, False otherwise
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
721 """
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
722 if not expr or not expr.strip():
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
723 return False
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
724
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
725 # Normalize the expression
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
726 normalized = expr.replace(' AND ', ' and ').replace(' OR ', ' or ')
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
727
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
728 # Check if it contains any AND operators
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
729 has_and = ' and ' in normalized.lower()
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
730
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
731 # Check if it contains OR operators
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
732 has_or = ' or ' in normalized.lower()
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
733
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
734 # Must have OR operators and no AND operators
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
735 return has_or and not has_and
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
736
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
737
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
738 def _flatten_or_only_gpr(expr: str) -> str:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
739 """
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
740 Flatten a GPR expression that contains only OR operators by:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
741 1. Removing all parentheses
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
742 2. Extracting unique gene names
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
743 3. Joining them with ' or '
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
744
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
745 Args:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
746 expr: GPR expression string with only OR operators
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
747
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
748 Returns:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
749 str: Flattened GPR expression
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
750 """
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
751 if not expr or not expr.strip():
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
752 return expr
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
753
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
754 # Extract all gene tokens (exclude logical operators and parentheses)
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
755 gene_pattern = r'\b[A-Za-z0-9:_.-]+\b'
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
756 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
757
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
758 tokens = re.findall(gene_pattern, expr)
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
759 genes = [t for t in tokens if t not in logical]
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
760
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
761 # Create set to remove duplicates, then convert back to list to maintain some order
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
762 unique_genes = list(dict.fromkeys(genes)) # Preserves insertion order
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
763
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
764 if len(unique_genes) == 0:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
765 return expr
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
766 elif len(unique_genes) == 1:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
767 return unique_genes[0]
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
768 else:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
769 return ' or '.join(unique_genes)
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
770
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
771
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
772 def _simplify_boolean_expression(expr: str) -> str:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
773 """
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
774 Simplify a boolean expression by removing duplicates while strictly preserving semantics.
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
775 This function handles simple duplicates within parentheses while being conservative about
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
776 complex expressions that could change semantics.
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
777 """
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
778 if not expr or not expr.strip():
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
779 return expr
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
780
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
781 # Normalize operators and whitespace
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
782 expr = expr.replace(' AND ', ' and ').replace(' OR ', ' or ')
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
783 expr = ' '.join(expr.split()) # Normalize whitespace
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
784
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
785 def simplify_parentheses_content(match_obj):
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
786 """Helper function to simplify content within parentheses."""
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
787 content = match_obj.group(1) # Content inside parentheses
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
788
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
789 # Only simplify if it's a pure OR or pure AND chain
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
790 if ' or ' in content and ' and ' not in content:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
791 # Pure OR chain - safe to deduplicate
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
792 parts = [p.strip() for p in content.split(' or ') if p.strip()]
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
793 unique_parts = []
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
794 seen = set()
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
795 for part in parts:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
796 if part not in seen:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
797 unique_parts.append(part)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
798 seen.add(part)
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
799
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
800 if len(unique_parts) == 1:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
801 return unique_parts[0] # Remove unnecessary parentheses for single items
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
802 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
803 return '(' + ' or '.join(unique_parts) + ')'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
804
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
805 elif ' and ' in content and ' or ' not in content:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
806 # Pure AND chain - safe to deduplicate
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
807 parts = [p.strip() for p in content.split(' and ') if p.strip()]
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
808 unique_parts = []
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
809 seen = set()
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
810 for part in parts:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
811 if part not in seen:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
812 unique_parts.append(part)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
813 seen.add(part)
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
814
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
815 if len(unique_parts) == 1:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
816 return unique_parts[0] # Remove unnecessary parentheses for single items
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
817 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
818 return '(' + ' and '.join(unique_parts) + ')'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
819 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
820 # Mixed operators or single item - return with parentheses as-is
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
821 return '(' + content + ')'
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
822
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
823 def remove_duplicates_simple(parts_str: str, separator: str) -> str:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
824 """Remove duplicates from a simple chain of operations."""
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
825 parts = [p.strip() for p in parts_str.split(separator) if p.strip()]
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
826
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
827 # Remove duplicates while preserving order
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
828 unique_parts = []
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
829 seen = set()
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
830 for part in parts:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
831 if part not in seen:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
832 unique_parts.append(part)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
833 seen.add(part)
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
834
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
835 if len(unique_parts) == 1:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
836 return unique_parts[0]
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
837 else:
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
838 return f' {separator} '.join(unique_parts)
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
839
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
840 try:
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
841 import re
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
842
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
843 # First, simplify content within parentheses
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
844 # This handles cases like (A or A) -> A and (B and B) -> B
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
845 expr_simplified = re.sub(r'\(([^()]+)\)', simplify_parentheses_content, expr)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
846
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
847 # Check if the resulting expression has mixed operators
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
848 has_and = ' and ' in expr_simplified
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
849 has_or = ' or ' in expr_simplified
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
850
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
851 # Only simplify top-level if it's pure AND or pure OR
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
852 if has_and and not has_or and '(' not in expr_simplified:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
853 # Pure AND chain at top level - safe to deduplicate
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
854 return remove_duplicates_simple(expr_simplified, 'and')
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
855 elif has_or and not has_and and '(' not in expr_simplified:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
856 # Pure OR chain at top level - safe to deduplicate
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
857 return remove_duplicates_simple(expr_simplified, 'or')
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
858 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
859 # Mixed operators or has parentheses - return the simplified version (with parentheses content cleaned)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
860 return expr_simplified
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
861
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
862 except Exception:
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
863 # If anything goes wrong, return the original expression
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
864 return expr
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
865
492
4ed95023af20 Uploaded
francesco_lapi
parents: 490
diff changeset
866
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
867 def translate_model_genes(model: 'cobra.Model',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
868 mapping_df: 'pd.DataFrame',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
869 target_nomenclature: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
870 source_nomenclature: str = 'hgnc_id',
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
871 allow_many_to_one: bool = False,
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
872 logger: Optional[logging.Logger] = None) -> Tuple['cobra.Model', Dict[str, str]]:
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
873 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
874 Translate model genes from source_nomenclature to target_nomenclature using a mapping table.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
875 mapping_df should contain columns enabling mapping (e.g., ensg, hgnc_id, hgnc_symbol, entrez).
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
876
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
877 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
878 model: COBRA model to translate.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
879 mapping_df: DataFrame containing the mapping information.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
880 target_nomenclature: Desired target key (e.g., 'hgnc_symbol').
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
881 source_nomenclature: Current source key in the model (default 'hgnc_id').
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
882 allow_many_to_one: If True, allow many-to-one mappings and handle duplicates in GPRs.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
883 logger: Optional logger.
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
884
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
885 Returns:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
886 Tuple containing:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
887 - Translated COBRA model
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
888 - Dictionary mapping reaction IDs to translation issue descriptions
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
889 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
890 if logger is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
891 logging.basicConfig(level=logging.INFO, format='%(asctime)s - %(levelname)s - %(message)s')
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
892 logger = logging.getLogger(__name__)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
893
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
894 logger.info(f"Translating genes from '{source_nomenclature}' to '{target_nomenclature}'")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
895
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
896 # normalize column names and choose relevant columns
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
897 chosen = _choose_columns(mapping_df)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
898 if not chosen:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
899 raise ValueError("Could not detect useful columns in mapping_df. Expected at least one of: ensg, hgnc_id, hgnc_symbol, entrez.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
900
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
901 # map source/target to actual dataframe column names (allow user-specified source/target keys)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
902 # normalize input args
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
903 src_key = source_nomenclature.strip().lower()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
904 tgt_key = target_nomenclature.strip().lower()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
905
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
906 # try to find the actual column names for requested keys
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
907 col_for_src = None
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
908 col_for_tgt = None
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
909 # first, try exact match
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
910 for k, actual in chosen.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
911 if k == src_key:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
912 col_for_src = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
913 if k == tgt_key:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
914 col_for_tgt = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
915
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
916 # if not found, try mapping common names
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
917 if col_for_src is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
918 possible_src_names = {k: v for k, v in chosen.items()}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
919 # try to match by contained substring
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
920 for k, actual in possible_src_names.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
921 if src_key in k:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
922 col_for_src = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
923 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
924
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
925 if col_for_tgt is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
926 for k, actual in chosen.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
927 if tgt_key in k:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
928 col_for_tgt = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
929 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
930
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
931 if col_for_src is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
932 raise ValueError(f"Source column for '{source_nomenclature}' not found in mapping dataframe.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
933 if col_for_tgt is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
934 raise ValueError(f"Target column for '{target_nomenclature}' not found in mapping dataframe.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
935
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
936 model_source_genes = { _normalize_gene_id(g.id) for g in model.genes }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
937 logger.info(f"Filtering mapping to {len(model_source_genes)} source genes present in model (normalized).")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
938
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
939 tmp_map = mapping_df[[col_for_src, col_for_tgt]].dropna().copy()
503
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
940 tmp_map[col_for_src + "_norm"] = tmp_map[col_for_src].astype(str).apply(_normalize_gene_id)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
941
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
942 filtered_map = tmp_map[tmp_map[col_for_src + "_norm"].isin(model_source_genes)].copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
943
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
944 if filtered_map.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
945 logger.warning("No mapping rows correspond to source genes present in the model after filtering. Proceeding with empty mapping (no translation will occur).")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
946
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
947 if not allow_many_to_one:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
948 _validate_target_uniqueness(filtered_map, col_for_src, col_for_tgt, model_source_genes=model_source_genes, logger=logger)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
949
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
950 # Crea il mapping
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
951 gene_mapping = _create_gene_mapping(filtered_map, col_for_src, col_for_tgt, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
952
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
953 # copy model
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
954 model_copy = model.copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
955
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
956 # statistics
493
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
957 stats = {'translated': 0, 'one_to_one': 0, 'one_to_many': 0, 'not_found': 0, 'simplified_gprs': 0, 'flattened_or_gprs': 0}
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
958 unmapped = []
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
959 multi = []
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
960
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
961 # Dictionary to store translation issues per reaction
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
962 reaction_translation_issues = {}
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
963
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
964 original_genes = {g.id for g in model_copy.genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
965 logger.info(f"Original genes count: {len(original_genes)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
966
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
967 # translate GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
968 for rxn in model_copy.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
969 gpr = rxn.gene_reaction_rule
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
970 if gpr and gpr.strip():
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
971 new_gpr, rxn_issues = _translate_gpr(gpr, gene_mapping, stats, unmapped, multi, logger, track_issues=True)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
972 if rxn_issues:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
973 reaction_translation_issues[rxn.id] = rxn_issues
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
974
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
975 if new_gpr != gpr:
493
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
976 # Check if this GPR has translation issues and contains only OR operators
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
977 if rxn_issues and _is_or_only_expression(new_gpr):
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
978 # Flatten the GPR: remove parentheses and create set of unique genes
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
979 flattened_gpr = _flatten_or_only_gpr(new_gpr)
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
980 if flattened_gpr != new_gpr:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
981 stats['flattened_or_gprs'] += 1
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
982 logger.debug(f"Flattened OR-only GPR with issues for {rxn.id}: '{new_gpr}' -> '{flattened_gpr}'")
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
983 new_gpr = flattened_gpr
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
984
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
985 simplified_gpr = _simplify_boolean_expression(new_gpr)
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
986 if simplified_gpr != new_gpr:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
987 stats['simplified_gprs'] += 1
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
988 logger.debug(f"Simplified GPR for {rxn.id}: '{new_gpr}' -> '{simplified_gpr}'")
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
989 rxn.gene_reaction_rule = simplified_gpr
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
990 logger.debug(f"Reaction {rxn.id}: '{gpr}' -> '{simplified_gpr}'")
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
991
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
992 # update model genes based on new GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
993 _update_model_genes(model_copy, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
994
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
995 # final logging
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
996 _log_translation_statistics(stats, unmapped, multi, original_genes, model_copy.genes, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
997
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
998 logger.info("Translation finished")
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
999 return model_copy, reaction_translation_issues
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1000
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1001
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1002 # ---------- helper functions ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1003 def _create_gene_mapping(mapping_df, source_col: str, target_col: str, logger: logging.Logger) -> Dict[str, List[str]]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1004 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1005 Build mapping dict: source_id -> list of target_ids
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1006 Normalizes IDs (removes prefixes like 'HGNC:' etc).
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1007 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1008 df = mapping_df[[source_col, target_col]].dropna().copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1009 # normalize to string
503
8dd07e59f631 Uploaded
francesco_lapi
parents: 501
diff changeset
1010 df[source_col] = df[source_col].astype(str).apply(_normalize_gene_id)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1011 df[target_col] = df[target_col].astype(str).str.strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1012
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1013 df = df.drop_duplicates()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1014
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1015 logger.info(f"Creating mapping from {len(df)} rows")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1016
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1017 mapping = defaultdict(list)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1018 for _, row in df.iterrows():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1019 s = row[source_col]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1020 t = row[target_col]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1021 if t not in mapping[s]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1022 mapping[s].append(t)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1023
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1024 # stats
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1025 one_to_one = sum(1 for v in mapping.values() if len(v) == 1)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1026 one_to_many = sum(1 for v in mapping.values() if len(v) > 1)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1027 logger.info(f"Mapping: {len(mapping)} source keys, {one_to_one} 1:1, {one_to_many} 1:many")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1028 return dict(mapping)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1029
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1030
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1031 def _translate_gpr(gpr_string: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1032 gene_mapping: Dict[str, List[str]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1033 stats: Dict[str, int],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1034 unmapped_genes: List[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1035 multi_mapping_genes: List[Tuple[str, List[str]]],
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1036 logger: logging.Logger,
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1037 track_issues: bool = False) -> Union[str, Tuple[str, str]]:
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1038 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1039 Translate genes inside a GPR string using gene_mapping.
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1040 Returns new GPR string, and optionally translation issues if track_issues=True.
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1041 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1042 # Generic token pattern: letters, digits, :, _, -, ., (captures HGNC:1234, ENSG000..., symbols)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1043 token_pattern = r'\b[A-Za-z0-9:_.-]+\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1044 tokens = re.findall(token_pattern, gpr_string)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1045
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1046 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1047 tokens = [t for t in tokens if t not in logical]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1048
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1049 new_gpr = gpr_string
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1050 issues = []
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1051
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1052 for token in sorted(set(tokens), key=lambda x: -len(x)): # longer tokens first to avoid partial replacement
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1053 norm = _normalize_gene_id(token)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1054 if norm in gene_mapping:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1055 targets = gene_mapping[norm]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1056 stats['translated'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1057 if len(targets) == 1:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1058 stats['one_to_one'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1059 replacement = targets[0]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1060 else:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1061 stats['one_to_many'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1062 multi_mapping_genes.append((token, targets))
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1063 replacement = "(" + " or ".join(targets) + ")"
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1064 if track_issues:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1065 issues.append(f"{token} -> {' or '.join(targets)}")
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1066
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1067 pattern = r'\b' + re.escape(token) + r'\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1068 new_gpr = re.sub(pattern, replacement, new_gpr)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1069 else:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1070 stats['not_found'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1071 if token not in unmapped_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1072 unmapped_genes.append(token)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1073 logger.debug(f"Token not found in mapping (left as-is): {token}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1074
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1075 # Check for many-to-one cases (multiple source genes mapping to same target)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1076 if track_issues:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1077 # Build reverse mapping to detect many-to-one cases from original tokens
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1078 original_to_target = {}
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1079
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1080 for orig_token in tokens:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1081 norm = _normalize_gene_id(orig_token)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1082 if norm in gene_mapping:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1083 targets = gene_mapping[norm]
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1084 for target in targets:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1085 if target not in original_to_target:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1086 original_to_target[target] = []
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1087 if orig_token not in original_to_target[target]:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1088 original_to_target[target].append(orig_token)
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1089
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1090 # Identify many-to-one mappings in this specific GPR
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1091 for target, original_genes in original_to_target.items():
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1092 if len(original_genes) > 1:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1093 issues.append(f"{' or '.join(original_genes)} -> {target}")
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1094
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1095 issue_text = "; ".join(issues) if issues else ""
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1096
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1097 if track_issues:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1098 return new_gpr, issue_text
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1099 else:
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1100 return new_gpr
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1101
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1102
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1103 def _update_model_genes(model: 'cobra.Model', logger: logging.Logger):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1104 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1105 Rebuild model.genes from gene_reaction_rule content.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1106 Removes genes not referenced and adds missing ones.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1107 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1108 # collect genes in GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1109 gene_pattern = r'\b[A-Za-z0-9:_.-]+\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1110 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1111 genes_in_gpr: Set[str] = set()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1112
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1113 for rxn in model.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1114 gpr = rxn.gene_reaction_rule
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1115 if gpr and gpr.strip():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1116 toks = re.findall(gene_pattern, gpr)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1117 toks = [t for t in toks if t not in logical]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1118 # normalize IDs consistent with mapping normalization
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1119 toks = [_normalize_gene_id(t) for t in toks]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1120 genes_in_gpr.update(toks)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1121
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1122 # existing gene ids
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1123 existing = {g.id for g in model.genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1124
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1125 # remove obsolete genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1126 to_remove = [gid for gid in existing if gid not in genes_in_gpr]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1127 removed = 0
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1128 for gid in to_remove:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1129 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1130 gene_obj = model.genes.get_by_id(gid)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1131 model.genes.remove(gene_obj)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1132 removed += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1133 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1134 # safe-ignore
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1135 pass
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1136
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1137 # add new genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1138 added = 0
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1139 for gid in genes_in_gpr:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1140 if gid not in existing:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1141 new_gene = cobra.Gene(gid)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1142 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1143 model.genes.add(new_gene)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1144 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1145 # fallback: if model.genes doesn't support add, try append or model.add_genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1146 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1147 model.genes.append(new_gene)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1148 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1149 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1150 model.add_genes([new_gene])
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1151 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1152 logger.warning(f"Could not add gene object for {gid}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1153 added += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1154
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1155 logger.info(f"Model genes updated: removed {removed}, added {added}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1156
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1157
508
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1158 def export_model_to_tabular(model: cobraModel,
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1159 output_path: str,
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1160 translation_issues: Dict = None,
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1161 include_objective: bool = True,
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1162 save_function = None) -> pd.DataFrame:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1163 """
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1164 Export a COBRA model to tabular format with optional components.
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1165
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1166 Args:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1167 model: COBRA model to export
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1168 output_path: Path where to save the tabular file
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1169 translation_issues: Optional dict of {reaction_id: issues} from gene translation
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1170 include_objective: Whether to include objective coefficient column
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1171 save_function: Optional custom save function, if None uses pd.DataFrame.to_csv
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1172
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1173 Returns:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1174 pd.DataFrame: The merged tabular data
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1175 """
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1176 # Generate model data
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1177 rules = generate_rules(model, asParsed=False)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1178
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1179 reactions = generate_reactions(model, asParsed=False)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1180 bounds = generate_bounds(model)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1181 medium = get_medium(model)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1182 compartments = generate_compartments(model)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1183
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1184 # Create base DataFrames
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1185 df_rules = pd.DataFrame(list(rules.items()), columns=["ReactionID", "GPR"])
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1186 df_reactions = pd.DataFrame(list(reactions.items()), columns=["ReactionID", "Formula"])
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1187 df_bounds = bounds.reset_index().rename(columns={"index": "ReactionID"})
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1188 df_medium = medium.rename(columns={"reaction": "ReactionID"})
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1189 df_medium["InMedium"] = True
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1190
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1191 # Start merging
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1192 merged = df_reactions.merge(df_rules, on="ReactionID", how="outer")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1193 merged = merged.merge(df_bounds, on="ReactionID", how="outer")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1194
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1195 # Add objective coefficients if requested
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1196 if include_objective:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1197 objective_function = extract_objective_coefficients(model)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1198 merged = merged.merge(objective_function, on="ReactionID", how="outer")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1199
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1200 # Add compartments/pathways if they exist
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1201 if compartments is not None:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1202 merged = merged.merge(compartments, on="ReactionID", how="outer")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1203
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1204 # Add medium information
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1205 merged = merged.merge(df_medium, on="ReactionID", how="left")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1206
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1207 # Add translation issues if provided
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1208 if translation_issues:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1209 df_translation_issues = pd.DataFrame([
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1210 {"ReactionID": rxn_id, "TranslationIssues": issues}
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1211 for rxn_id, issues in translation_issues.items()
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1212 ])
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1213 if not df_translation_issues.empty:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1214 merged = merged.merge(df_translation_issues, on="ReactionID", how="left")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1215 merged["TranslationIssues"] = merged["TranslationIssues"].fillna("")
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1216
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1217 # Final processing
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1218 merged["InMedium"] = merged["InMedium"].fillna(False)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1219 merged = merged.sort_values(by="InMedium", ascending=False)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1220
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1221 # Save the file
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1222 if save_function:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1223 save_function(merged, output_path)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1224 else:
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1225 merged.to_csv(output_path, sep="\t", index=False)
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1226
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1227 return merged
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1228
ca98c149ec61 Uploaded
francesco_lapi
parents: 506
diff changeset
1229
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1230 def _log_translation_statistics(stats: Dict[str, int],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1231 unmapped_genes: List[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1232 multi_mapping_genes: List[Tuple[str, List[str]]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1233 original_genes: Set[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1234 final_genes,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1235 logger: logging.Logger):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1236 logger.info("=== TRANSLATION STATISTICS ===")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1237 logger.info(f"Translated: {stats.get('translated', 0)} (1:1 = {stats.get('one_to_one', 0)}, 1:many = {stats.get('one_to_many', 0)})")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1238 logger.info(f"Not found tokens: {stats.get('not_found', 0)}")
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
1239 logger.info(f"Simplified GPRs: {stats.get('simplified_gprs', 0)}")
493
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
1240 logger.info(f"Flattened OR-only GPRs with issues: {stats.get('flattened_or_gprs', 0)}")
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1241
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1242 final_ids = {g.id for g in final_genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1243 logger.info(f"Genes in model: {len(original_genes)} -> {len(final_ids)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1244
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1245 if unmapped_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1246 logger.warning(f"Unmapped tokens ({len(unmapped_genes)}): {', '.join(unmapped_genes[:20])}{(' ...' if len(unmapped_genes)>20 else '')}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1247 if multi_mapping_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1248 logger.info(f"Multi-mapping examples ({len(multi_mapping_genes)}):")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
1249 for orig, targets in multi_mapping_genes[:10]:
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1250 logger.info(f" {orig} -> {', '.join(targets)}")
493
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
1251
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
1252 # Log summary of flattened GPRs if any
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
1253 if stats.get('flattened_or_gprs', 0) > 0:
a92d21f92956 Uploaded
francesco_lapi
parents: 492
diff changeset
1254 logger.info(f"Flattened {stats['flattened_or_gprs']} OR-only GPRs that had translation issues (removed parentheses, created unique gene sets)")
490
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1255
c6ea189ea7e9 Uploaded
francesco_lapi
parents: 456
diff changeset
1256