annotate COBRAxy/utils/model_utils.py @ 426:00a78da611ba draft

Uploaded
author francesco_lapi
date Wed, 10 Sep 2025 09:25:32 +0000
parents ed2c1f9e20ba
children 4a385fdb9e58
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
1 import os
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
2 import csv
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
3 import cobra
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
4 import pickle
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
5 import argparse
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
6 import pandas as pd
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
7 import re
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
8 import logging
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
9 from typing import Optional, Tuple, Union, List, Dict, Set
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
10 from collections import defaultdict
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
11 import utils.general_utils as utils
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
12 import utils.rule_parsing as rulesUtils
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
13 import utils.reaction_parsing as reactionUtils
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
14 from cobra import Model as cobraModel, Reaction, Metabolite
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
15
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
16 ################################- DATA GENERATION -################################
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
17 ReactionId = str
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
18 def generate_rules(model: cobraModel, *, asParsed = True) -> Union[Dict[ReactionId, rulesUtils.OpList], Dict[ReactionId, str]]:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
19 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
20 Generates a dictionary mapping reaction ids to rules from the model.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
21
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
22 Args:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
23 model : the model to derive data from.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
24 asParsed : if True parses the rules to an optimized runtime format, otherwise leaves them as strings.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
25
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
26 Returns:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
27 Dict[ReactionId, rulesUtils.OpList] : the generated dictionary of parsed rules.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
28 Dict[ReactionId, str] : the generated dictionary of raw rules.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
29 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
30 # Is the below approach convoluted? yes
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
31 # Ok but is it inefficient? probably
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
32 # Ok but at least I don't have to repeat the check at every rule (I'm clinically insane)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
33 _ruleGetter = lambda reaction : reaction.gene_reaction_rule
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
34 ruleExtractor = (lambda reaction :
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
35 rulesUtils.parseRuleToNestedList(_ruleGetter(reaction))) if asParsed else _ruleGetter
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
36
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
37 return {
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
38 reaction.id : ruleExtractor(reaction)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
39 for reaction in model.reactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
40 if reaction.gene_reaction_rule }
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
41
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
42 def generate_reactions(model :cobraModel, *, asParsed = True) -> Dict[ReactionId, str]:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
43 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
44 Generates a dictionary mapping reaction ids to reaction formulas from the model.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
45
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
46 Args:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
47 model : the model to derive data from.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
48 asParsed : if True parses the reactions to an optimized runtime format, otherwise leaves them as they are.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
49
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
50 Returns:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
51 Dict[ReactionId, str] : the generated dictionary.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
52 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
53
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
54 unparsedReactions = {
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
55 reaction.id : reaction.reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
56 for reaction in model.reactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
57 if reaction.reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
58 }
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
59
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
60 if not asParsed: return unparsedReactions
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
61
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
62 return reactionUtils.create_reaction_dict(unparsedReactions)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
63
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
64 def get_medium(model:cobraModel) -> pd.DataFrame:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
65 trueMedium=[]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
66 for r in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
67 positiveCoeff=0
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
68 for m in r.metabolites:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
69 if r.get_coefficient(m.id)>0:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
70 positiveCoeff=1;
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
71 if (positiveCoeff==0 and r.lower_bound<0):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
72 trueMedium.append(r.id)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
73
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
74 df_medium = pd.DataFrame()
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
75 df_medium["reaction"] = trueMedium
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
76 return df_medium
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
77
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
78 def extract_objective_coefficients(model: cobraModel) -> pd.DataFrame:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
79 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
80 Estrae i coefficienti della funzione obiettivo per ciascuna reazione del modello.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
81
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
82 Args:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
83 model : cobra.Model
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
84
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
85 Returns:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
86 pd.DataFrame con colonne: ReactionID, ObjectiveCoefficient
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
87 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
88 coeffs = []
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
89 # model.objective.expression รจ un'espressione lineare
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
90 objective_expr = model.objective.expression.as_coefficients_dict()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
91
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
92 for reaction in model.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
93 coeff = objective_expr.get(reaction.forward_variable, 0.0)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
94 coeffs.append({
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
95 "ReactionID": reaction.id,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
96 "ObjectiveCoefficient": coeff
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
97 })
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
98
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
99 return pd.DataFrame(coeffs)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
100
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
101 def generate_bounds(model:cobraModel) -> pd.DataFrame:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
102
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
103 rxns = []
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
104 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
105 rxns.append(reaction.id)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
106
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
107 bounds = pd.DataFrame(columns = ["lower_bound", "upper_bound"], index=rxns)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
108
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
109 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
110 bounds.loc[reaction.id] = [reaction.lower_bound, reaction.upper_bound]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
111 return bounds
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
112
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
113
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
114
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
115 def generate_compartments(model: cobraModel) -> pd.DataFrame:
418
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
116 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
117 Generates a DataFrame containing compartment information for each reaction.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
118 Creates columns for each compartment position (Compartment_1, Compartment_2, etc.)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
119
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
120 Args:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
121 model: the COBRA model to extract compartment data from.
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
122
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
123 Returns:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
124 pd.DataFrame: DataFrame with ReactionID and compartment columns
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
125 """
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
126 pathway_data = []
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
127
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
128 # First pass: determine the maximum number of pathways any reaction has
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
129 max_pathways = 0
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
130 reaction_pathways = {}
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
131
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
132 for reaction in model.reactions:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
133 # Get unique pathways from all metabolites in the reaction
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
134 if type(reaction.annotation['pathways']) == list:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
135 reaction_pathways[reaction.id] = reaction.annotation['pathways']
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
136 max_pathways = max(max_pathways, len(reaction.annotation['pathways']))
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
137 else:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
138 reaction_pathways[reaction.id] = [reaction.annotation['pathways']]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
139
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
140 # Create column names for pathways
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
141 pathway_columns = [f"Pathway_{i+1}" for i in range(max_pathways)]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
142
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
143 # Second pass: create the data
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
144 for reaction_id, pathways in reaction_pathways.items():
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
145 row = {"ReactionID": reaction_id}
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
146
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
147 # Fill pathway columns
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
148 for i in range(max_pathways):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
149 col_name = pathway_columns[i]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
150 if i < len(pathways):
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
151 row[col_name] = pathways[i]
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
152 else:
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
153 row[col_name] = None # or "" if you prefer empty strings
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
154
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
155 pathway_data.append(row)
919b5b71a61c Uploaded
francesco_lapi
parents:
diff changeset
156
419
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
157 return pd.DataFrame(pathway_data)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
158
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
159
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
160
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
161 def build_cobra_model_from_csv(csv_path: str, model_id: str = "new_model") -> cobraModel:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
162 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
163 Costruisce un modello COBRApy a partire da un file CSV con i dati delle reazioni.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
164
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
165 Args:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
166 csv_path: Path al file CSV (separato da tab)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
167 model_id: ID del modello da creare
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
168
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
169 Returns:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
170 cobra.Model: Il modello COBRApy costruito
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
171 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
172
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
173 # Leggi i dati dal CSV
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
174 df = pd.read_csv(csv_path, sep='\t')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
175
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
176 # Crea il modello vuoto
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
177 model = cobraModel(model_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
178
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
179 # Dict per tenere traccia di metaboliti e compartimenti
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
180 metabolites_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
181 compartments_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
182
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
183 print(f"Costruendo modello da {len(df)} reazioni...")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
184
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
185 # Prima passata: estrai metaboliti e compartimenti dalle formule delle reazioni
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
186 for idx, row in df.iterrows():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
187 reaction_formula = str(row['Reaction']).strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
188 if not reaction_formula or reaction_formula == 'nan':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
189 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
190
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
191 # Estrai metaboliti dalla formula della reazione
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
192 metabolites = extract_metabolites_from_reaction(reaction_formula)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
193
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
194 for met_id in metabolites:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
195 compartment = extract_compartment_from_metabolite(met_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
196
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
197 # Aggiungi compartimento se non esiste
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
198 if compartment not in compartments_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
199 compartments_dict[compartment] = compartment
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
200
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
201 # Aggiungi metabolita se non esiste
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
202 if met_id not in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
203 metabolites_dict[met_id] = Metabolite(
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
204 id=met_id,
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
205 compartment=compartment,
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
206 name=met_id.replace(f"_{compartment}", "").replace("__", "_")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
207 )
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
208
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
209 # Aggiungi compartimenti al modello
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
210 model.compartments = compartments_dict
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
211
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
212 # Aggiungi metaboliti al modello
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
213 model.add_metabolites(list(metabolites_dict.values()))
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
214
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
215 print(f"Aggiunti {len(metabolites_dict)} metaboliti e {len(compartments_dict)} compartimenti")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
216
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
217 # Seconda passata: aggiungi le reazioni
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
218 reactions_added = 0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
219 reactions_skipped = 0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
220
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
221 for idx, row in df.iterrows():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
222
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
223 reaction_id = str(row['ReactionID']).strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
224 reaction_formula = str(row['Reaction']).strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
225
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
226 # Salta reazioni senza formula
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
227 if not reaction_formula or reaction_formula == 'nan':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
228 raise ValueError(f"Formula della reazione mancante {reaction_id}")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
229
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
230 # Crea la reazione
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
231 reaction = Reaction(reaction_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
232 reaction.name = reaction_id
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
233
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
234 # Imposta bounds
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
235 reaction.lower_bound = float(row['lower_bound']) if pd.notna(row['lower_bound']) else -1000.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
236 reaction.upper_bound = float(row['upper_bound']) if pd.notna(row['upper_bound']) else 1000.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
237
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
238 # Aggiungi gene rule se presente
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
239 if pd.notna(row['Rule']) and str(row['Rule']).strip():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
240 reaction.gene_reaction_rule = str(row['Rule']).strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
241
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
242 # Parse della formula della reazione
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:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
246 print(f"Errore nel parsing della reazione {reaction_id}: {e}")
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 # Aggiungi la reazione al modello
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
251 model.add_reactions([reaction])
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
252 reactions_added += 1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
253
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
254
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
255 print(f"Aggiunte {reactions_added} reazioni, saltate {reactions_skipped} reazioni")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
256
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
257 # Imposta l'obiettivo di biomassa
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
258 set_biomass_objective(model)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
259
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
260 # Imposta il medium
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
261 set_medium_from_data(model, df)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
262
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
263 print(f"Modello completato: {len(model.reactions)} reazioni, {len(model.metabolites)} metaboliti")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
264
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
265 return model
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
266
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
267
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
268 # Estrae tutti gli ID metaboliti nella formula (gestisce prefissi numerici + underscore)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
269 def extract_metabolites_from_reaction(reaction_formula: str) -> Set[str]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
270 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
271 Estrae gli ID dei metaboliti da una formula di reazione.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
272 Pattern robusto: cattura token che terminano con _<compartimento> (es. _c, _m, _e)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
273 e permette che comincino con cifre o underscore.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
274 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
275 metabolites = set()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
276 # coefficiente opzionale seguito da un token che termina con _<letters>
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
277 pattern = r'(?:\d+(?:\.\d+)?\s+)?([A-Za-z0-9_]+_[a-z]+)'
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
278 matches = re.findall(pattern, reaction_formula)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
279 metabolites.update(matches)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
280 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
281
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
282
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
283 def extract_compartment_from_metabolite(metabolite_id: str) -> str:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
284 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
285 Estrae il compartimento dall'ID del metabolita.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
286 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
287 # Il compartimento รจ solitamente l'ultima lettera dopo l'underscore
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
288 if '_' in metabolite_id:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
289 return metabolite_id.split('_')[-1]
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
290 return 'c' # default cytoplasm
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
291
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
292
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
293 def parse_reaction_formula(reaction: Reaction, formula: str, metabolites_dict: Dict[str, Metabolite]):
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
294 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
295 Parsa una formula di reazione e imposta i metaboliti con i loro coefficienti.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
296 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
297
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
298 if reaction.id == 'EX_thbpt_e':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
299 print(reaction.id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
300 print(formula)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
301 # Dividi in parte sinistra e destra
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
302 if '<=>' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
303 left, right = formula.split('<=>')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
304 reversible = True
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
305 elif '<--' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
306 left, right = formula.split('<--')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
307 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
308 left, right = left, right
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
309 elif '-->' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
310 left, right = formula.split('-->')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
311 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
312 elif '<-' in formula:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
313 left, right = formula.split('<-')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
314 reversible = False
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
315 left, right = left, right
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
316 else:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
317 raise ValueError(f"Formato reazione non riconosciuto: {formula}")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
318
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
319 # Parse dei metaboliti e coefficienti
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
320 reactants = parse_metabolites_side(left.strip())
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
321 products = parse_metabolites_side(right.strip())
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
322
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
323 # Aggiungi metaboliti alla reazione
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
324 metabolites_to_add = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
325
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
326 # Reagenti (coefficienti negativi)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
327 for met_id, coeff in reactants.items():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
328 if met_id in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
329 metabolites_to_add[metabolites_dict[met_id]] = -coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
330
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
331 # Prodotti (coefficienti positivi)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
332 for met_id, coeff in products.items():
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
333 if met_id in metabolites_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
334 metabolites_to_add[metabolites_dict[met_id]] = coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
335
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
336 reaction.add_metabolites(metabolites_to_add)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
337
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
338
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
339 def parse_metabolites_side(side_str: str) -> Dict[str, float]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
340 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
341 Parsa un lato della reazione per estrarre metaboliti e coefficienti.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
342 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
343 metabolites = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
344 if not side_str or side_str.strip() == '':
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
345 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
346
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
347 terms = side_str.split('+')
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
348 for term in terms:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
349 term = term.strip()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
350 if not term:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
351 continue
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
352
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
353 # pattern allineato: coefficiente opzionale + id che termina con _<compartimento>
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
354 match = re.match(r'(?:(\d+\.?\d*)\s+)?([A-Za-z0-9_]+_[a-z]+)', term)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
355 if match:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
356 coeff_str, met_id = match.groups()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
357 coeff = float(coeff_str) if coeff_str else 1.0
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
358 metabolites[met_id] = coeff
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
359
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
360 return metabolites
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
361
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
362
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
363
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
364 def set_biomass_objective(model: cobraModel):
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
365 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
366 Imposta la reazione di biomassa come obiettivo.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
367 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
368 biomass_reactions = [r for r in model.reactions if 'biomass' in r.id.lower()]
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
369
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
370 if biomass_reactions:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
371 model.objective = biomass_reactions[0].id
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
372 print(f"Obiettivo impostato su: {biomass_reactions[0].id}")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
373 else:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
374 print("Nessuna reazione di biomassa trovata")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
375
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
376
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
377 def set_medium_from_data(model: cobraModel, df: pd.DataFrame):
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
378 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
379 Imposta il medium basato sulla colonna InMedium.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
380 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
381 medium_reactions = df[df['InMedium'] == True]['ReactionID'].tolist()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
382
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
383 medium_dict = {}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
384 for rxn_id in medium_reactions:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
385 if rxn_id in [r.id for r in model.reactions]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
386 reaction = model.reactions.get_by_id(rxn_id)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
387 if reaction.lower_bound < 0: # Solo reazioni di uptake
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
388 medium_dict[rxn_id] = abs(reaction.lower_bound)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
389
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
390 if medium_dict:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
391 model.medium = medium_dict
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
392 print(f"Medium impostato con {len(medium_dict)} componenti")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
393
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
394
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
395 def validate_model(model: cobraModel) -> Dict[str, any]:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
396 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
397 Valida il modello e fornisce statistiche di base.
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
398 """
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
399 validation = {
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
400 'num_reactions': len(model.reactions),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
401 'num_metabolites': len(model.metabolites),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
402 'num_genes': len(model.genes),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
403 'num_compartments': len(model.compartments),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
404 'objective': str(model.objective),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
405 'medium_size': len(model.medium),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
406 'reversible_reactions': len([r for r in model.reactions if r.reversibility]),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
407 'exchange_reactions': len([r for r in model.reactions if r.id.startswith('EX_')]),
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
408 }
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
409
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
410 try:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
411 # Test di crescita
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
412 solution = model.optimize()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
413 validation['growth_rate'] = solution.objective_value
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
414 validation['status'] = solution.status
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
415 except Exception as e:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
416 validation['growth_rate'] = None
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
417 validation['status'] = f"Error: {e}"
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
418
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
419 return validation
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
420
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
421 def convert_genes(model,annotation):
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
422 from cobra.manipulation import rename_genes
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
423 model2=model.copy()
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
424 try:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
425 dict_genes={gene.id:gene.notes[annotation] for gene in model2.genes}
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
426 except:
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
427 print("No annotation in gene dict!")
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
428 return -1
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
429 rename_genes(model2,dict_genes)
ed2c1f9e20ba Uploaded
francesco_lapi
parents: 418
diff changeset
430
426
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
431 return model2
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
432
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
433
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
434
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
435 # ---------- Utility helpers ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
436 def _normalize_colname(col: str) -> str:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
437 return col.strip().lower().replace(' ', '_')
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
438
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
439 def _choose_columns(mapping_df: 'pd.DataFrame') -> Dict[str, str]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
440 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
441 Cerca colonne utili e ritorna dict {ensg: colname1, hgnc_id: colname2, ...}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
442 Lancia ValueError se non trova almeno un mapping utile.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
443 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
444 cols = { _normalize_colname(c): c for c in mapping_df.columns }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
445 chosen = {}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
446 # possibili nomi per ciascuna categoria
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
447 candidates = {
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
448 'ensg': ['ensg', 'ensembl_gene_id', 'ensembl'],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
449 'hgnc_id': ['hgnc_id', 'hgnc', 'hgnc:'],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
450 'hgnc_symbol': ['hgnc_symbol', 'hgnc_symbol', 'symbol'],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
451 'entrez_id': ['entrez', 'entrez_id', 'entrezgene']
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
452 }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
453 for key, names in candidates.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
454 for n in names:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
455 if n in cols:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
456 chosen[key] = cols[n]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
457 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
458 return chosen
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
459
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
460 def _validate_target_uniqueness(mapping_df: 'pd.DataFrame',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
461 source_col: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
462 target_col: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
463 model_source_genes: Optional[Set[str]] = None,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
464 logger: Optional[logging.Logger] = None) -> None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
465 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
466 Verifica che, nel mapping_df (eventualmente giร  filtrato sui source di interesse),
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
467 ogni target sia associato ad al massimo un source. Se trova target associati a
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
468 >1 source solleva ValueError mostrando esempi.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
469
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
470 - mapping_df: DataFrame con colonne source_col, target_col
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
471 - model_source_genes: se fornito, รจ un set di source normalizzati che stiamo traducendo
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
472 (se None, si usa tutto mapping_df)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
473 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
474 if logger is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
475 logger = logging.getLogger(__name__)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
476
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
477 if mapping_df.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
478 logger.warning("Mapping dataframe is empty for the requested source genes; skipping uniqueness validation.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
479 return
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
480
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
481 # normalizza le colonne temporanee per gruppi (senza modificare il df originale)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
482 tmp = mapping_df[[source_col, target_col]].copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
483 tmp['_src_norm'] = tmp[source_col].astype(str).map(_normalize_gene_id)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
484 tmp['_tgt_norm'] = tmp[target_col].astype(str).str.strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
485
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
486 # se รจ passato un insieme di geni modello, filtra qui (giร  fatto nella chiamata, ma doppio-check ok)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
487 if model_source_genes is not None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
488 tmp = tmp[tmp['_src_norm'].isin(model_source_genes)]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
489
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
490 if tmp.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
491 logger.warning("After filtering to model source genes, mapping table is empty โ€” nothing to validate.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
492 return
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
493
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
494 # costruisci il reverse mapping target -> set(sources)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
495 grouped = tmp.groupby('_tgt_norm')['_src_norm'].agg(lambda s: set(s.dropna()))
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
496 # trova target con piรน di 1 source
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
497 problematic = {t: sorted(list(s)) for t, s in grouped.items() if len(s) > 1}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
498
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
499 if problematic:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
500 # prepara messaggio di errore con esempi (fino a 20)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
501 sample_items = list(problematic.items())[:20]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
502 msg_lines = ["Mapping validation failed: some target IDs are associated with multiple source IDs."]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
503 for tgt, sources in sample_items:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
504 msg_lines.append(f" - target '{tgt}' <- sources: {', '.join(sources)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
505 if len(problematic) > len(sample_items):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
506 msg_lines.append(f" ... and {len(problematic) - len(sample_items)} more cases.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
507 full_msg = "\n".join(msg_lines)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
508 # loggare e sollevare errore
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
509 logger.error(full_msg)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
510 raise ValueError(full_msg)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
511
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
512 # se tutto ok
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
513 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
514
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
515
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
516 def _normalize_gene_id(g: str) -> str:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
517 """Rendi consistente un gene id per l'uso come chiave (rimuove prefissi come 'HGNC:' e strip)."""
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
518 if g is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
519 return ""
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
520 g = str(g).strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
521 # remove common prefixes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
522 g = re.sub(r'^(HGNC:)', '', g, flags=re.IGNORECASE)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
523 g = re.sub(r'^(ENSG:)', '', g, flags=re.IGNORECASE)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
524 return g
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
525
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
526 # ---------- Main public function ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
527 def translate_model_genes(model: 'cobra.Model',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
528 mapping_df: 'pd.DataFrame',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
529 target_nomenclature: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
530 source_nomenclature: str = 'hgnc_id',
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
531 logger: Optional[logging.Logger] = None) -> 'cobra.Model':
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
532 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
533 Translate model genes from source_nomenclature to target_nomenclature.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
534 mapping_df should contain at least columns that allow the mapping
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
535 (e.g. ensg, hgnc_id, hgnc_symbol, entrez).
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
536 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
537 if logger is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
538 logging.basicConfig(level=logging.INFO, format='%(asctime)s - %(levelname)s - %(message)s')
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
539 logger = logging.getLogger(__name__)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
540
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
541 logger.info(f"Translating genes from '{source_nomenclature}' to '{target_nomenclature}'")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
542
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
543 # normalize column names and choose relevant columns
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
544 chosen = _choose_columns(mapping_df)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
545 if not chosen:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
546 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
547
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
548 # map source/target to actual dataframe column names (allow user-specified source/target keys)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
549 # normalize input args
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
550 src_key = source_nomenclature.strip().lower()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
551 tgt_key = target_nomenclature.strip().lower()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
552
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
553 # try to find the actual column names for requested keys
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
554 # support synonyms: user may pass "ensg" or "ENSG" etc.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
555 col_for_src = None
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
556 col_for_tgt = None
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
557 # first, try exact match
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
558 for k, actual in chosen.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
559 if k == src_key:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
560 col_for_src = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
561 if k == tgt_key:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
562 col_for_tgt = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
563
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
564 # if not found, try mapping common names
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
565 if col_for_src is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
566 # fallback: if user passed 'hgnc_id' but chosen has only 'hgnc_symbol', it's not useful
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
567 # we require at least the source column to exist
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
568 possible_src_names = {k: v for k, v in chosen.items()}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
569 # try to match by contained substring
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
570 for k, actual in possible_src_names.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
571 if src_key in k:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
572 col_for_src = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
573 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
574
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
575 if col_for_tgt is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
576 for k, actual in chosen.items():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
577 if tgt_key in k:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
578 col_for_tgt = actual
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
579 break
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
580
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
581 if col_for_src is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
582 raise ValueError(f"Source column for '{source_nomenclature}' not found in mapping dataframe.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
583 if col_for_tgt is None:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
584 raise ValueError(f"Target column for '{target_nomenclature}' not found in mapping dataframe.")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
585
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
586
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
587 model_source_genes = { _normalize_gene_id(g.id) for g in model.genes }
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
588 logger.info(f"Filtering mapping to {len(model_source_genes)} source genes present in model (normalized).")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
589
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
590 tmp_map = mapping_df[[col_for_src, col_for_tgt]].dropna().copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
591 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
592
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
593 filtered_map = tmp_map[tmp_map[col_for_src + "_norm"].isin(model_source_genes)].copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
594
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
595 # Se non ci sono righe rilevanti, avvisa (possono non esserci mapping per i geni presenti)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
596 if filtered_map.empty:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
597 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
598
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
599 # --- VALIDAZIONE: nessun target deve essere mappato da piu' di un source (nell'insieme filtrato) ---
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
600 # Se vuoi la verifica su tutto il dataframe (non solo sui geni del modello), passa model_source_genes=None.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
601 _validate_target_uniqueness(filtered_map, col_for_src, col_for_tgt, model_source_genes=model_source_genes, logger=logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
602
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
603 # Ora crea il mapping solo sul sottoinsieme filtrato (piu' efficiente)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
604 # ATTENZIONE: _create_gene_mapping si aspetta i nomi originali delle colonne
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
605 # quindi passiamo filtered_map con le colonne rimappate (senza la col_for_src + "_norm")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
606 gene_mapping = _create_gene_mapping(filtered_map, col_for_src, col_for_tgt, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
607
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
608 # copy model
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
609 model_copy = model.copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
610
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
611 # statistics
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
612 stats = {'translated': 0, 'one_to_one': 0, 'one_to_many': 0, 'not_found': 0}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
613 unmapped = []
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
614 multi = []
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
615
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
616 original_genes = {g.id for g in model_copy.genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
617 logger.info(f"Original genes count: {len(original_genes)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
618
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
619 # translate GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
620 for rxn in model_copy.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
621 gpr = rxn.gene_reaction_rule
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
622 if gpr and gpr.strip():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
623 new_gpr = _translate_gpr(gpr, gene_mapping, stats, unmapped, multi, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
624 if new_gpr != gpr:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
625 rxn.gene_reaction_rule = new_gpr
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
626 logger.debug(f"Reaction {rxn.id}: '{gpr}' -> '{new_gpr}'")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
627
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
628 # update model genes based on new GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
629 _update_model_genes(model_copy, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
630
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
631 # final logging
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
632 _log_translation_statistics(stats, unmapped, multi, original_genes, model_copy.genes, logger)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
633
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
634 logger.info("Translation finished")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
635 return model_copy
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
636
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
637
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
638 # ---------- helper functions ----------
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
639 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
640 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
641 Build mapping dict: source_id -> list of target_ids
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
642 Normalizes IDs (removes prefixes like 'HGNC:' etc).
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
643 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
644 df = mapping_df[[source_col, target_col]].dropna().copy()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
645 # normalize to string
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
646 df[source_col] = df[source_col].astype(str).map(_normalize_gene_id)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
647 df[target_col] = df[target_col].astype(str).str.strip()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
648
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
649 df = df.drop_duplicates()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
650
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
651 logger.info(f"Creating mapping from {len(df)} rows")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
652
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
653 mapping = defaultdict(list)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
654 for _, row in df.iterrows():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
655 s = row[source_col]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
656 t = row[target_col]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
657 if t not in mapping[s]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
658 mapping[s].append(t)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
659
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
660 # stats
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
661 one_to_one = sum(1 for v in mapping.values() if len(v) == 1)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
662 one_to_many = sum(1 for v in mapping.values() if len(v) > 1)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
663 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
664 return dict(mapping)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
665
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
666
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
667 def _translate_gpr(gpr_string: str,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
668 gene_mapping: Dict[str, List[str]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
669 stats: Dict[str, int],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
670 unmapped_genes: List[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
671 multi_mapping_genes: List[Tuple[str, List[str]]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
672 logger: logging.Logger) -> str:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
673 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
674 Translate genes inside a GPR string using gene_mapping.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
675 Returns new GPR string.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
676 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
677 # Generic token pattern: letters, digits, :, _, -, ., (captures HGNC:1234, ENSG000..., symbols)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
678 token_pattern = r'\b[A-Za-z0-9:_.-]+\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
679 tokens = re.findall(token_pattern, gpr_string)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
680
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
681 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
682 tokens = [t for t in tokens if t not in logical]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
683
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
684 new_gpr = gpr_string
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
685
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
686 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
687 norm = _normalize_gene_id(token)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
688 if norm in gene_mapping:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
689 targets = gene_mapping[norm]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
690 stats['translated'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
691 if len(targets) == 1:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
692 stats['one_to_one'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
693 replacement = targets[0]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
694 else:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
695 stats['one_to_many'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
696 multi_mapping_genes.append((token, targets))
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
697 replacement = "(" + " or ".join(targets) + ")"
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
698
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
699 pattern = r'\b' + re.escape(token) + r'\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
700 new_gpr = re.sub(pattern, replacement, new_gpr)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
701 else:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
702 stats['not_found'] += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
703 if token not in unmapped_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
704 unmapped_genes.append(token)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
705 logger.debug(f"Token not found in mapping (left as-is): {token}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
706
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
707 return new_gpr
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
708
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
709
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
710 def _update_model_genes(model: 'cobra.Model', logger: logging.Logger):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
711 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
712 Rebuild model.genes from gene_reaction_rule content.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
713 Removes genes not referenced and adds missing ones.
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
714 """
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
715 # collect genes in GPRs
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
716 gene_pattern = r'\b[A-Za-z0-9:_.-]+\b'
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
717 logical = {'and', 'or', 'AND', 'OR', '(', ')'}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
718 genes_in_gpr: Set[str] = set()
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
719
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
720 for rxn in model.reactions:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
721 gpr = rxn.gene_reaction_rule
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
722 if gpr and gpr.strip():
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
723 toks = re.findall(gene_pattern, gpr)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
724 toks = [t for t in toks if t not in logical]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
725 # normalize IDs consistent with mapping normalization
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
726 toks = [_normalize_gene_id(t) for t in toks]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
727 genes_in_gpr.update(toks)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
728
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
729 # existing gene ids
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
730 existing = {g.id for g in model.genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
731
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
732 # remove obsolete genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
733 to_remove = [gid for gid in existing if gid not in genes_in_gpr]
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
734 removed = 0
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
735 for gid in to_remove:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
736 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
737 gene_obj = model.genes.get_by_id(gid)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
738 model.genes.remove(gene_obj)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
739 removed += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
740 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
741 # safe-ignore
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
742 pass
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
743
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
744 # add new genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
745 added = 0
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
746 for gid in genes_in_gpr:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
747 if gid not in existing:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
748 new_gene = cobra.Gene(gid)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
749 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
750 model.genes.add(new_gene)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
751 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
752 # fallback: if model.genes doesn't support add, try append or model.add_genes
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
753 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
754 model.genes.append(new_gene)
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
755 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
756 try:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
757 model.add_genes([new_gene])
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
758 except Exception:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
759 logger.warning(f"Could not add gene object for {gid}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
760 added += 1
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
761
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
762 logger.info(f"Model genes updated: removed {removed}, added {added}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
763
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
764
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
765 def _log_translation_statistics(stats: Dict[str, int],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
766 unmapped_genes: List[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
767 multi_mapping_genes: List[Tuple[str, List[str]]],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
768 original_genes: Set[str],
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
769 final_genes,
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
770 logger: logging.Logger):
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
771 logger.info("=== TRANSLATION STATISTICS ===")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
772 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
773 logger.info(f"Not found tokens: {stats.get('not_found', 0)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
774
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
775 final_ids = {g.id for g in final_genes}
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
776 logger.info(f"Genes in model: {len(original_genes)} -> {len(final_ids)}")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
777
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
778 if unmapped_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
779 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
780 if multi_mapping_genes:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
781 logger.info(f"Multi-mapping examples ({len(multi_mapping_genes)}):")
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
782 for orig, targets in multi_mapping_genes[:10]:
00a78da611ba Uploaded
francesco_lapi
parents: 419
diff changeset
783 logger.info(f" {orig} -> {', '.join(targets)}")