Bioconductor / GenomicRanges

Representation and manipulation of genomic intervals
https://bioconductor.org/packages/GenomicRanges
43 stars 18 forks source link

GenomicRanges::findKNN masks BiocNeighbors::findKNN #42

Closed LTLA closed 4 years ago

LTLA commented 4 years ago

Or vice versa. It's not a disaster as people can always disambiguate with ::. But it seems like we have an opportunity to just avoid such problems with a different name, given that GenomicRanges::findKNN was only been exported in this devel cycle. findKNearest() would thematically fit with the names of other functions, e.g., nearest(), distanceToNearest().