ModelSEED SBML model can't be read due to duplicated IDs

[dengxiao01]

Problem description

Code Sample

Create a minimal, complete, verifiable example.

Context

[matthiaskoenig]

What is the result of the SBML validation? I am pretty sure your SBML is invalid and you should fix the issues in your model file.

[dengxiao01]

What is the result of the SBML validation? I am pretty sure your SBML is invalid and you should fix the issues in your model file.

SBML validation of web site, http://sbml.org/validator, is invalid, 404, is there where you can verify？

[matthiaskoenig]

Sorry, link changed with the site migration.
https://sbml.org/validator_servlet/
Best Matthias

[matthiaskoenig]

Fixed in #1239

[Midnighter]

Any chance that you can add a redirect on the proxy that runs sbml.org?

[dengxiao01]

Sorry, link changed with the site migration. https://sbml.org/validator_servlet/ Best Matthias
thanks,this is the result of the SBML validation:
This document is not valid SBML!
Error Line 1181 Column 54: (SBML Validation Rule #10301) The value of the 'id' field on every instance of the following type of object in a model must be unique: , , , , , , , , , , and model-wide s. Note that and parameters defined inside a reaction are treated separately. Reference: L2V1 Section 3.5 The id 'M_cpd02701_b' conflicts with the previously defined id 'M_cpd02701_b' at line 674.

[dengxiao01]

Any chance that you can add a redirect on the proxy that runs sbml.org?

Thank you. How do I do that? I'm not adding weighted direction

[Midnighter]

Any chance that you can add a redirect on the proxy that runs sbml.org?

Thank you. How do I do that? I'm not adding weighted direction

Sorry, that was a comment for @matthiaskoenig

[cdiener]

Here is a MRE:

1. Go to https://modelseed.org/genomes/Plants and download the A. Thaliana (2nd in the list) SBML model.
2. Read the model with cobra.io.read_sbml_model

This will result in:

[...]
ValueError: id 'cpd02701_b' at index 1154 is non-unique. Is it present twice?

The above exception was the direct cause of the following exception:

CobraSBMLError                            Traceback (most recent call last)
Input In [8], in <cell line: 1>()
----> 1 mod = read_sbml_model("Athaliana-TAIR10.sbml")

File ~/code/cobrapy/src/cobra/io/sbml.py:472, in read_sbml_model(filename, number, f_replace, **kwargs)
    462 except Exception as original_error:
    463     cobra_error = CobraSBMLError(
    464         "Something went wrong reading the SBML model. Most likely the SBML"
    465         " model is not valid. Please check that your model is valid using "
   (...)
    470         "at https://github.com/opencobra/cobrapy/issues ."
    471     )
--> 472     raise cobra_error from original_error

CobraSBMLError: Something went wrong reading the SBML model. Most likely the SBML model is not valid. Please check that your model is valid using the `cobra.io.sbml.validate_sbml_model` function or via the online validator at http://sbml.org/validator .
	`(model, errors) = validate_sbml_model(filename)`
If the model is valid and cannot be read please open an issue at https://github.com/opencobra/cobrapy/issues .

Which points towards the same issue as the SBML validator. The issue is that ModelSEED provided invalid SBML models (duplicated IDs). @Midnighter Do you know where we should report this issue and what ModelSEED uses to generate the SBML model?

[matthiaskoenig]

@Midnighter I just opened the issue for the redirect. This should be fixed soon. See sbmlteam/sbml-org-website#39

[dengxiao01]

I tried to read it without error using libsbml：
reader = libsbml.SBMLReader()
document = reader.readSBML(sbml_test_file)
print(document.getNumErrors())
print(document.printErrors())
but the ModelSEED provided invalid SBML models (duplicated IDs),How does cobrapy solve this problem？

[matthiaskoenig]

Reading the document with libsbml does not perform validation. You have to run the validation before you will see the actual errors. LibSBML will provide exactly the same errors (i.e. you model is invalid because you have duplicate species in it), because cobrapy and the sbml.org validator are using libsbml.
In summary, your model is invalid and you have to fix the model. This is not related to cobrapy but to ModelSEED creating/distributing invalid/broken models. You have to fix your duplicate species. This can be complicated, because both will most likely appear in reactions and you have to resolve which of the species belongs actual to which reaction.
In summary: this is not an issue of cobrapy, but of your model.

[cdiener]

Closing in favor of the attached ModelSEED issue where that issue originates.