Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to read SED-ML file with csymbol definitionURL="http://sed-ml.org/#max" #44

Closed
0u812 opened this issue Mar 12, 2017 · 9 comments
Closed
Assignees

Comments

@0u812
Copy link
Contributor

0u812 commented Mar 12, 2017

Hi Frank,
I'm trying to read the SED-ML from your Combine archive example published with your paper here. It is the SED-ML included in the first supplementary file (also attached for reference). The problem seems to be caused by the http://sed-ml.org/#max csymbol definition. Here is the error output:

In SBML Level 2, the only values permitted for 'definitionURL' on a <csymbol> element are "http://www.sbml.org/sbml/symbols/time" and "http://www.sbml.org/sbml/symbols/delay". SBML Level 3 added "http://www.sbml.org/sbml/symbols/avogadro". SBML Level 3 Version 2 added "http://www.sbml.org/sbml/symbols/rateOf"
Reference: L3V2 Section 3.4.6
 The <csymbol> definitionURL 'http://sed-ml.org/#max' is not allowed for this level and version of SBML.

I am using libSEDML 0.4.2 built against libSBML 5.14.1 experimental from svn. I'm using the Python bindings.

repressilator-sedml.xml.txt

@fbergmann fbergmann self-assigned this Mar 13, 2017
@fbergmann
Copy link
Owner

Hello Kyle,

this error is obviously coming from libSBML, so it would seem like a new validation rule was added recently for this. Can you send me the CMakeCache.txt file that you used to compile libSBML? It would be good to know whether you used the new ASTNode implementation, or the old one.

I will also let the team know about this ...

cheers
Frank

@0u812
Copy link
Contributor Author

0u812 commented Mar 13, 2017

Thanks for your reply! Here is the CMakeCache.txt. I'm on revision r23826 of the experimental branch of libSBML. I am not currently using LIBSBML_USE_LEGACY_MATH, but I can give that a shot.
CMakeCache.txt

@0u812
Copy link
Contributor Author

0u812 commented Mar 14, 2017

Unfortunately, trying to open the same file with LIBSBML_USE_LEGACY_MATH enabled results in a segmentation fault.

@fbergmann
Copy link
Owner

Hello Kyle, please try the libSBML repo from today, we fixed this one earlier in the day ...

@0u812
Copy link
Contributor Author

0u812 commented Mar 14, 2017

Thanks so much! Now I am able to import the file with LIBSBML_USE_LEGACY_MATH enabled.

@0u812 0u812 closed this as completed Mar 14, 2017
@0u812
Copy link
Contributor Author

0u812 commented Sep 20, 2017

Hi Frank, I was just checking if this csymbol issue still requires legacy math enabled in libsbml. In Tellurium, we're currently using legacy math all the way through, but ideally we would like to switch it off at some point in the future:)

@fbergmann
Copy link
Owner

Hello Kyle, I've verified that the error is no longer being issued. However, currently it seems that libNuml will fail if libsbml is used without namespaces and arrays enabled.

@0u812
Copy link
Contributor Author

0u812 commented Sep 21, 2017

Thanks Frank! It sounds like enabling namespaces in libsbml, libsedml, and libnuml is a workable solution, correct?

@fbergmann
Copy link
Owner

Indeed, that sames the safest approach ... and of course to remain careful with the import order so that the correct symbols are available at all time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants