Checking sample width, is it necessary? about inaspeechsegmenter HOT 6 CLOSED

Hi,

This value is actually hard-coded in sidekit >= 1.3.6 (which is used to load the audio file), can you make sure you have sidekit 1.3.6 or 1.3.7 installed?

from inaspeechsegmenter.

Yes, I installed the segmenter via pypi and it came with sidekit 1.3.6.9 (when I installed a while ago)

$ pip list  | grep -i sidekit
SIDEKIT                1.3.6.9

Then I tried upgrading sidekit but got the same error.

$ pip install --upgrade sidekit
Requirement already satisfied: sidekit in /usr/local/anaconda3/envs/some-env/lib/python3.6/site-packages (1.3.6.9)
Collecting sidekit
  Using cached SIDEKIT-1.3.8.5.2.tar.gz (169 kB)
Requirement already satisfied: numpy in /...
...

$ ina_speech_segmenter.py -i /input -o /output
2021-04-12 07:11:07.762836: I tensorflow/compiler/jit/xla_cpu_device.cc:41] Not creating XLA devices, tf_xla_enable_xla_devices not set
2021-04-12 07:11:07.763054: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
batch_processing 1 files
1/1 [('/output/some-file.csv', 2, "error: <class 'AssertionError'>")]

Finally, I also tried the same input files on a linux machine (ubuntu 20.04, ffmpeg 4.2.4), with no luck.

from inaspeechsegmenter.

#56 also reports on an Assertion Error and suggesting using py>=3.8. So I tried py 3.8.8, but it didn't fix the problem.

from inaspeechsegmenter.

Dear @keighrim ,

We experienced some issues due to recent updates of dependency SIDEKIT.
The issue is now solved, and the SIDEKIT related code has been copied directly within inaSpeechSegmenter source code.
In other words, the dependency to SIDEKIT has been removed.

It has been tested using python 3.6.9 and python 3.8.5.
It is supposed to work well for any python >= 3.6 .

Could you clone the latest version available on github, install it in a new virtual environment, and let me know if your issue is now solved ?

I'm looking forward to hearing from you before updating the pip repository as well.

Kind regards, Sorry for the inconvenience, and thanks a lot for this feedback !

from inaspeechsegmenter.

#56 also reports on an Assertion Error and suggesting using py>=3.8. So I tried py 3.8.8, but it didn't fix the problem.

This isn't true as when I was testing the segmenter 0.6.6 on python 3.8.8, I mistakenly passed a directory instead of a file to --input argument. When parameters are properly set, the segmenter successfully ran without an AssersionError. (FileNotFoundError might be a better message in this case, though)

Could you clone the latest version available on github, install it in a new virtual environment, and let me know if your issue is now solved ?

I tested on python 3.7 & segmenter 0.6.7 (from the master branch) and it ran fine. Also I'm very glad to see that pytorch is not installed as a dependency of the sidekit.

I think the issue is resolved. Please feel free to close the issue whenever you think so.

from inaspeechsegmenter.

Thanks a lot for this prompt answer, and for this constructive feedback !
This issue is now closed !

from inaspeechsegmenter.