n_trees is annoy_n_trees in bbknn 1.5.x

[pinin4fjords]

Latest BBKNN versions (>=1.5.x) uses annoy_n_trees in place of n_trees, just applying that fix here.

[codecov]

Codecov Report

Merging #1873 (f48f234) into master (4dd8de9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1873   +/-   ##
=======================================
  Coverage   71.22%   71.22%           
=======================================
  Files          92       92           
  Lines       11181    11181           
=======================================
  Hits         7964     7964           
  Misses       3217     3217           

Impacted Files	Coverage Δ
scanpy/external/pp/_bbknn.py	58.33% <ø> (ø)

[ktpolanski]

I've already opened a more comprehensive PR for this five days ago, but haven't heard back on it.

[pinin4fjords]

Ahh, apologies @ktpolanski , closing now.

[ktpolanski]

All good, thanks for doing this :)

[genecell]

Hi,

I also met this problem. I am using Scanpy 1.7.2, and could you please suggest which version of BBKNN I should use if I don't want to update Scanpy to 1.8.0.

Thanks,
Min

[ktpolanski]

Two options:

• bbknn < 1.5.0
• use bbknn.bbknn() instead of the scanpy wrapper

[genecell]

Two options:

• bbknn < 1.5.0
• use bbknn.bbknn() instead of the scanpy wrapper

Thank you for your rapid reply, bbknn 1.4 works for me (scanpy 1.7.2). The second option is good.
Thanks!