-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Invalid match and poor confidence #63
Comments
Please shared the modifications to the script so I or others can run? Just so the issue is completely reproducible. |
Sure, I will try and add more instructions.
{
"database": {
"host": "127.0.0.1",
"user": "root",
"passwd": "",
"db": "dejavu_test2"
}
}
Results
Thanks! |
@balnagy ah I see. There is no problem with Dejavu, but the testing framework has a bug where if the name of your track on disk includes an underscore ( If you look in the
But if I extract a random 3 second segment from the Talyor Swift track to $ python dejavu.py recognize file mytest.wav
{'match_time': 0.3390800952911377, 'song_id': 1, 'confidence': 1326, 'song_name': 'Taylor Swift - Shake It Off-nfWlot6h_JM', 'offset': 646L} The problem is here. I didn't originally write the testing suite, but had a contributor kind enough to make it. It needs some love, though. Long term, this obviously needs to be fixed. In the short term, I might use instead 3-4 underscores ( |
@worldveil, wow, thanks. Now I repeated the test after renaming the file to 1.wav, so I have higher confidence (40-700), but the offset still doesn't match. I could imagine the song is repeatative, but none of the 5 samples matches, which I think very unlikely.
Song: https://www.youtube.com/watch?v=nfWlot6h_JM&t=170s |
@balnagy, apologies, I haven't had much time to look into these issues lately. Any progress or thoughts? |
It's very hard to debug such a problem, so I gave up and implemented my own algorithm just to find the offset, since I know the song. And it's kind of weird, since if the offset is not precise, then it makes the whole result questionable. Where would you start debugging? |
Not necessarily. The way music is produced now, many of the sounds are direct copies or looped clips, meaning that it might actually be legitimately ambiguous as to which loop the fingerprints matched to. I would start with the test case script and ensure the algorithm is actually messing up and not just the test suite. The test suite was contributed by someone, that, while I applaud the effort, leaves some room for improvement. If that isn't it, you might just need to tweak the parameters of the hashing to ensure better offset matching. |
Hi,
I would like to use this library to find the position only in one song, but it's not working at all for the song I'm using. I don't know if it's the problem of the song or the algorithm, but even the tests are failing with invalid match and zero confidence.
The song is very popular (Taylor Swift - Shake It Off), but it's licensed. You can try to get it with a few commands.
Then what I did was that I just modified test_dejavu.sh to scan wav files and then execute. I used wav files, because mp3 had a strange length, but wav seems to be ok.
Can you help me to fix this issue?
Thanks!
The text was updated successfully, but these errors were encountered: