essentia icon indicating copy to clipboard operation
essentia copied to clipboard

Published 20 hours ago • verified publisher icon MTG

Test Cases for MultiPitchKlapuri and MultiPitchMelodia

Open cvf-bcn-gituser opened this issue 4 years ago • 2 comments

Similar to PR 1137 https://github.com/MTG/essentia/pull/1137 Extending test coverage to consider multipitch algorithms.

cvf-bcn-gituser avatar Apr 19 '21 15:04 cvf-bcn-gituser

@dbogdanov , updated to add melodia. this algorithm has a very similar i/o. It is the monophonic pre-decessor of Kalpuri. Still in draft mode but comments welcome.

cvf-bcn-gituser avatar May 03 '21 12:05 cvf-bcn-gituser

There is a FIXME in the Unit Tests where a crash occurs. The origin of the DIV by zero error is not in pitchmelodia.cpp. Its origin is "PSF" algorithm chain. This is being dealt with in PR # 1142.

cvf-bcn-gituser avatar May 10 '21 15:05 cvf-bcn-gituser