VarIr / scikit-hubness

A Python package for hubness analysis and high-dimensional data mining
BSD 3-Clause "New" or "Revised" License
44 stars 10 forks source link

Puffinn LSH #30

Closed VarIr closed 4 years ago

VarIr commented 4 years ago

Introduce locality-sensitive hashing with the puffinn library #22 . Per default, when using algorithm='lsh', now puffinn is used instead of falconn. Puffinn is currently actively developed, and supports pickle-ing its index #20 #28

codecov-io commented 4 years ago

Codecov Report

Merging #30 into develop will decrease coverage by 0.35%. The diff coverage is 93.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #30      +/-   ##
===========================================
- Coverage    99.08%   98.72%   -0.36%     
===========================================
  Files           53       54       +1     
  Lines         4348     4380      +32     
  Branches       464      484      +20     
===========================================
+ Hits          4308     4324      +16     
- Misses          20       31      +11     
- Partials        20       25       +5
Impacted Files Coverage Δ
skhubness/neighbors/classification.py 100% <ø> (ø) :arrow_up:
skhubness/neighbors/graph.py 96.15% <ø> (ø) :arrow_up:
skhubness/neighbors/unsupervised.py 100% <ø> (ø) :arrow_up:
skhubness/neighbors/regression.py 100% <ø> (ø) :arrow_up:
skhubness/neighbors/lof.py 100% <ø> (ø) :arrow_up:
skhubness/analysis/estimation.py 99.57% <ø> (ø) :arrow_up:
skhubness/neighbors/__init__.py 100% <100%> (ø) :arrow_up:
skhubness/neighbors/tests/test_lof.py 96.59% <66.66%> (-1.28%) :arrow_down:
skhubness/utils/platform.py 71.42% <71.42%> (ø)
skhubness/neighbors/tests/test_lsh.py 96.2% <89.65%> (-3.8%) :arrow_down:
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4942550...40b7342. Read the comment docs.

lgtm-com[bot] commented 4 years ago

This pull request introduces 1 alert when merging 8d7852b6b5c06964434c9bc58bd99ce24bfd13cb into 4942550d29069856f7521010c5ad5e78cf1d7d81 - view on LGTM.com

new alerts:

VarIr commented 4 years ago

Closes #20 and #22