phyloref / phyloref-ontology

Phyloreferencing Ontology and OWL DL reasoning with phyloreferences
Creative Commons Zero v1.0 Universal
5 stars 1 forks source link

Translated ontology in RDF/XML with an OWL Functional Syntax #20

Closed gaurav closed 6 years ago

gaurav commented 6 years ago

This PR translates phyloref.owl into OWL Functional Syntax as phyloref.ofn. To prevent duplication, it also deletes the old phyloref.owl -- phyloref.ofn will become the definitive ontology, and consumers can recreate the RDF/XML file as needed.

gaurav commented 6 years ago

I think those are both good reasons not to merge right away, and there isn't an urgent need to do so until we start adding terms to the Phyloref Ontology in earnest. Maybe we should close this PR for now and revisit it at that time?

hlapp commented 6 years ago

I think what you're trying to suggest is the following:

  1. Let's turn this from a PR into an issue because although there are good reasons to assume we'll come back to this in the not too distant future, we'll have to redo it then, because it's a conversion.
  2. Even in RDF/XML changes will still be relatively easy to identify and evaluate so long as the OWL API version does not also change dramatically between old and new. I.e., so long as we take to separate OWL API upgrades from actual changes to the ontology, we should be OK, at least for some time?
  3. Although ultimately we will want an automated release process, you think implementing it now is premature.

I don't necessarily feel the same way about (3) but am willing to buy into (1) and (2), so if this captures your argument, feel free to take the corresponding action (file issue, close this PR, convert ontology to latest OWL API).

gaurav commented 6 years ago

Yup, that's what I meant! Closing this PR now; we can continue the discussion in #24 and re-open it when we're ready to build a proper release workflow.