gsrap 0.7.0__tar.gz → 0.7.1__tar.gz

{gsrap-0.7.0 → gsrap-0.7.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: gsrap
-Version: 0.7.0
+Version: 0.7.1
 Summary: 
 License: GNU General Public License v3.0
 Author: Gioele Lazzari

{gsrap-0.7.0 → gsrap-0.7.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "gsrap"
-version = "0.7.0"
+version = "0.7.1"
 description = ""
 authors = ["Gioele Lazzari"]
 license = "GNU General Public License v3.0"

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/.ipynb_checkpoints/__init__-checkpoint.py

@@ -1,8 +1,10 @@
 import argparse
 import sys
 import traceback
+import requests
 import importlib.metadata
 from datetime import datetime
+from packaging import version
 
 
 import cobra
@@ -29,8 +31,9 @@ solver_name = solver_name.replace("_interface", '')
 def main():
     
     
-    # define the header of main- and sub-commands. 
-    header = f'gsrap v{importlib.metadata.metadata("gsrap")["Version"]},\ndeveloped by Gioele Lazzari (gioele.lazzari@univr.it).'
+    # define the header of main- and sub-commands.
+    current_version = importlib.metadata.metadata("gsrap")["Version"]
+    header = f'gsrap v{current_version},\ndeveloped by Gioele Lazzari (gioele.lazzari@univr.it).'
     
     
     # create the command line arguments:
@@ -117,12 +120,36 @@ def main():
     # set up the logger:
     logger = get_logger('gsrap', args.verbose)
     
-    
+        
     
     # show a welcome message:
     set_header_trailer_formatter(logger.handlers[0])
     logger.info(header + '\n')
-    command_line = '' # print the full command line:
+    
+    
+    
+    # check if newer version is available
+    try:
+        response = requests.get(f"https://pypi.org/pypi/gsrap/json", timeout=3)  # sends an HTTP GET request to the given URL
+        response.raise_for_status()  # check the HTTP status code (e.g. 200, 404, 500): if not in the 2xx success range, raise requests.exceptions.HTTPError
+        data = response.json()
+        newest_version = data["info"]["version"] 
+    except Exception as error:  # eg requests.exceptions.Timeout, requests.exceptions.HTTPError
+        logger.info(f'Can\'t retrieve the number of the newest version. Please contact the developer reporting the following error: "{error}".')
+        logger.info('')  # still no formatting here
+        # do not exit, continue with the program
+    if version.parse(current_version) < version.parse(newest_version):
+        warning_message = f"███ Last version is v{newest_version} and you have v{current_version}: please update gsrap! ███"
+        border = ''.join(['█' for i in range(len(warning_message))])
+        logger.info(border)
+        logger.info(warning_message)
+        logger.info(border)
+        logger.info('')  # still no formatting here
+    
+    
+    
+    # print the full command line:
+    command_line = '' 
     for arg, value in vars(args).items():
         if arg == 'subcommand': command_line = command_line + f"gsrap {value} "
         else: command_line = command_line + f"--{arg} {value} "

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/__init__.py

@@ -1,8 +1,10 @@
 import argparse
 import sys
 import traceback
+import requests
 import importlib.metadata
 from datetime import datetime
+from packaging import version
 
 
 import cobra
@@ -29,8 +31,9 @@ solver_name = solver_name.replace("_interface", '')
 def main():
     
     
-    # define the header of main- and sub-commands. 
-    header = f'gsrap v{importlib.metadata.metadata("gsrap")["Version"]},\ndeveloped by Gioele Lazzari (gioele.lazzari@univr.it).'
+    # define the header of main- and sub-commands.
+    current_version = importlib.metadata.metadata("gsrap")["Version"]
+    header = f'gsrap v{current_version},\ndeveloped by Gioele Lazzari (gioele.lazzari@univr.it).'
     
     
     # create the command line arguments:
@@ -117,12 +120,36 @@ def main():
     # set up the logger:
     logger = get_logger('gsrap', args.verbose)
     
-    
+        
     
     # show a welcome message:
     set_header_trailer_formatter(logger.handlers[0])
     logger.info(header + '\n')
-    command_line = '' # print the full command line:
+    
+    
+    
+    # check if newer version is available
+    try:
+        response = requests.get(f"https://pypi.org/pypi/gsrap/json", timeout=3)  # sends an HTTP GET request to the given URL
+        response.raise_for_status()  # check the HTTP status code (e.g. 200, 404, 500): if not in the 2xx success range, raise requests.exceptions.HTTPError
+        data = response.json()
+        newest_version = data["info"]["version"] 
+    except Exception as error:  # eg requests.exceptions.Timeout, requests.exceptions.HTTPError
+        logger.info(f'Can\'t retrieve the number of the newest version. Please contact the developer reporting the following error: "{error}".')
+        logger.info('')  # still no formatting here
+        # do not exit, continue with the program
+    if version.parse(current_version) < version.parse(newest_version):
+        warning_message = f"███ Last version is v{newest_version} and you have v{current_version}: please update gsrap! ███"
+        border = ''.join(['█' for i in range(len(warning_message))])
+        logger.info(border)
+        logger.info(warning_message)
+        logger.info(border)
+        logger.info('')  # still no formatting here
+    
+    
+    
+    # print the full command line:
+    command_line = '' 
     for arg, value in vars(args).items():
         if arg == 'subcommand': command_line = command_line + f"gsrap {value} "
         else: command_line = command_line + f"--{arg} {value} "

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/commons/.ipynb_checkpoints/biomass-checkpoint.py

@@ -190,6 +190,10 @@ def introduce_universal_biomass(logger, dbexp, universe):
     r.build_reaction_from_string(rstring)
     
     
+    # add SBO annotation
+    r.annotation['sbo'] = ['SBO:0000629']  # biomass reaction 
+    
+    
     # set as objective:
     universe.objective = 'Biomass'
     

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/commons/.ipynb_checkpoints/coeffs-checkpoint.py

@@ -267,7 +267,7 @@ def compute_exp_LIPIDS_coeffs(logger, model, MWF, LIPIDS_PL, LIPIDS_FA):
     r.bounds = (0, 1000)
     r.gene_reaction_rule = 'spontaneous'   
     r.update_genes_from_gpr()
-    
+
     
     # determine 'L' formula and charge (charge should be -1 like every fatty acid)
     L_dict = dict()              # for 1 mol

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/commons/.ipynb_checkpoints/fluxbal-checkpoint.py

@@ -44,7 +44,7 @@ def verify_growth(model, boolean=True):
         if status =='infeasible':
             return 'infeasible'
         elif obj_value < get_optthr():
-            return 0
+            return 0.0
         else:
             return round(obj_value, 3)
         

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/commons/biomass.py

@@ -190,6 +190,10 @@ def introduce_universal_biomass(logger, dbexp, universe):
     r.build_reaction_from_string(rstring)
     
     
+    # add SBO annotation
+    r.annotation['sbo'] = ['SBO:0000629']  # biomass reaction 
+    
+    
     # set as objective:
     universe.objective = 'Biomass'
     

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/commons/coeffs.py

@@ -267,7 +267,7 @@ def compute_exp_LIPIDS_coeffs(logger, model, MWF, LIPIDS_PL, LIPIDS_FA):
     r.bounds = (0, 1000)
     r.gene_reaction_rule = 'spontaneous'   
     r.update_genes_from_gpr()
-    
+
     
     # determine 'L' formula and charge (charge should be -1 like every fatty acid)
     L_dict = dict()              # for 1 mol

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/commons/fluxbal.py

@@ -44,7 +44,7 @@ def verify_growth(model, boolean=True):
         if status =='infeasible':
             return 'infeasible'
         elif obj_value < get_optthr():
-            return 0
+            return 0.0
         else:
             return round(obj_value, 3)
         

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/mkmodel/.ipynb_checkpoints/gapfillutils-checkpoint.py

@@ -33,6 +33,9 @@ def import_from_universe(model, universe, rid, bounds=None, gpr=None):
     else:
         r.gene_reaction_rule = ''
     r.update_genes_from_gpr()
+    
+    # set annotations
+    r.annotation = ru.annotation
 
 
 

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/mkmodel/.ipynb_checkpoints/mkmodel-checkpoint.py

@@ -64,6 +64,7 @@ def create_model_incore(params):
     # remove universal orphans
     model = remove_universal_orphans(logger, model)
 
+    
 
     ###### PRUNING
     logger.info("Reading provided eggnog-mapper annotation...")
@@ -77,6 +78,7 @@ def create_model_incore(params):
     translate_remaining_kos(logger, model, eggnog_ko_to_gids)
     restore_gene_annotations(logger, model, universe, eggonog_gid_to_kos)
 
+    
 
     ###### GAPFILLING
     # force inclusion of reactions:  
@@ -103,30 +105,35 @@ def create_model_incore(params):
     if type(df_P)==int: return 1
 
 
-    ###### POLISHING 2
-    # remove disconnected metabolites
-    model = remove_disconnected(logger, model)
 
+    ###### POLISHING 2
     # remove unsed sinks and demands
     model = remove_sinks_demands(logger, model)  
+    
+    # remove disconnected metabolites
+    model = remove_disconnected(logger, model)
 
+    
 
     # # # # #   DERIVATION ENDS HERE   # # # # #
     log_metrics(logger, model)
     log_unbalances(logger, model)
 
 
+    
     ###### CHECKS
     # check blocked metabolites / dead-ends
     df_S = biosynthesis_on_media(logger, model, dbexp, args.gap_fill, args.biosynth)
     if type(df_S)==int: return 1
 
 
+
     ###### POLISHING 3
     # reset growth environment befor saving the model
     gempipe.reset_growth_env(model)
 
 
+    
     # output the model:
     logger.info("Writing strain-specific model...")
     cobra.io.save_json_model(model, f'{args.outdir}/{model.id}.json')        # JSON

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/mkmodel/gapfillutils.py

@@ -33,6 +33,9 @@ def import_from_universe(model, universe, rid, bounds=None, gpr=None):
     else:
         r.gene_reaction_rule = ''
     r.update_genes_from_gpr()
+    
+    # set annotations
+    r.annotation = ru.annotation
 
 
 

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/mkmodel/mkmodel.py

@@ -64,6 +64,7 @@ def create_model_incore(params):
     # remove universal orphans
     model = remove_universal_orphans(logger, model)
 
+    
 
     ###### PRUNING
     logger.info("Reading provided eggnog-mapper annotation...")
@@ -77,6 +78,7 @@ def create_model_incore(params):
     translate_remaining_kos(logger, model, eggnog_ko_to_gids)
     restore_gene_annotations(logger, model, universe, eggonog_gid_to_kos)
 
+    
 
     ###### GAPFILLING
     # force inclusion of reactions:  
@@ -103,30 +105,35 @@ def create_model_incore(params):
     if type(df_P)==int: return 1
 
 
-    ###### POLISHING 2
-    # remove disconnected metabolites
-    model = remove_disconnected(logger, model)
 
+    ###### POLISHING 2
     # remove unsed sinks and demands
     model = remove_sinks_demands(logger, model)  
+    
+    # remove disconnected metabolites
+    model = remove_disconnected(logger, model)
 
+    
 
     # # # # #   DERIVATION ENDS HERE   # # # # #
     log_metrics(logger, model)
     log_unbalances(logger, model)
 
 
+    
     ###### CHECKS
     # check blocked metabolites / dead-ends
     df_S = biosynthesis_on_media(logger, model, dbexp, args.gap_fill, args.biosynth)
     if type(df_S)==int: return 1
 
 
+
     ###### POLISHING 3
     # reset growth environment befor saving the model
     gempipe.reset_growth_env(model)
 
 
+    
     # output the model:
     logger.info("Writing strain-specific model...")
     cobra.io.save_json_model(model, f'{args.outdir}/{model.id}.json')        # JSON

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/.ipynb_checkpoints/annotation-checkpoint.py

@@ -66,6 +66,9 @@ def translate_annotate_genes(logger, model, idcollection_dict):
             g.annotation['ec'] = list(ko_to_ecs[ko])
             g.annotation['cog'] = list(ko_to_cogs[ko])
             g.annotation['go'] = list(ko_to_gos[ko])
+            
+        # add SBO annotation
+        g.annotation['sbo'] = ['SBO:0000243']  # demand reaction 
         
     
         

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/.ipynb_checkpoints/completeness-checkpoint.py

@@ -43,10 +43,10 @@ def check_completeness(logger, model, progress, module, focus, eggnog, zeroes, i
         for eggfile in eggnog:
             eggset = parse_eggnog(model, eggfile, idcollection_dict)
             kr_uni = kr_uni.union(eggset)
-        kr_uni_label = f"'{len(eggnog)} eggnog annotations'"
+        kr_uni_label = f"{len(eggnog)} eggnog annotations"
     else: 
         kr_uni = idcollection_dict['kr']
-        kr_uni_label = "'whole KEGG'"
+        kr_uni_label = "whole KEGG"
     
     
     # get all the 'kr' annotations in the model
@@ -55,7 +55,10 @@ def check_completeness(logger, model, progress, module, focus, eggnog, zeroes, i
         if 'kegg.reaction' in r.annotation.keys():
             for kr_id in r.annotation['kegg.reaction']:
                 kr_ids_modeled.add(kr_id)
-    logger.info(f"Universe coverage for {kr_uni_label}: {round(len(kr_ids_modeled.intersection(kr_uni))/len(kr_uni)*100, 0)}%!")
+    kr_uni_missing = len(kr_uni - kr_ids_modeled.intersection(kr_uni))
+    kr_uni_coverage = len(kr_ids_modeled.intersection(kr_uni)) / len(kr_uni) * 100
+    logger.info(f"Coverage for '{kr_uni_label}': {round(kr_uni_coverage, 0)}% ({kr_uni_missing} missing).")
+    
     
     
     # get all the map / md codes:
@@ -147,7 +150,7 @@ def check_completeness(logger, model, progress, module, focus, eggnog, zeroes, i
                 'missing': missing,
                 'md_ids': [j['md_id'] for j in i['mds']],
             })
-                
+            
             
     # order list by '%' of completness and print:
     list_coverage = sorted(list_coverage, key=lambda x: x['perc_completeness'], reverse=True)

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/.ipynb_checkpoints/introduce-checkpoint.py

@@ -141,6 +141,10 @@ def introduce_metabolites(logger, db, model, idcollection_dict, kegg_compound_to
         # save as list: 
         for ankey in ankeys:
             m.annotation[ankey] = list(m.annotation[ankey])
+            
+            
+        # add SBO annotation
+        m.annotation['sbo'] = ['SBO:0000247']  # generic metabolite
    
         
     
@@ -264,7 +268,8 @@ def introduce_transporters(logger, db, model, idcollection_dict, kegg_reaction_t
         m_e.name = m_c.name
         m_e.formula = m_c.formula
         m_e.charge = m_c.charge
-        m_e.annotation = m_c.annotation
+        
+        m_e.annotation = m_c.annotation   # transfer all annotations, including SBO!
             
     
     def add_exchange_reaction(model, mid_e):
@@ -283,6 +288,10 @@ def introduce_transporters(logger, db, model, idcollection_dict, kegg_reaction_t
             r.bounds = (-1000, 1000)
         else:
             r.bounds = (0, 1000)
+            
+        # add SBO annotation
+        r.annotation['sbo'] = ['SBO:0000627']  # exchange reaction    
+        
     
     
     
@@ -418,6 +427,9 @@ def introduce_sinks_demands(logger, model):
         r.name = f"Sink for {model.metabolites.get_by_id(f'{puremid}_c').name}"
         r.build_reaction_from_string(f'{puremid}_c <=> ')
         r.bounds = (-1000, 1000)
+        
+        # add SBO annotation
+        r.annotation['sbo'] = ['SBO:0000632']  # sink reaction 
     
     
     for puremid in demands: 
@@ -427,6 +439,9 @@ def introduce_sinks_demands(logger, model):
         r.name = f"Demand for {model.metabolites.get_by_id(f'{puremid}_c').name}"
         r.build_reaction_from_string(f'{puremid}_c --> ')
         r.bounds = (0, 1000)
+        
+        # add SBO annotation
+        r.annotation['sbo'] = ['SBO:0000628']  # demand reaction 
     
     
     return model

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/.ipynb_checkpoints/parsedb-checkpoint.py

@@ -113,7 +113,7 @@ def main(args, logger):
     
     ###### RECONSTRUCTION
     # create the model
-    universe = cobra.Model('newuni')   
+    universe = cobra.Model('universe')   
     logger.info("Parsing gsrap database...")
     
     # introduce M / R / T

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/.ipynb_checkpoints/repeating-checkpoint.py

@@ -125,6 +125,13 @@ def add_reaction(logger, model, rid, row, kr_ids, kegg_reaction_to_others, addty
         r.annotation[ankey] = list(r.annotation[ankey])
         
         
+    # add SBO annotation
+    if addtype=='R':
+        r.annotation['sbo'] = ['SBO:0000176']  # metabolic reaction
+    else:
+        r.annotation['sbo'] = ['SBO:0000185']  # transport reaction
+        
+        
     # check if unbalanced
     if r.check_mass_balance() != {}: 
         logger.error(f"{itemtype} '{r.id}' is unbalanced: {r.check_mass_balance()}.")

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/annotation.py

@@ -66,6 +66,9 @@ def translate_annotate_genes(logger, model, idcollection_dict):
             g.annotation['ec'] = list(ko_to_ecs[ko])
             g.annotation['cog'] = list(ko_to_cogs[ko])
             g.annotation['go'] = list(ko_to_gos[ko])
+            
+        # add SBO annotation
+        g.annotation['sbo'] = ['SBO:0000243']  # demand reaction 
         
     
         

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/completeness.py

@@ -43,10 +43,10 @@ def check_completeness(logger, model, progress, module, focus, eggnog, zeroes, i
         for eggfile in eggnog:
             eggset = parse_eggnog(model, eggfile, idcollection_dict)
             kr_uni = kr_uni.union(eggset)
-        kr_uni_label = f"'{len(eggnog)} eggnog annotations'"
+        kr_uni_label = f"{len(eggnog)} eggnog annotations"
     else: 
         kr_uni = idcollection_dict['kr']
-        kr_uni_label = "'whole KEGG'"
+        kr_uni_label = "whole KEGG"
     
     
     # get all the 'kr' annotations in the model
@@ -55,7 +55,10 @@ def check_completeness(logger, model, progress, module, focus, eggnog, zeroes, i
         if 'kegg.reaction' in r.annotation.keys():
             for kr_id in r.annotation['kegg.reaction']:
                 kr_ids_modeled.add(kr_id)
-    logger.info(f"Universe coverage for {kr_uni_label}: {round(len(kr_ids_modeled.intersection(kr_uni))/len(kr_uni)*100, 0)}%!")
+    kr_uni_missing = len(kr_uni - kr_ids_modeled.intersection(kr_uni))
+    kr_uni_coverage = len(kr_ids_modeled.intersection(kr_uni)) / len(kr_uni) * 100
+    logger.info(f"Coverage for '{kr_uni_label}': {round(kr_uni_coverage, 0)}% ({kr_uni_missing} missing).")
+    
     
     
     # get all the map / md codes:
@@ -147,7 +150,7 @@ def check_completeness(logger, model, progress, module, focus, eggnog, zeroes, i
                 'missing': missing,
                 'md_ids': [j['md_id'] for j in i['mds']],
             })
-                
+            
             
     # order list by '%' of completness and print:
     list_coverage = sorted(list_coverage, key=lambda x: x['perc_completeness'], reverse=True)

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/introduce.py

@@ -141,6 +141,10 @@ def introduce_metabolites(logger, db, model, idcollection_dict, kegg_compound_to
         # save as list: 
         for ankey in ankeys:
             m.annotation[ankey] = list(m.annotation[ankey])
+            
+            
+        # add SBO annotation
+        m.annotation['sbo'] = ['SBO:0000247']  # generic metabolite
    
         
     
@@ -264,7 +268,8 @@ def introduce_transporters(logger, db, model, idcollection_dict, kegg_reaction_t
         m_e.name = m_c.name
         m_e.formula = m_c.formula
         m_e.charge = m_c.charge
-        m_e.annotation = m_c.annotation
+        
+        m_e.annotation = m_c.annotation   # transfer all annotations, including SBO!
             
     
     def add_exchange_reaction(model, mid_e):
@@ -283,6 +288,10 @@ def introduce_transporters(logger, db, model, idcollection_dict, kegg_reaction_t
             r.bounds = (-1000, 1000)
         else:
             r.bounds = (0, 1000)
+            
+        # add SBO annotation
+        r.annotation['sbo'] = ['SBO:0000627']  # exchange reaction    
+        
     
     
     
@@ -418,6 +427,9 @@ def introduce_sinks_demands(logger, model):
         r.name = f"Sink for {model.metabolites.get_by_id(f'{puremid}_c').name}"
         r.build_reaction_from_string(f'{puremid}_c <=> ')
         r.bounds = (-1000, 1000)
+        
+        # add SBO annotation
+        r.annotation['sbo'] = ['SBO:0000632']  # sink reaction 
     
     
     for puremid in demands: 
@@ -427,6 +439,9 @@ def introduce_sinks_demands(logger, model):
         r.name = f"Demand for {model.metabolites.get_by_id(f'{puremid}_c').name}"
         r.build_reaction_from_string(f'{puremid}_c --> ')
         r.bounds = (0, 1000)
+        
+        # add SBO annotation
+        r.annotation['sbo'] = ['SBO:0000628']  # demand reaction 
     
     
     return model

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/parsedb.py

@@ -113,7 +113,7 @@ def main(args, logger):
     
     ###### RECONSTRUCTION
     # create the model
-    universe = cobra.Model('newuni')   
+    universe = cobra.Model('universe')   
     logger.info("Parsing gsrap database...")
     
     # introduce M / R / T

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/parsedb/repeating.py

@@ -125,6 +125,13 @@ def add_reaction(logger, model, rid, row, kr_ids, kegg_reaction_to_others, addty
         r.annotation[ankey] = list(r.annotation[ankey])
         
         
+    # add SBO annotation
+    if addtype=='R':
+        r.annotation['sbo'] = ['SBO:0000176']  # metabolic reaction
+    else:
+        r.annotation['sbo'] = ['SBO:0000185']  # transport reaction
+        
+        
     # check if unbalanced
     if r.check_mass_balance() != {}: 
         logger.error(f"{itemtype} '{r.id}' is unbalanced: {r.check_mass_balance()}.")

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/runsims/.ipynb_checkpoints/simplegrowth-checkpoint.py

@@ -12,9 +12,7 @@ from ..commons import verify_growth
 def grow_on_media(logger, model, dbexp, media, fva, universe_in_parsedb=False):
     
     
-    # if working with the universe:
-    if universe_in_parsedb:
-        log_for_parsedb = []
+
     
     
     # check if requested
@@ -58,7 +56,10 @@ def grow_on_media(logger, model, dbexp, media, fva, universe_in_parsedb=False):
         res_fba = verify_growth(model, boolean=False)
         df_G.loc[obj_id, f'{medium}'] = res_fba
         if universe_in_parsedb:
-            log_for_parsedb.append(f"'{medium}': {res_fba}")
+            if res_fba == 'infeasible' or res_fba == 0.0:
+                logger.warning(f"Growth on '{medium}': {res_fba}.")
+            else:
+                logger.info(f"Growth on '{medium}': {res_fba}.")
         
         
         # perform FVA if requested:
@@ -68,7 +69,5 @@ def grow_on_media(logger, model, dbexp, media, fva, universe_in_parsedb=False):
             for rid, row in df_fva.iterrows():
                 df_G.loc[rid, f'{medium}'] = f"({round(row['minimum'], 3)}, {round(row['maximum'], 3)})"
 
-    
-    if universe_in_parsedb:
-        logger.info(f"Results: {'; '.join(log_for_parsedb)}.")
+
     return df_G

{gsrap-0.7.0 → gsrap-0.7.1}/src/gsrap/runsims/simplegrowth.py

@@ -12,9 +12,7 @@ from ..commons import verify_growth
 def grow_on_media(logger, model, dbexp, media, fva, universe_in_parsedb=False):
     
     
-    # if working with the universe:
-    if universe_in_parsedb:
-        log_for_parsedb = []
+
     
     
     # check if requested
@@ -58,7 +56,10 @@ def grow_on_media(logger, model, dbexp, media, fva, universe_in_parsedb=False):
         res_fba = verify_growth(model, boolean=False)
         df_G.loc[obj_id, f'{medium}'] = res_fba
         if universe_in_parsedb:
-            log_for_parsedb.append(f"'{medium}': {res_fba}")
+            if res_fba == 'infeasible' or res_fba == 0.0:
+                logger.warning(f"Growth on '{medium}': {res_fba}.")
+            else:
+                logger.info(f"Growth on '{medium}': {res_fba}.")
         
         
         # perform FVA if requested:
@@ -68,7 +69,5 @@ def grow_on_media(logger, model, dbexp, media, fva, universe_in_parsedb=False):
             for rid, row in df_fva.iterrows():
                 df_G.loc[rid, f'{medium}'] = f"({round(row['minimum'], 3)}, {round(row['maximum'], 3)})"
 
-    
-    if universe_in_parsedb:
-        logger.info(f"Results: {'; '.join(log_for_parsedb)}.")
+
     return df_G