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

Error: middle_length is greater than the length of the audio file #168

Closed
GennadiiSherstiuk opened this issue Mar 15, 2017 · 7 comments
Closed
Labels

Comments

@GennadiiSherstiuk
Copy link

GennadiiSherstiuk commented Mar 15, 2017

We have a problem ('STEP 8 (compute alignment level 2) FAILURE') with multipass-based audio alignment of relatively small fragments of text.

Part of the log:
[CRIT] ExecuteTask: STEP 8 (compute alignment level 2) FAILURE
[CRIT] ExecuteTask: Unexpected error while executing task
[CRIT] ExecuteTask: interval.end is after self.end
[ERRO] CLI: An unexpected error occurred while executing the task:
[ERRO] An unexpected error occurred while executing the task:
[ERRO] CLI: Unexpected error while executing task : interval.end is after self.end
[ERRO] Unexpected error while executing task : interval.end is after self.end
[DEBU] CLI: Execution completed with code 1

The following are:
https://www.dropbox.com/s/1ft6b2d2ax3twfg/audio.wav?dl=0 - audio file;
https://www.dropbox.com/s/j6ph20dycf0tgub/text.txt?dl=0 - text file;
https://www.dropbox.com/s/qj0s8xn0fi821tb/cmdAeneas.txt?dl=0 - call aeneas by command;
https://www.dropbox.com/s/r60ge28l71z6hq8/log.txt?dl=0 - full log.

A large fragment containing the same text passed normally.

@readbeyond
Copy link
Owner

readbeyond commented Mar 15, 2017 via email

@GennadiiSherstiuk GennadiiSherstiuk changed the title STEP 8 (compute alignment level 2) FAILURE STEP 9 (compute alignment level 3) FAILURE Mar 15, 2017
@GennadiiSherstiuk
Copy link
Author

GennadiiSherstiuk commented Mar 15, 2017 via email

@GennadiiSherstiuk GennadiiSherstiuk changed the title STEP 9 (compute alignment level 3) FAILURE STEP 8 (compute alignment level 2) FAILURE Mar 15, 2017
@readbeyond
Copy link
Owner

readbeyond commented Mar 15, 2017 via email

pettarin pushed a commit to pettarin/aeneas that referenced this issue Mar 15, 2017
readbeyond added a commit that referenced this issue Mar 15, 2017
@readbeyond
Copy link
Owner

readbeyond commented Mar 15, 2017

Hi,

I have just pushed on the devel/ branch a fix which should solve the issue.

The problem was here:

b22b3a0#diff-9bcb44f3905b7921ec0fa793a802bad3

Basically, that division TimeValue(self.__samples_length / self.audio_sample_rate) produced 5.6 (float) as a result. However, the TimeValue( ... / ... ) created a TimeValue(5.5999999...) instead of TimeValue(5.6), and this made the check in audiofilemfcc.py to fail when comparing 5.6 < 5.59999... .

I am a bit puzzled by the fact that this issue never occurred before --- but of course the fact that your audio file was exactly 5.600 s long (i.e., 89600 samples at 16000 Hz) is a bit peculiar.

I am leaving this issue open until I release a new version on master/ and on PyPI, which will happen in the next days, after adding a new test for this case, and running the full tests. If you want to fetch this fix, just git clone and switch to the devel/ branch.

Thank you for reporting the issue and providing the files to reproduce the issue.

@readbeyond readbeyond changed the title STEP 8 (compute alignment level 2) FAILURE Error: middle_length is greater than the length of the audio file Mar 15, 2017
@readbeyond readbeyond added the bug label Mar 15, 2017
@GennadiiSherstiuk
Copy link
Author

Hi. Thank you for your quick reply. We have updated aeneas to version 1.7.3 and all works fine.

pettarin pushed a commit to pettarin/aeneas that referenced this issue Mar 16, 2017
@readbeyond
Copy link
Owner

readbeyond commented Mar 16, 2017 via email

readbeyond added a commit that referenced this issue Mar 16, 2017
Added regression test for issue #168. Added -f/--full to read_audio
readbeyond added a commit that referenced this issue Mar 16, 2017
@readbeyond
Copy link
Owner

Merged into master. Closing.

cbenhagen pushed a commit to cbenhagen/aeneas that referenced this issue Aug 27, 2017
cbenhagen pushed a commit to cbenhagen/aeneas that referenced this issue Aug 27, 2017
andreasscherman pushed a commit to andreasscherman/aeneas that referenced this issue Jan 11, 2018
andreasscherman pushed a commit to andreasscherman/aeneas that referenced this issue Jan 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants