biothings / biothings.species

This implements species API as part of BioThings API
Apache License 2.0
0 stars 3 forks source link

include_children parameter does not work #3

Closed newgene closed 3 years ago

newgene commented 3 years ago

For this query:

http://t.biothings.io/v1/taxon/1280?include_children=1

when include_children=1 is passed, the children field should return a list of children nodes of the current taxonomy node. Currently it's returned as an empty list.

The expected children nodes are like this list:

https://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi?id=1280

newgene commented 3 years ago

Another example is the human species node (id: 9606):

http://t.biothings.io/v1/taxon/9606?include_children=1

v.s.

https://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi?id=9606

zcqian commented 3 years ago

fixed via https://github.com/biothings/biothings.species/commit/f1a768de83aad416154713e89ee32460f431992e