mir_eval icon indicating copy to clipboard operation
mir_eval copied to clipboard

Better docstrings for separation

Open craffel opened this issue 9 years ago • 2 comments

Currently, the docstrings for the metrics in mir_eval.separation are not super helpful - they basically say "Computes xyz function from bss_eval". Specifically, this makes it unclear what the difference between bss_eval_sources and bss_eval_images is. @ErikJohnsonCU @faroit @carlthome when one of you have a minute, can you make a PR to make them a bit more informative? Thanks!

craffel avatar Aug 28 '16 01:08 craffel

@craffel, I don't feel I have a solid enough grasp of the single-channel and multi-channel difference aside from bss_eval_sources pretty much only being a fair estimate for point sources and instantaneous mixes (let's say dry vocals recorded in a dampened booth, or line-in recorded Moogs), so I don't dare to add anything to the docstring. @faroit is the guru here. :hamster:

carlthome avatar Aug 30 '16 14:08 carlthome

Ping @aliutkus

carlthome avatar Aug 30 '16 14:08 carlthome