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

nan values in the probability #68

Open
matnshrn opened this issue Jan 25, 2022 · 1 comment
Open

nan values in the probability #68

matnshrn opened this issue Jan 25, 2022 · 1 comment

Comments

@matnshrn
Copy link

Hey, thank you for this project!
I'm using the branch with the probability field (https://github.com/ina-foss/inaSpeechSegmenter/tree/mf_proba),
and found that in some cases the probability field contains nan.
After debugging, I found that it happens because in the segmenter.py file, row 251, you replaced 0.5 probabilities in the probability scores array (named framepred) with nan, and after that when calculating the overall probability, you are averaging an array with nans, resulting with nan probability.

Is this the desired behavior? Why not just keeping probability 0.5?

Thank you.

@matnshrn matnshrn changed the title nan values in the probaiblity nan values in the probability Jan 25, 2022
@DavidDoukhan
Copy link
Member

Dear @matnshrn ,
thanks for this comment related to this "not-so-used" branch.
In order to help you with your issue, could you send me a sample of audio excerpt resulting in NaNs ?
I haven't work on this branch since few months, and I'll need some time to remember if this behavior is normal :-)

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

No branches or pull requests

2 participants