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

ZeroDivisionError: Weights sum to zero, can't be normalized #12

Closed
shinchiro opened this issue Jan 10, 2015 · 8 comments
Closed

ZeroDivisionError: Weights sum to zero, can't be normalized #12

shinchiro opened this issue Jan 10, 2015 · 8 comments
Labels

Comments

@shinchiro
Copy link
Contributor

Encountered this error when testing a file:
http://www.datafilehost.com/d/d0cb08e1

Running in Sushi 0.4.3..

@tp7
Copy link
Owner

tp7 commented Jan 10, 2015

I'll also need the chapters. Sushi doesn't extract them to a separate file automatically, you can do this with mkvextract(gui).

@tp7 tp7 added the bug label Jan 10, 2015
@shinchiro
Copy link
Contributor Author

chapters:
http://www.datafilehost.com/d/a826697c

@tp7 tp7 closed this as completed in 78b6beb Jan 10, 2015
@shinchiro
Copy link
Contributor Author

thanks for fixing this problem. Anyhow when can I expect new release with this bug fixed

@tp7
Copy link
Owner

tp7 commented Jan 16, 2015

You can try this build, I won't be releasing a new official version for every minor update.

@shinchiro
Copy link
Contributor Author

thx, but I still getting the same error when trying other video:

Traceback (most recent call last):
File "sushi.py", line 730, in
File "sushi.py", line 723, in parse_args_and_run
File "sushi.py", line 603, in run
File "sushi.py", line 314, in average_shifts
File "numpy\lib\function_base.pyo", line 539, in average
ZeroDivisionError: Weights sum to zero, can't be normalized

@tp7
Copy link
Owner

tp7 commented Jan 16, 2015

You get same error every time one of the chapter groups ended up empty (all evens in there are linked to others). I'd have to look at your other sample to determine if it's the same kind of error or a completely different one.

@shinchiro
Copy link
Contributor Author

Here's sample:
http://www.datafilehost.com/d/c6563a21

tp7 added a commit that referenced this issue Jan 17, 2015
Previously we might create a chapter group where every single event would be linked, for example a group with only comments or zero-length lines. Now we merge events from these groups to their parents' groups and remove the empty leftovers.
Reported in #12
@tp7
Copy link
Owner

tp7 commented Jan 17, 2015

It was a completely different error - this one happened because your script had a chapter group with a single comment it in. Fixed in this build.

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