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

high_volubility.py: using only sad #122

Open
GladB opened this issue May 20, 2019 · 1 comment
Open

high_volubility.py: using only sad #122

GladB opened this issue May 20, 2019 · 1 comment

Comments

@GladB
Copy link

GladB commented May 20, 2019

When using the sad and not diar parameter, high_volubility.py seems to crash on my end. I ran

python utils/high_volubility.py data/ACLEW_high_volubility/0256.wav --sad NoisemesSad &> log.txt

and got the log.txt file that I am attaching here.

log.txt

Since it crashes, the temp files still exist at the end of the run (which may or may not be something to fix)

@fmetze
Copy link
Contributor

fmetze commented May 21, 2019

So, I am not able to reproduce the first error

Traceback (most recent call last):
File "yunified.py", line 159, in
for feat in readHtk(INPUT_DIR+"/"+file, HTK_CHUNKSIZE, preSamples):
File "G/coconut/fileutils/htk.py", line 43, in readHtk
f.seek(-(preSamples * sampSize), 1)
OSError: [Errno 22] Invalid argument

without access to 0256.wav - please provide the file if you want me to look into this.

I can reproduce the second error

Traceback (most recent call last):
File "utils/high_volubility.py", line 604, in
main()
File "utils/high_volubility.py", line 571, in main
sorted_files = read_analyses(temp_abs, args.sad, nb_chunks*3, args.diar, args.mode)
File "utils/high_volubility.py", line 344, in read_analyses
"to increase the size of the chunks.")
ValueError: No moments for the {} mode has been found.
Try to decrease the step parameter or to increase the size of the chunks.

and this seems to be a problem in the fundamental way the high_volubility.py script is written - the script crashes, if no segments of the target type are being found in a "chunk". It seems @alecristia is the original author, maybe she can look into this.

fmetze added a commit that referenced this issue Jun 18, 2019
…ally wrt #122 and #125. may still need improvements to cmd line parameters, expected behavior, or documentation/ code match
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