-
Notifications
You must be signed in to change notification settings - Fork 12
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
Missing namespaces from OWX round trip #46
Comments
The OWL API produces this:
Where the prefixes are both specified as namespaces and prefixes. Horned just produces the prefixes. I may be slightly biased, but horned's approach seems more sensible. I have no ideal at all who is closer to the spec. @ignazio1977 @jamesaoverton Anyone got any ideas who is right? (Or rather, ignazio1977 will be right, of course, but why is he right?) |
I'm hardly ever right (just ask my kid), however here the difference is that the namespaces are XML functionality and the prefixes are syntax specific. So with the XML namespaces you can shorten XML elements, without you might have illegal XML, depending on the renderer. |
Thanks to @Eugenio2192, I have added an attempt at fixing this now. I'd welcome feedback on whether this work for its intended purpose. |
I think this can be closed, the namespaces are rendering properly. Regarding the pretty printing, I was working on a PR in tafia/quick-xml#731 but it may take some time until they decide if they merge it so we might need to wait for that. |
As we think this is all sorted, I will close now. Pretty printing can come later! |
ontology-tools/py-horned-owl#6
The text was updated successfully, but these errors were encountered: