-
-
Notifications
You must be signed in to change notification settings - Fork 311
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
IndexError: list index out of range (on video_track = mkv.video_tracks[0]) #256
Comments
That's because your MKV has weird metadata, it seems like the SeekHead contains other SeekHead elements. |
No problem with subliminal 0.6.4
|
As you can see in subliminal 0.6.4 logs, there is a SeekHead element in the SeekHead element:
https://github.com/Diaoul/enzyme-old/blob/develop/enzyme/mkv.py#L440 is responsible for this. New enzyme doesn't support this and I don't think it should as it might lead to an infinite loop if the nested SeekHead element references itself. |
Another solution used by VLC is to bail at 10 seekhead to prevent recursion: http://www.videolan.org/developers/vlc/modules/demux/mkv/matroska_segment.cpp |
Thank you for detailed explanation. And sorry for adding new problems to your TODO :) |
Fixed with ccfd341 |
Although due to performance impact, I limit subliminal to the first SeekHead element, that should be enough in most cases. |
Thanks. Tested and it works like expected now. No crash and continue searching other files on directory.
|
Don't know why I get this error.
The text was updated successfully, but these errors were encountered: