Skip to content
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

resolves the infinite loop issue when recognizing song through a micr… #89

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vc13
Copy link

@vc13 vc13 commented Sep 7, 2015

Executing "python dejavu.py --recognize mic 10" causes script to fall into a very long loop or a memory error because the argument for time (10 in this case) is passed as string to djv.recognize() function. Inside this function this string argument causes either a very long loop or a memory error. Casting time argument to integer solves the issue.

@pimpmypixel
Copy link

Hello vc13 - I'm trying to get the microphone recognize done on a raspberry pi and keep getting the error you mention http://pastebin.com/Hjf92w5M and this one http://pastebin.com/84exHUvq
I edited the code to cast the seconds as int() but still the same error and now wondering if perhaps you know what to do from here?

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

Successfully merging this pull request may close these issues.

3 participants