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

load on the processor. #12

Closed
masksshow opened this issue Aug 25, 2021 · 7 comments
Closed

load on the processor. #12

masksshow opened this issue Aug 25, 2021 · 7 comments
Labels
enhancement New feature or request

Comments

@masksshow
Copy link

Hi. Great project.
You can tell why recognition is causing such a heavy load on the processor.
I am sending a file of 500 kb, this is enough for the detection, but the CPU load is huge at the moment of recognition.
Can the load be reduced? Does it depend on the file format for recognition?
Thanks!

@dotX12
Copy link
Collaborator

dotX12 commented Aug 26, 2021

I know about this, but there is no way to get rid of it, the only solution is to transfer the task of searching for a song by file to a separate processor so as not to block the code. This is a difficult task of converting files, changing hertz patterns and searching for a signature in the shazam base.

@masksshow
Copy link
Author

Thanks.

Is there a way to reduce the load?
Can you explain why get_next_signature is needed?

I remember that in older versions of api shazam, it was necessary to send only RAW file data 44100Hz, 1 channel (Mono), signed 16 bit PCM. No re-encoding.
Thanks a lot!

@dotX12
Copy link
Collaborator

dotX12 commented Aug 27, 2021

Thanks.

Is there a way to reduce the load?
Can you explain why get_next_signature is needed?

I remember that in older versions of api shazam, it was necessary to send only RAW file data 44100Hz, 1 channel (Mono), signed 16 bit PCM. No re-encoding.
Thanks a lot!

Thanks, I'll check it out over the weekend.

@dotX12 dotX12 added the enhancement New feature or request label Aug 27, 2021
@dotX12
Copy link
Collaborator

dotX12 commented Sep 16, 2021

@masksshow,
Unfortunately, I don't have much time to check it myself.
If you have the time and opportunities, check it out and do a PR.

@dotX12
Copy link
Collaborator

dotX12 commented Sep 4, 2022

@dotX12
Copy link
Collaborator

dotX12 commented Dec 20, 2023

#78

@dotX12
Copy link
Collaborator

dotX12 commented Feb 23, 2024

Closed, problem fixed.
#76 (comment)

@dotX12 dotX12 closed this as completed Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants