annotate COBRAxy/utils/model_utils.py @ 456:a6e45049c1b9 draft default tip

Uploaded
author francesco_lapi
date Fri, 12 Sep 2025 17:28:45 +0000
parents 4e2bc80764b6
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
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
21
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
22 ################################- DATA GENERATION -################################
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
23 ReactionId = str
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
24 def generate_rules(model: cobraModel, *, asParsed = True) -> Union[Dict[ReactionId, rulesUtils.OpList], Dict[ReactionId, str]]:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
25 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
26 Generate a dictionary mapping reaction IDs to GPR rules from the model.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
27
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
28 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
29 model: COBRA model to derive data from.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
30 asParsed: If True, parse rules into a nested list structure; otherwise keep raw strings.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
31
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
32 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
33 Dict[ReactionId, rulesUtils.OpList]: Parsed rules by reaction ID.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
34 Dict[ReactionId, str]: Raw rules by reaction ID.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
35 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
36 _ruleGetter = lambda reaction : reaction.gene_reaction_rule
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
37 ruleExtractor = (lambda reaction :
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
38 rulesUtils.parseRuleToNestedList(_ruleGetter(reaction))) if asParsed else _ruleGetter
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
39
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
40 return {
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
41 reaction.id : ruleExtractor(reaction)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
42 for reaction in model.reactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
43 if reaction.gene_reaction_rule }
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
44
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
45 def generate_reactions(model :cobraModel, *, asParsed = True) -> Dict[ReactionId, str]:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
46 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
47 Generate a dictionary mapping reaction IDs to reaction formulas from the model.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
48
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
49 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
50 model: COBRA model to derive data from.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
51 asParsed: If True, convert formulas into a parsed representation; otherwise keep raw strings.
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
52
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
53 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
54 Dict[ReactionId, str]: Reactions by reaction ID (parsed if requested).
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
55 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
56
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
57 unparsedReactions = {
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
58 reaction.id : reaction.reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
59 for reaction in model.reactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
60 if reaction.reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
61 }
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
62
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
63 if not asParsed: return unparsedReactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
64
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
65 return reactionUtils.create_reaction_dict(unparsedReactions)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
66
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
67 def get_medium(model:cobraModel) -> pd.DataFrame:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
68 """
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
69 Extract the uptake reactions representing the model medium.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
70
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
71 Returns a DataFrame with a single column 'reaction' listing exchange reactions
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
72 with negative lower bound and no positive stoichiometric coefficients (uptake only).
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
73 """
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
74 trueMedium=[]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
75 for r in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
76 positiveCoeff=0
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
77 for m in r.metabolites:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
78 if r.get_coefficient(m.id)>0:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
79 positiveCoeff=1;
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
80 if (positiveCoeff==0 and r.lower_bound<0):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
81 trueMedium.append(r.id)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
82
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
83 df_medium = pd.DataFrame()
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
84 df_medium["reaction"] = trueMedium
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
85 return df_medium
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
86
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
87 def extract_objective_coefficients(model: cobraModel) -> pd.DataFrame:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
88 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
89 Extract objective coefficients for each reaction.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
90
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
91 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
92 model: COBRA model
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
93
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
94 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
95 pd.DataFrame with columns: ReactionID, ObjectiveCoefficient
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
96 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
97 coeffs = []
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
98 # model.objective.expression is a linear expression
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
99 objective_expr = model.objective.expression.as_coefficients_dict()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
100
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
101 for reaction in model.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
102 coeff = objective_expr.get(reaction.forward_variable, 0.0)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
103 coeffs.append({
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
104 "ReactionID": reaction.id,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
105 "ObjectiveCoefficient": coeff
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
106 })
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
107
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
108 return pd.DataFrame(coeffs)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
109
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
110 def generate_bounds(model:cobraModel) -> pd.DataFrame:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
111 """
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
112 Build a DataFrame of lower/upper bounds for all reactions.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
113
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
114 Returns:
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
115 pd.DataFrame indexed by reaction IDs with columns ['lower_bound', 'upper_bound'].
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
116 """
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
117
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
118 rxns = []
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
119 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
120 rxns.append(reaction.id)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
121
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
122 bounds = pd.DataFrame(columns = ["lower_bound", "upper_bound"], index=rxns)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
123
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
124 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
125 bounds.loc[reaction.id] = [reaction.lower_bound, reaction.upper_bound]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
126 return bounds
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
127
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
128
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
129
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
130 def generate_compartments(model: cobraModel) -> pd.DataFrame:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
131 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
132 Generates a DataFrame containing compartment information for each reaction.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
133 Creates columns for each compartment position (Compartment_1, Compartment_2, etc.)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
134
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
135 Args:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
136 model: the COBRA model to extract compartment data from.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
137
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
138 Returns:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
139 pd.DataFrame: DataFrame with ReactionID and compartment columns
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
140 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
141 pathway_data = []
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
142
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
143 # First pass: determine the maximum number of pathways any reaction has
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
144 max_pathways = 0
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
145 reaction_pathways = {}
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
146
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
147 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
148 # Get unique pathways from all metabolites in the reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
149 if type(reaction.annotation['pathways']) == list:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
150 reaction_pathways[reaction.id] = reaction.annotation['pathways']
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
151 max_pathways = max(max_pathways, len(reaction.annotation['pathways']))
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
152 else:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
153 reaction_pathways[reaction.id] = [reaction.annotation['pathways']]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
154
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
155 # Create column names for pathways
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
156 pathway_columns = [f"Pathway_{i+1}" for i in range(max_pathways)]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
157
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
158 # Second pass: create the data
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
159 for reaction_id, pathways in reaction_pathways.items():
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
160 row = {"ReactionID": reaction_id}
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
161
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
162 # Fill pathway columns
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
163 for i in range(max_pathways):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
164 col_name = pathway_columns[i]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
165 if i < len(pathways):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
166 row[col_name] = pathways[i]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
167 else:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
168 row[col_name] = None # or "" if you prefer empty strings
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
169
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
170 pathway_data.append(row)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
171
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
172 return pd.DataFrame(pathway_data)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
173
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
174
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
175
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
176 def build_cobra_model_from_csv(csv_path: str, model_id: str = "new_model") -> cobraModel:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
177 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
178 Build a COBRApy model from a tabular file with reaction data.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
179
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
180 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
181 csv_path: Path to the tab-separated file.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
182 model_id: ID for the newly created model.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
183
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
184 Returns:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
185 cobra.Model: The constructed COBRApy model.
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
186 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
187
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
188 df = pd.read_csv(csv_path, sep='\t')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
189
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
190 model = cobraModel(model_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
191
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
192 metabolites_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
193 compartments_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
194
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
195 print(f"Building model from {len(df)} reactions...")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
196
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
197 for idx, row in df.iterrows():
448
f8b1761eee37 Uploaded
francesco_lapi
parents: 444
diff changeset
198 reaction_formula = str(row['Formula']).strip()
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
199 if not reaction_formula or reaction_formula == 'nan':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
200 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
201
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
202 metabolites = extract_metabolites_from_reaction(reaction_formula)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
203
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
204 for met_id in metabolites:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
205 compartment = extract_compartment_from_metabolite(met_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
206
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
207 if compartment not in compartments_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
208 compartments_dict[compartment] = compartment
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
209
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
210 if met_id not in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
211 metabolites_dict[met_id] = Metabolite(
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
212 id=met_id,
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
213 compartment=compartment,
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
214 name=met_id.replace(f"_{compartment}", "").replace("__", "_")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
215 )
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
216
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
217 model.compartments = compartments_dict
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
218
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
219 model.add_metabolites(list(metabolites_dict.values()))
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
220
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
221 print(f"Added {len(metabolites_dict)} metabolites and {len(compartments_dict)} compartments")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
222
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
223 reactions_added = 0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
224 reactions_skipped = 0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
225
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
226 for idx, row in df.iterrows():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
227
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
228 reaction_id = str(row['ReactionID']).strip()
427
4a385fdb9e58 Uploaded
francesco_lapi
parents: 426
diff changeset
229 reaction_formula = str(row['Formula']).strip()
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
230
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
231 if not reaction_formula or reaction_formula == 'nan':
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
232 raise ValueError(f"Missing reaction formula for {reaction_id}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
233
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
234 reaction = Reaction(reaction_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
235 reaction.name = reaction_id
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
236
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
237 reaction.lower_bound = float(row['lower_bound']) if pd.notna(row['lower_bound']) else -1000.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
238 reaction.upper_bound = float(row['upper_bound']) if pd.notna(row['upper_bound']) else 1000.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
239
427
4a385fdb9e58 Uploaded
francesco_lapi
parents: 426
diff changeset
240 if pd.notna(row['GPR']) and str(row['GPR']).strip():
4a385fdb9e58 Uploaded
francesco_lapi
parents: 426
diff changeset
241 reaction.gene_reaction_rule = str(row['GPR']).strip()
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
242
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
243 try:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
244 parse_reaction_formula(reaction, reaction_formula, metabolites_dict)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
245 except Exception as e:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
246 print(f"Error parsing reaction {reaction_id}: {e}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
247 reactions_skipped += 1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
248 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
249
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
250 model.add_reactions([reaction])
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
251 reactions_added += 1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
252
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
253
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
254 print(f"Added {reactions_added} reactions, skipped {reactions_skipped} reactions")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
255
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
256 # set objective function
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
257 set_objective_from_csv(model, df, obj_col="ObjectiveCoefficient")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
258
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
259 set_medium_from_data(model, df)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
260
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
261 print(f"Model completed: {len(model.reactions)} reactions, {len(model.metabolites)} metabolites")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
262
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
263 return model
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
264
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
265
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
266 # Estrae tutti gli ID metaboliti nella formula (gestisce prefissi numerici + underscore)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
267 def extract_metabolites_from_reaction(reaction_formula: str) -> Set[str]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
268 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
269 Extract metabolite IDs from a reaction formula.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
270 Robust pattern: tokens ending with _<compartment> (e.g., _c, _m, _e),
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
271 allowing leading digits/underscores.
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
272 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
273 metabolites = set()
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
274 # optional coefficient followed by a token ending with _<letters>
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
275 pattern = r'(?:\d+(?:\.\d+)?\s+)?([A-Za-z0-9_]+_[a-z]+)'
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
276 matches = re.findall(pattern, reaction_formula)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
277 metabolites.update(matches)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
278 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
279
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
280
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
281 def extract_compartment_from_metabolite(metabolite_id: str) -> str:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
282 """Extract the compartment from a metabolite ID."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
283 if '_' in metabolite_id:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
284 return metabolite_id.split('_')[-1]
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
285 return 'c' # default cytoplasm
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
286
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
287
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
288 def parse_reaction_formula(reaction: Reaction, formula: str, metabolites_dict: Dict[str, Metabolite]):
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
289 """Parse a reaction formula and set metabolites with their coefficients."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
290
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
291 if '<=>' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
292 left, right = formula.split('<=>')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
293 reversible = True
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
294 elif '<--' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
295 left, right = formula.split('<--')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
296 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
297 elif '-->' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
298 left, right = formula.split('-->')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
299 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
300 elif '<-' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
301 left, right = formula.split('<-')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
302 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
303 else:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
304 raise ValueError(f"Unrecognized reaction format: {formula}")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
305
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
306 reactants = parse_metabolites_side(left.strip())
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
307 products = parse_metabolites_side(right.strip())
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
308
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
309 metabolites_to_add = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
310
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
311 for met_id, coeff in reactants.items():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
312 if met_id in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
313 metabolites_to_add[metabolites_dict[met_id]] = -coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
314
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
315 for met_id, coeff in products.items():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
316 if met_id in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
317 metabolites_to_add[metabolites_dict[met_id]] = coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
318
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
319 reaction.add_metabolites(metabolites_to_add)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
320
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
321
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
322 def parse_metabolites_side(side_str: str) -> Dict[str, float]:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
323 """Parse one side of a reaction and extract metabolites with coefficients."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
324 metabolites = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
325 if not side_str or side_str.strip() == '':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
326 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
327
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
328 terms = side_str.split('+')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
329 for term in terms:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
330 term = term.strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
331 if not term:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
332 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
333
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
334 # optional coefficient + id ending with _<compartment>
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
335 match = re.match(r'(?:(\d+\.?\d*)\s+)?([A-Za-z0-9_]+_[a-z]+)', term)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
336 if match:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
337 coeff_str, met_id = match.groups()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
338 coeff = float(coeff_str) if coeff_str else 1.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
339 metabolites[met_id] = coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
340
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
341 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
342
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
343
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
344
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
345 def set_objective_from_csv(model: cobra.Model, df: pd.DataFrame, obj_col: str = "ObjectiveCoefficient"):
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
346 """
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
347 Sets the model's objective function based on a column of coefficients in the CSV.
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
348 Can be any reaction(s), not necessarily biomass.
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
349 """
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
350 obj_dict = {}
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
351
430
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
352 for idx, row in df.iterrows():
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
353 reaction_id = str(row['ReactionID']).strip()
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
354 coeff = float(row[obj_col]) if pd.notna(row[obj_col]) else 0.0
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
355 if coeff != 0:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
356 if reaction_id in model.reactions:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
357 obj_dict[model.reactions.get_by_id(reaction_id)] = coeff
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
358 else:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
359 print(f"Warning: reaction {reaction_id} not found in model, skipping for objective.")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
360
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
361 if not obj_dict:
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
362 raise ValueError("No reactions found with non-zero objective coefficient.")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
363
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
364 model.objective = obj_dict
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
365 print(f"Objective set with {len(obj_dict)} reactions.")
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
366
f49c951c9fe6 Uploaded
francesco_lapi
parents: 427
diff changeset
367
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
368
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
369
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
370 def set_medium_from_data(model: cobraModel, df: pd.DataFrame):
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
371 """Set the medium based on the 'InMedium' column in the dataframe."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
372 medium_reactions = df[df['InMedium'] == True]['ReactionID'].tolist()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
373
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
374 medium_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
375 for rxn_id in medium_reactions:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
376 if rxn_id in [r.id for r in model.reactions]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
377 reaction = model.reactions.get_by_id(rxn_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
378 if reaction.lower_bound < 0: # Solo reazioni di uptake
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
379 medium_dict[rxn_id] = abs(reaction.lower_bound)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
380
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
381 if medium_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
382 model.medium = medium_dict
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
383 print(f"Medium set with {len(medium_dict)} components")
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
384
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
385
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
386 def validate_model(model: cobraModel) -> Dict[str, any]:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
387 """Validate the model and return basic statistics."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
388 validation = {
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
389 'num_reactions': len(model.reactions),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
390 'num_metabolites': len(model.metabolites),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
391 'num_genes': len(model.genes),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
392 'num_compartments': len(model.compartments),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
393 'objective': str(model.objective),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
394 'medium_size': len(model.medium),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
395 'reversible_reactions': len([r for r in model.reactions if r.reversibility]),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
396 'exchange_reactions': len([r for r in model.reactions if r.id.startswith('EX_')]),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
397 }
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
398
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
399 try:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
400 # Growth test
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
401 solution = model.optimize()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
402 validation['growth_rate'] = solution.objective_value
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
403 validation['status'] = solution.status
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
404 except Exception as e:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
405 validation['growth_rate'] = None
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
406 validation['status'] = f"Error: {e}"
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
407
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
408 return validation
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
409
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
410 def convert_genes(model, annotation):
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
411 """Rename genes using a selected annotation key in gene.notes; returns a model copy."""
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
412 from cobra.manipulation import rename_genes
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
413 model2=model.copy()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
414 try:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
415 dict_genes={gene.id:gene.notes[annotation] for gene in model2.genes}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
416 except:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
417 print("No annotation in gene dict!")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
418 return -1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
419 rename_genes(model2,dict_genes)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
420
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
421 return model2
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
422
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
423 # ---------- Utility helpers ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
424 def _normalize_colname(col: str) -> str:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
425 return col.strip().lower().replace(' ', '_')
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
427 def _choose_columns(mapping_df: 'pd.DataFrame') -> Dict[str, str]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
428 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
429 Find useful columns and return a dict {ensg: colname1, hgnc_id: colname2, ...}.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
430 Raise ValueError if no suitable mapping is found.
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
431 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
432 cols = { _normalize_colname(c): c for c in mapping_df.columns }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
433 chosen = {}
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
434 # candidate names for each category
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
435 candidates = {
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
436 'ensg': ['ensg', 'ensembl_gene_id', 'ensembl'],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
437 'hgnc_id': ['hgnc_id', 'hgnc', 'hgnc:'],
444
06564187fba3 Uploaded
francesco_lapi
parents: 430
diff changeset
438 'hgnc_symbol': ['hgnc_symbol', 'hgnc symbol', 'symbol'],
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
439 'entrez_id': ['entrez', 'entrez_id', 'entrezgene'],
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
440 'gene_number': ['gene_number']
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
441 }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
442 for key, names in candidates.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
443 for n in names:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
444 if n in cols:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
445 chosen[key] = cols[n]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
446 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
447 return chosen
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
448
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
449 def _validate_target_uniqueness(mapping_df: 'pd.DataFrame',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
450 source_col: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
451 target_col: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
452 model_source_genes: Optional[Set[str]] = None,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
453 logger: Optional[logging.Logger] = None) -> None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
454 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
455 Check that, within the filtered mapping_df, each target maps to at most one source.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
456 Log examples if duplicates are found.
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
457 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
458 if logger is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
459 logger = logging.getLogger(__name__)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
460
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
461 if mapping_df.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
462 logger.warning("Mapping dataframe is empty for the requested source genes; skipping uniqueness validation.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
463 return
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
464
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
465 # normalize temporary columns for grouping (without altering the original df)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
466 tmp = mapping_df[[source_col, target_col]].copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
467 tmp['_src_norm'] = tmp[source_col].astype(str).map(_normalize_gene_id)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
468 tmp['_tgt_norm'] = tmp[target_col].astype(str).str.strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
469
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
470 # optionally filter to the set of model source genes
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
471 if model_source_genes is not None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
472 tmp = tmp[tmp['_src_norm'].isin(model_source_genes)]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
473
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
474 if tmp.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
475 logger.warning("After filtering to model source genes, mapping table is empty — nothing to validate.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
476 return
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
477
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
478 # build reverse mapping: target -> set(sources)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
479 grouped = tmp.groupby('_tgt_norm')['_src_norm'].agg(lambda s: set(s.dropna()))
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
480 # find targets with more than one source
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
481 problematic = {t: sorted(list(s)) for t, s in grouped.items() if len(s) > 1}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
482
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
483 if problematic:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
484 # prepare warning message with examples (limited subset)
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
485 sample_items = list(problematic.items())
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
486 msg_lines = ["Mapping validation failed: some target IDs are associated with multiple source IDs."]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
487 for tgt, sources in sample_items:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
488 msg_lines.append(f" - target '{tgt}' <- sources: {', '.join(sources)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
489 full_msg = "\n".join(msg_lines)
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
490 # log warning
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
491 logger.warning(full_msg)
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
492
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
493 # if everything is fine
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
494 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
495
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
496
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
497 def _normalize_gene_id(g: str) -> str:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
498 """Normalize a gene ID for use as a key (removes prefixes like 'HGNC:' and strips)."""
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
499 if g is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
500 return ""
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
501 g = str(g).strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
502 # remove common prefixes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
503 g = re.sub(r'^(HGNC:)', '', g, flags=re.IGNORECASE)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
504 g = re.sub(r'^(ENSG:)', '', g, flags=re.IGNORECASE)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
505 return g
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
506
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
507 def _simplify_boolean_expression(expr: str) -> str:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
508 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
509 Simplify a boolean expression by removing duplicates and redundancies.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
510 Handles expressions with 'and' and 'or'.
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
511 """
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
512 if not expr or not expr.strip():
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
513 return expr
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
514
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
515 # normalize operators
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
516 expr = expr.replace(' AND ', ' and ').replace(' OR ', ' or ')
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
517
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
518 # recursive helper to process expressions
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
519 def process_expression(s: str) -> str:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
520 s = s.strip()
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
521 if not s:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
522 return s
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
523
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
524 # handle parentheses
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
525 while '(' in s:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
526 # find the innermost parentheses
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
527 start = -1
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
528 for i, c in enumerate(s):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
529 if c == '(':
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
530 start = i
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
531 elif c == ')' and start != -1:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
532 # process inner content
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
533 inner = s[start+1:i]
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
534 processed_inner = process_expression(inner)
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
535 s = s[:start] + processed_inner + s[i+1:]
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
536 break
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
537 else:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
538 break
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
539
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
540 # split by 'or' at top level
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
541 or_parts = []
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
542 current_part = ""
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
543 paren_count = 0
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
544
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
545 tokens = s.split()
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
546 i = 0
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
547 while i < len(tokens):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
548 token = tokens[i]
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
549 if token == 'or' and paren_count == 0:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
550 if current_part.strip():
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
551 or_parts.append(current_part.strip())
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
552 current_part = ""
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
553 else:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
554 if token.count('(') > token.count(')'):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
555 paren_count += token.count('(') - token.count(')')
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
556 elif token.count(')') > token.count('('):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
557 paren_count -= token.count(')') - token.count('(')
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
558 current_part += token + " "
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
559 i += 1
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
560
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
561 if current_part.strip():
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
562 or_parts.append(current_part.strip())
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
563
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
564 # process each OR part
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
565 processed_or_parts = []
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
566 for or_part in or_parts:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
567 # split by 'and' within each OR part
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
568 and_parts = []
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
569 current_and = ""
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
570 paren_count = 0
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
571
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
572 and_tokens = or_part.split()
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
573 j = 0
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
574 while j < len(and_tokens):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
575 token = and_tokens[j]
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
576 if token == 'and' and paren_count == 0:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
577 if current_and.strip():
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
578 and_parts.append(current_and.strip())
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
579 current_and = ""
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
580 else:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
581 if token.count('(') > token.count(')'):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
582 paren_count += token.count('(') - token.count(')')
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
583 elif token.count(')') > token.count('('):
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
584 paren_count -= token.count(')') - token.count('(')
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
585 current_and += token + " "
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
586 j += 1
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
587
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
588 if current_and.strip():
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
589 and_parts.append(current_and.strip())
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
590
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
591 # deduplicate AND parts
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
592 unique_and_parts = list(dict.fromkeys(and_parts)) # mantiene l'ordine
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
593
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
594 if len(unique_and_parts) == 1:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
595 processed_or_parts.append(unique_and_parts[0])
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
596 elif len(unique_and_parts) > 1:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
597 processed_or_parts.append(" and ".join(unique_and_parts))
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
598
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
599 # deduplicate OR parts
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
600 unique_or_parts = list(dict.fromkeys(processed_or_parts))
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
601
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
602 if len(unique_or_parts) == 1:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
603 return unique_or_parts[0]
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
604 elif len(unique_or_parts) > 1:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
605 return " or ".join(unique_or_parts)
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
606 else:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
607 return ""
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
608
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
609 try:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
610 return process_expression(expr)
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
611 except Exception:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
612 # if simplification fails, return the original expression
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
613 return expr
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
614
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
615 # ---------- Main public function ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
616 def translate_model_genes(model: 'cobra.Model',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
617 mapping_df: 'pd.DataFrame',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
618 target_nomenclature: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
619 source_nomenclature: str = 'hgnc_id',
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
620 allow_many_to_one: bool = False,
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
621 logger: Optional[logging.Logger] = None) -> 'cobra.Model':
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
622 """
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
623 Translate model genes from source_nomenclature to target_nomenclature using a mapping table.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
624 mapping_df should contain columns enabling mapping (e.g., ensg, hgnc_id, hgnc_symbol, entrez).
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
625
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
626 Args:
456
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
627 model: COBRA model to translate.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
628 mapping_df: DataFrame containing the mapping information.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
629 target_nomenclature: Desired target key (e.g., 'hgnc_symbol').
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
630 source_nomenclature: Current source key in the model (default 'hgnc_id').
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
631 allow_many_to_one: If True, allow many-to-one mappings and handle duplicates in GPRs.
a6e45049c1b9 Uploaded
francesco_lapi
parents: 455
diff changeset
632 logger: Optional logger.
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
633 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
634 if logger is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
635 logging.basicConfig(level=logging.INFO, format='%(asctime)s - %(levelname)s - %(message)s')
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
636 logger = logging.getLogger(__name__)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
637
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
638 logger.info(f"Translating genes from '{source_nomenclature}' to '{target_nomenclature}'")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
639
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
640 # normalize column names and choose relevant columns
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
641 chosen = _choose_columns(mapping_df)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
642 if not chosen:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
643 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
644
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
645 # map source/target to actual dataframe column names (allow user-specified source/target keys)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
646 # normalize input args
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
647 src_key = source_nomenclature.strip().lower()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
648 tgt_key = target_nomenclature.strip().lower()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
649
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
650 # try to find the actual column names for requested keys
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
651 col_for_src = None
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
652 col_for_tgt = None
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
653 # first, try exact match
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
654 for k, actual in chosen.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
655 if k == src_key:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
656 col_for_src = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
657 if k == tgt_key:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
658 col_for_tgt = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
659
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
660 # if not found, try mapping common names
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
661 if col_for_src is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
662 possible_src_names = {k: v for k, v in chosen.items()}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
663 # try to match by contained substring
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
664 for k, actual in possible_src_names.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
665 if src_key in k:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
666 col_for_src = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
667 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
668
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
669 if col_for_tgt is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
670 for k, actual in chosen.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
671 if tgt_key in k:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
672 col_for_tgt = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
673 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
674
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
675 if col_for_src is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
676 raise ValueError(f"Source column for '{source_nomenclature}' not found in mapping dataframe.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
677 if col_for_tgt is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
678 raise ValueError(f"Target column for '{target_nomenclature}' not found in mapping dataframe.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
679
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
680 model_source_genes = { _normalize_gene_id(g.id) for g in model.genes }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
681 logger.info(f"Filtering mapping to {len(model_source_genes)} source genes present in model (normalized).")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
682
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
683 tmp_map = mapping_df[[col_for_src, col_for_tgt]].dropna().copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
684 tmp_map[col_for_src + "_norm"] = tmp_map[col_for_src].astype(str).map(_normalize_gene_id)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
685
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
686 filtered_map = tmp_map[tmp_map[col_for_src + "_norm"].isin(model_source_genes)].copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
687
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
688 if filtered_map.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
689 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
690
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
691 if not allow_many_to_one:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
692 _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
693
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
694 # Crea il mapping
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
695 gene_mapping = _create_gene_mapping(filtered_map, col_for_src, col_for_tgt, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
696
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
697 # copy model
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
698 model_copy = model.copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
699
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
700 # statistics
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
701 stats = {'translated': 0, 'one_to_one': 0, 'one_to_many': 0, 'not_found': 0, 'simplified_gprs': 0}
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
702 unmapped = []
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
703 multi = []
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
704
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
705 original_genes = {g.id for g in model_copy.genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
706 logger.info(f"Original genes count: {len(original_genes)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
707
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
708 # translate GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
709 for rxn in model_copy.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
710 gpr = rxn.gene_reaction_rule
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
711 if gpr and gpr.strip():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
712 new_gpr = _translate_gpr(gpr, gene_mapping, stats, unmapped, multi, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
713 if new_gpr != gpr:
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
714 simplified_gpr = _simplify_boolean_expression(new_gpr)
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
715 if simplified_gpr != new_gpr:
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
716 stats['simplified_gprs'] += 1
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
717 logger.debug(f"Simplified GPR for {rxn.id}: '{new_gpr}' -> '{simplified_gpr}'")
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
718 rxn.gene_reaction_rule = simplified_gpr
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
719 logger.debug(f"Reaction {rxn.id}: '{gpr}' -> '{simplified_gpr}'")
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
720
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
721 # update model genes based on new GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
722 _update_model_genes(model_copy, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
723
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
724 # final logging
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
725 _log_translation_statistics(stats, unmapped, multi, original_genes, model_copy.genes, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
726
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
727 logger.info("Translation finished")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
728 return model_copy
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
729
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
730
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
731 # ---------- helper functions ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
732 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
733 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
734 Build mapping dict: source_id -> list of target_ids
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
735 Normalizes IDs (removes prefixes like 'HGNC:' etc).
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
736 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
737 df = mapping_df[[source_col, target_col]].dropna().copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
738 # normalize to string
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
739 df[source_col] = df[source_col].astype(str).map(_normalize_gene_id)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
740 df[target_col] = df[target_col].astype(str).str.strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
741
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
742 df = df.drop_duplicates()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
743
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
744 logger.info(f"Creating mapping from {len(df)} rows")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
745
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
746 mapping = defaultdict(list)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
747 for _, row in df.iterrows():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
748 s = row[source_col]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
749 t = row[target_col]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
750 if t not in mapping[s]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
751 mapping[s].append(t)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
752
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
753 # stats
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
754 one_to_one = sum(1 for v in mapping.values() if len(v) == 1)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
755 one_to_many = sum(1 for v in mapping.values() if len(v) > 1)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
756 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
757 return dict(mapping)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
758
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
759
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
760 def _translate_gpr(gpr_string: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
761 gene_mapping: Dict[str, List[str]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
762 stats: Dict[str, int],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
763 unmapped_genes: List[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
764 multi_mapping_genes: List[Tuple[str, List[str]]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
765 logger: logging.Logger) -> str:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
766 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
767 Translate genes inside a GPR string using gene_mapping.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
768 Returns new GPR string.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
769 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
770 # Generic token pattern: letters, digits, :, _, -, ., (captures HGNC:1234, ENSG000..., symbols)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
771 token_pattern = r'\b[A-Za-z0-9:_.-]+\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
772 tokens = re.findall(token_pattern, gpr_string)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
773
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
774 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
775 tokens = [t for t in tokens if t not in logical]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
776
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
777 new_gpr = gpr_string
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
778
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
779 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
780 norm = _normalize_gene_id(token)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
781 if norm in gene_mapping:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
782 targets = gene_mapping[norm]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
783 stats['translated'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
784 if len(targets) == 1:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
785 stats['one_to_one'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
786 replacement = targets[0]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
787 else:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
788 stats['one_to_many'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
789 multi_mapping_genes.append((token, targets))
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
790 replacement = "(" + " or ".join(targets) + ")"
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
791
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
792 pattern = r'\b' + re.escape(token) + r'\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
793 new_gpr = re.sub(pattern, replacement, new_gpr)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
794 else:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
795 stats['not_found'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
796 if token not in unmapped_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
797 unmapped_genes.append(token)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
798 logger.debug(f"Token not found in mapping (left as-is): {token}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
799
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
800 return new_gpr
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
801
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
802
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
803 def _update_model_genes(model: 'cobra.Model', logger: logging.Logger):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
804 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
805 Rebuild model.genes from gene_reaction_rule content.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
806 Removes genes not referenced and adds missing ones.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
807 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
808 # collect genes in GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
809 gene_pattern = r'\b[A-Za-z0-9:_.-]+\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
810 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
811 genes_in_gpr: Set[str] = set()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
812
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
813 for rxn in model.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
814 gpr = rxn.gene_reaction_rule
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
815 if gpr and gpr.strip():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
816 toks = re.findall(gene_pattern, gpr)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
817 toks = [t for t in toks if t not in logical]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
818 # normalize IDs consistent with mapping normalization
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
819 toks = [_normalize_gene_id(t) for t in toks]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
820 genes_in_gpr.update(toks)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
821
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
822 # existing gene ids
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
823 existing = {g.id for g in model.genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
824
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
825 # remove obsolete genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
826 to_remove = [gid for gid in existing if gid not in genes_in_gpr]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
827 removed = 0
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
828 for gid in to_remove:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
829 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
830 gene_obj = model.genes.get_by_id(gid)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
831 model.genes.remove(gene_obj)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
832 removed += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
833 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
834 # safe-ignore
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
835 pass
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
836
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
837 # add new genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
838 added = 0
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
839 for gid in genes_in_gpr:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
840 if gid not in existing:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
841 new_gene = cobra.Gene(gid)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
842 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
843 model.genes.add(new_gene)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
844 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
845 # fallback: if model.genes doesn't support add, try append or model.add_genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
846 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
847 model.genes.append(new_gene)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
848 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
849 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
850 model.add_genes([new_gene])
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
851 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
852 logger.warning(f"Could not add gene object for {gid}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
853 added += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
854
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
855 logger.info(f"Model genes updated: removed {removed}, added {added}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
856
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
857
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
858 def _log_translation_statistics(stats: Dict[str, int],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
859 unmapped_genes: List[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
860 multi_mapping_genes: List[Tuple[str, List[str]]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
861 original_genes: Set[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
862 final_genes,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
863 logger: logging.Logger):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
864 logger.info("=== TRANSLATION STATISTICS ===")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
865 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
866 logger.info(f"Not found tokens: {stats.get('not_found', 0)}")
455
4e2bc80764b6 Uploaded
francesco_lapi
parents: 448
diff changeset
867 logger.info(f"Simplified GPRs: {stats.get('simplified_gprs', 0)}")
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
868
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
869 final_ids = {g.id for g in final_genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
870 logger.info(f"Genes in model: {len(original_genes)} -> {len(final_ids)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
871
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
872 if unmapped_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
873 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
874 if multi_mapping_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
875 logger.info(f"Multi-mapping examples ({len(multi_mapping_genes)}):")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
876 for orig, targets in multi_mapping_genes[:10]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
877 logger.info(f" {orig} -> {', '.join(targets)}")