Skip to content
This repository has been archived by the owner on Jul 15, 2019. It is now read-only.

doc/examples.html and lsh.py seem to have outdated information. #1

Open
yvesx opened this issue Nov 10, 2013 · 0 comments
Open

doc/examples.html and lsh.py seem to have outdated information. #1

yvesx opened this issue Nov 10, 2013 · 0 comments

Comments

@yvesx
Copy link

yvesx commented Nov 10, 2013

line 1056 in lsh.py suggests to use N=100000 data points.
but the create procedure python2.6 lsh.py -d 5 -create needs to find out the nearest neighbors for N points using brutal force i.e. quadratic complexity. For N=100000, it would never finish. I tried just N=1000 and the create procedure finishes within 1 minute.

Another thing on doc/examples.html is also, I believe, inaccurate.

D=5;
N=100000;

clear results
for r=0:2
results(r+1) = CalculateMPLSHParameters(D, N, ... dnnHist, dnnBins, danyHist, danyBins, deltaTarget, r, uHash, uCheck);
end

But the matlab function CalculateMPLSHParameters isn't in the repo (did I miss something here?).
I changed that line to this and worked for me.
results(r+1) = CalculateLSHParameters( N, dnnHist, dnnBins, danyHist, danyBins);

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant