OpenTreeOfLife / treemachine

Source tree graph database
Other
16 stars 6 forks source link

clarify mrca web service docs on the issue of MRTA #148

Closed mtholder closed 8 years ago

mtholder commented 9 years ago

Currently it says the nearest_taxon... properties refer to the most recent taxonomic ancestor which is "the closest taxonomic node to the MRCA node in the synthetic tree"

It is really "the closest taxonomic node to the MRCA node in the synthetic tree as one moves towards the root" right? (it will always be the smallest taxon that encompasses the query, not just the "closest" node).

perhaps the current text is fine for a directional distance, but it would not hurt to spell it out.

josephwb commented 9 years ago

Fixed with https://github.com/OpenTreeOfLife/treemachine/commit/659d903fad1ce0cdc8d5e966c3ac71c9d3c010ee