-
Notifications
You must be signed in to change notification settings - Fork 22
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
Ontology Resource object weird properties names #75
Comments
@mbaudis wasn't that schema inherited from somewhere else? |
This a conceptual discrepancy. "filteringTerms": [
{
"id": "NCIT:C20197",
"label": "Male",
"resources": [
{
"id": "NCIT",
"iriPrefix": "http://ncicb.nci.nih.gov/xml/owl/EVS/Thesaurus.owl#",
"name": "NCI Thesaurus",
"nameSpacePrefix": "NCIT",
"url": "https://www.ebi.ac.uk/ols/ontologies/ncit/download",
"version": ""
} I see Michael uses "response": {
"filteringTerms": [
{
"id": "EDAM:operation_3227",
"label": "Variant Calling",
"type": "EDAM ontology"
"resources": [
{
"id": "EDAMoperation",
"iriPrefix": "http://purl.obolibrary.org/obo/EDAM_",
"name": "EDAM ontology",
"url": "https://github.com/edamontology/edam-browser"
}, Progenetx uses "type" for the resource matching and the "iriPrefix" is not the namespace, but lexical prefix. |
@redmitry Well,
|
Hello,
Although not an issue, but rather comment.
The beaconFilteringTermsResults.json Resource object has two bad-named properties:
"iriPrefix" - is essentially the "namespace" - the default ontology namespace.
"nameSpacePrefix" - which is the namespace prefix which is used to shorthand the identifiers (IRIs).
from the ontology point of view should be just "namespace" and "prefix".
Best,
Dmitry
The text was updated successfully, but these errors were encountered: