-
Notifications
You must be signed in to change notification settings - Fork 41
Description
I noticed recently that the models generated via all of the modelseed services have explicitly-represented boundary metabolites rather than the standard use of a true boundary (e.g., standard is extracellular metabolites -> Ø as boundary reactions; modelseed-generated SBML files now have extracellular metabolites -> boundary metabolite with no exchange reaction for the boundary metabolite). I am struggling to find which ModelSEED repository this change was implemented in so thought this would be a decent start.
This doesn't affect reconstructions read into COBRApy, since COBRApy will automatically add exchange reactions for boundary metabolites that don't have them, as in the current SBML models output by modelseed. But, this likely breaks many bespoke pipelines which had been using the _e/_e0 suffix for identification of exchange reactions or for other programs that do not automatically introduce exchange reactions for the new boundary metabolites.
I strongly recommend reverting the change, but understand there are some cases where this is convenient, e.g. multi-species models that need an explicit boundary (or an additional compartment) to differentiate conveniently between community and single-species uptake/secretion rates. But, this seems like it should not be the default for models generated for individual species.