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

TempoCNN minimum expected audio length #1387

Open
dbogdanov opened this issue Nov 30, 2023 · 0 comments
Open

TempoCNN minimum expected audio length #1387

dbogdanov opened this issue Nov 30, 2023 · 0 comments

Comments

@dbogdanov
Copy link
Member

dbogdanov commented Nov 30, 2023

On very short inputs (e.g., 8 seconds) the predictions by TempoCNN are 0 BPM:

[ WARNING ] TempoCNN: On the computation of majority voting, the second candidate, 0, obtained the same number of votes as the winning candidate, 0

The minimum expected length for the the input audio should be clarified in the documentation.

Should the E_WARNING above be more explicit on the reasons for such cases? Or there can be an input length check with a separate E_WARNING (or even exception) before running the network.

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

No branches or pull requests

2 participants