geneontology / go-fastapi

https://api.geneontology.org/
4 stars 3 forks source link

taxon:1 vs NCBITaxon:1 for goid response from ontology/term/ endpoint #95

Open tmushayahama opened 3 weeks ago

tmushayahama commented 3 weeks ago

The API endpoint https://api.geneontology.org/api/ontology/term/NCBITaxon:1 returns id: taxon:1 instead of NCBITaxon:1

    {
       "goid":"taxon:1",
       "label":"root",
       ...
    }

However plugging taxon:1 as a closure to GOlr doesn't seem to work, but NCBITaxon:1 works

cc @sierra-moxon @kltm @vanaukenk

cmungall commented 3 weeks ago

It should be NCBITaxon:1

And to point out the obvious: it's not a GO ID. It's too late to fix this now I assume

vanaukenk commented 3 weeks ago

Noting from the 2024-06-13 workbenches call, that it's possible that other applications are expecting the existing value. We'll want to check on this before putting a fix in.

kltm commented 3 weeks ago

Also, as this has been there a while, it might be good to see if any of our widgets, etc., already code to this "bug".

sierra-moxon commented 1 week ago

debugging results:

cmungall commented 1 week ago

should the db-xrefs.yaml 'database' parameter for NCBITaxon change from taxon to NCBITaxon?

I think this is a good idea. We should also include a comment field indicating that taxon: is used in some file formats like GAF for legacy reasons.

sierra-moxon commented 3 days ago

Per discussion in developers' Slack, we will not change the hard coding of this endpoint to reflect its ability to search and return non-GO ontology identifiers.

The other half of this ticket is fixed in geneontology/go-site, but also has the potential to break other things.

So for now, this ticket won't be fixed.