rcenvironment / rce

Distributed, workflow-driven integration environment
https://rcenvironment.de/
51 stars 16 forks source link

Attach xsd files when integrating CPACS tools #23

Open MarAlder opened 4 years ago

MarAlder commented 4 years ago

In issue #484 we plan to exclude the toolspecifics from CPACS. This is realized by the xsd:any element with processContents=strict attribute, i.e. the XML processor must obtain the schema for the required namespaces for a proper validation of the XML files. The tool owners must provide this schema file.

The integration of a CPACS tool in RCE provides the possibility to add a documentation file. The same way tool integrators need a possibility to attach the corresponding schema file. This would enable users can run proper validation.

ArneBachmannDLR commented 4 years ago

I like the idea of keeping the schema with the integrated tool in RCE, but this is no strict requirement. If the XSD is specified as an URL inside the XML, any processor/validator could just download and cache the file, when validation becomes necessary (first run?). Also this assumes only one schema file is necessary to validate any dataset to this integrated tool, doesn't it?

MarAlder commented 4 years ago

Of course, both variants are and should be possible. Attaching several schema files would also be practical (this also applies to the documentation).

From the user's point of view, workflow integrators in particular want quick access to the corresponding schema files. Here it would be extremely laborious to find the Github/Gitlab/SVN repository first (if such exists or is accessible...), to find the corresponding version of the integrated tool and to finally find the corresponding schema file. Therefore I think that the tool owner should get the possibility to attach one or more schema files when integrating his tool into RCE.

Edit: I get your point that the URL replaces the need to search for the correct schema file. But I still think it prevents many problems if we just provide the schema in RCE.

janflink commented 4 years ago

An issue has been added to our internal issue tracker: https://mantis.sc.dlr.de/view.php?id=17074