-
Notifications
You must be signed in to change notification settings - Fork 38
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
Make versionIRIs resolvable #131
Comments
TO DO: |
The versioned URL is http://purl.obolibrary.org/obo/ms/releases/4.1.94/psi-ms.obo |
Ironically, the versioned IRI for Unit Ontology listed at OLS gives a 404 error: It should be But the OP's recommendation was "URI is the non-versioned PURL for either the obo or owl file". So I'm not sure we need to change to versioned URIs at all? |
your obo and owl files have data version metadata which is good. For example:
However, these are not resolvable
Fortunately it should be not too hard to make these resolvable as you are following good versioning practice and making a github releases for every release
The way to do this is to make a PR on this that performs the necessary redirect to the raw github file:
https://github.com/OBOFoundry/purl.obolibrary.org/blob/master/config/ms.yml
I would recommend doing this for both the obo and the owl file
This is a little fiddly as you don't include a
/version/
in the IRI so either you will have a separate entry for every release (not recommended) OR have a regex that matches semver. I can help with this if you would like versions resolvableI would also recommend that PURLs are used in mzML. E.g. I see files like this:
however, the raw github URL resolves to a newer version and if the user really did want to match the PSI-MS version with that which was used for the file it's not obvious
Also these github URLs are subject to change. Perhaps in future UO may not be served from github, or it may move to a different org, or paths may change - this is why we created the OBO purl system
In fact we can see this with the current version of the standard which implicitly recommends sourceforge URLs
http://www.peptideatlas.org/tmp/mzML1.1.0.html#cvList
I recommend
The text was updated successfully, but these errors were encountered: