-
Notifications
You must be signed in to change notification settings - Fork 8
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
[Collection] IRI Generation #368
Comments
The summary looks good to me! I'll see if we can't also add ARC, our MIAPPE and CREDiT into the TS4TIB. That would take a while though. I'll update you on any changes |
@Hannah-Doerpholz I just remembered, that Example: https://ontobee.org/ontology/MS?iri=http://purl.obolibrary.org/obo/MS_1000031 Do we have any replacement for this? |
I will implement a hardcoded logic to cover the cases where we know that the standard PURL is wrong and we have a functioning alternative. |
@HLWeil Thank you! A workaround for MS would be to not rely on the purls but link directly to OLS4, since the terms are displayed correctly there. That would mean the following:
|
Another option might be to go through Bioregistry. Bioregistry is an identifier resolver. Here, there is always some RDF information about how the URI format for a term should look like. For example in ENVO:
Our imported ontologies that are in Bioregistry: Ontologies we host on GitHub ourselves that are in Bioregistry: Ontologies we host on GitHub that are NOT in Bioregistry (I could add them though): For PO and RO, the workarounds as MS could be:
|
Thanks a lot for your thorough input, @Hannah-Doerpholz! So maybe we could use bioregistry in general? E.g. instead of And instead of Would be kind of a practical unification. Edit: Doesn't work for |
@Hannah-Doerpholz, I opened a PR with the changes, this took a quite a bit longer as other tests still used hard-coded "deprecated" URLs. With this change, the namespaces discussed here should result in working URLs, but working towards unification would still be highly welcome. Especially the If parsing demands change (e.g. if ontologies are added to bioregistry), feel free to reopen this issue and add the requirements. |
Yes! We are working on the update for Swate with ARCtrl 2.0.0 integration. The expected release date (if not required earlier) is the 27.06.2024. |
I think we should finally create a unified logic for uri generation.
The current logic, shown here might not be sufficient, to handle all different kinds of url.
Below i will try to summarize the requirements:
http://purl.org/nfdi4plants/ontology/dpbo/DPBO_0002006
(TS4TIB-Service)http://www.ebi.ac.uk/efo/EFO_0006571
TS4TIB
An external ontology service in cooperation with DataPLANT:
quoting @Hannah-Doerpholz
I will close all related issues to track them here. @Hannah-Doerpholz please verify if this issue roughly sums up the requirements 🙂
The text was updated successfully, but these errors were encountered: