search

OP-TED / ePO

The eProcurement Ontology provides the formal, semantic foundation for the creation and reuse of linked open data in the domain of public procurement in the EU.
European Union Public License 1.2
58 stars 18 forks source link

Invalid ttl file for ePO core #469

Closed kvistgaard closed 6 months ago

kvistgaard commented 10 months ago

There is at least an error on this line https://github.com/OP-TED/ePO/blob/b68b9005eb5af1e5e518235becf015e3eef19fcc/implementation/ePO_core/owl_ontology/ePO_core_restrictions.ttl#L127, but the full file needs to pass Turtle validation. There might be some invalid URIs.

andreea-pasare commented 9 months ago

This was fixed for the latest release candidate, ePO v4.0.0-rc3 tag: https://github.com/OP-TED/ePO/blob/v4.0.0-rc.3/implementation/ePO_core/owl_ontology/ePO_core_restrictions.ttl

Indeed, there was a wrong value provided for an URI. Check the resolution of the following issue form model2owl repository: https://github.com/OP-TED/model2owl/issues/188

kvistgaard commented 9 months ago

@andreea-pasare , but that's the case also for the core ontology. The file in the master branch currently contains this line https://github.com/OP-TED/ePO/blob/master/implementation/ePO_core/owl_ontology/ePO_core.ttl#L5809

kvistgaard commented 9 months ago

Seems that not on the TLL but the RDF?XML serializations has issues as well org.semanticweb.owlapi.rdf.rdfxml.parser.RDFParserException: [line=8:column=157] Expecting rdf:RDF element.

cristianvasquez commented 8 months ago

This was fixed for the latest release candidate, ePO v4.0.0-rc3 tag: v4.0.0-rc.3/implementation/ePO_core/owl_ontology/ePO_core_restrictions.ttl

Indeed, there was a wrong value provided for an URI. Check the resolution of the following issue form model2owl repository: OP-TED/model2owl#188

@andreea-pasare, The problem was fixed in branch 4.0.0 but it persists in branch 4.1.0

I think the engine did not fail with malformed URIs in resources.

Effect: The RDF-XMl serialization is produced including a malformed URI, as seen here: https://github.com/OP-TED/ePO/blob/98e9dad5ef45682041587a5e69add8aa9a7bad0d/implementation/ePO_core/owl_ontology/ePO_core_restrictions.rdf#L77

When producing the turtle serialization it appears to be trunked: https://github.com/OP-TED/ePO/blob/98e9dad5ef45682041587a5e69add8aa9a7bad0d/implementation/ePO_core/owl_ontology/ePO_core.ttl#L5809

andreea-pasare commented 8 months ago

@cristianvasquez ePO v4.1.0 is not yet published. This is a work in progress, so the RDF outputs are not up to date yet.

cristianvasquez commented 6 months ago

Fixed in #475