change the start of j val to 1

worldveil / dejavu

Audio fingerprinting and recognition in Python

MIT License

6.36k stars 1.43k forks source link

change the start of j val to 1 #35

Closed suxianbaozi closed 10 years ago

suxianbaozi commented 10 years ago

when j=0 ,i+j is also equal to i,that will make so many same hashes from diffent songs

worldveil commented 10 years ago

There is a UNIQUE constraint on the fingerprints table, thus no duplicates will be added.

However for sake of correctness I'll merge this. Thanks for contributing!

suxianbaozi commented 10 years ago

I'm sorry,I mean diffents songs has same hash，，when reconizing, one hash will affect too more rows that will spend so much time，I know that UNIQUE on the fingerprints,but that is in the same song,you can make a test to about 500 songs,and check the time spended!