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

Crash in refine_linelocs_hsync when hsync_area is empty #405

Closed
atsampson opened this issue Jan 11, 2020 · 4 comments
Closed

Crash in refine_linelocs_hsync when hsync_area is empty #405

atsampson opened this issue Jan 11, 2020 · 4 comments
Assignees
Labels
ld-decode An issue only affecting the ld-decode[r]

Comments

@atsampson
Copy link
Collaborator

While decoding a clip from the middle of EE1053:

$ ld-decode --pal ee1053-lineloc-crash.raw.oga out/tmp
file frame 1 CLV timecode 41:06.09 frame 61659
file frame 2 CLV timecode 41:06.10 frame 61660
file frame 3 CLV timecode 41:06.11 frame 61661
[...]
file frame 147 CLV timecode 41:12.06 frame 61806
file frame 148 CLV timecode 41:12.07 frame 61807
file frame 149 CLV timecode 41:12.08 frame 61808
ERROR - please paste the following into a bug report:
current sample: 240714034.2967033
arguments: Namespace(MTF=None, MTF_offset=None, WibbleRemover=False, daa=False, ignoreleadout=False, infile='ee1053-lineloc-crash.raw.oga', inputfreq=None, length=110000, lowband=False, nodod=False, noefm=False, ntsc=False, ntscj=False, outfile='out/tmp', pal=True, seek=-1, start=0, start_fileloc=-1, threads=5, vbpf_high=None, verboseVITS=False, vlpf=None)
Exception: zero-size array to reduction operation minimum which has no identity  Traceback:
  File "/home/ats/pkg/ld-decode/bin/ld-decode", line 129, in <module>
    f = ldd.readfield()
  File "/home/ats/src/ld-decode/lddecode/core.py", line 2607, in readfield
    f, offset = self.decodefield(initphase = initphase)
  File "/home/ats/src/ld-decode/lddecode/core.py", line 2590, in decodefield
    f = self.FieldClass(self.rf, self.rawdecode, audio_offset = self.audio_offset, prevfield = self.curfield, initphase = initphase)
  File "/home/ats/src/ld-decode/lddecode/core.py", line 2047, in __init__
    super(FieldPAL, self).__init__(*args, **kwargs)
  File "/home/ats/src/ld-decode/lddecode/core.py", line 1823, in __init__
    self.linelocs2 = self.refine_linelocs_hsync()
  File "/home/ats/src/ld-decode/lddecode/core.py", line 1630, in refine_linelocs_hsync
    if nb_min(hsync_area) < self.rf.iretohz(-55) or nb_max(hsync_area) > self.rf.iretohz(30):
  File "/gar/lib/python3.8/site-packages/numba/targets/arraymath.py", line 502, in array_min_impl
    raise ValueError(MSG)

Starting later just produces ERROR:root:Skipped field on the offending field.

I've put ee1053-lineloc-crash.raw.oga in sftp:adamsampson/Faults.

@atsampson atsampson added the ld-decode An issue only affecting the ld-decode[r] label Jan 11, 2020
@happycube happycube self-assigned this Jan 12, 2020
@happycube
Copy link
Owner

happycube commented Jan 12, 2020

Got it downloaded and reproduced, investigating slowly. Scratching my head over the skipped field bit, too. (repro'd starting at field 299). I'll see if I can figure it out in the morning...

@happycube
Copy link
Owner

happycube commented Jan 12, 2020

Started an issue405 branch. This is going to take a while as I'm going to clean things up to make them more deterministic (i.e. starting each field decode at a block boundary) and at least somewhat better commented.

@happycube
Copy link
Owner

happycube commented Jan 21, 2020

There's something wrong with the player tracking and a skip in the demod here. That said, it still shouldn't crash. :)

https://drive.google.com/open?id=1VB3UliqrqJlxVCQrFVgxcRhq0fB5zyJm

happycube added a commit that referenced this issue Jan 21, 2020
@happycube
Copy link
Owner

This should be fixed now, feel free to reopen if you see otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ld-decode An issue only affecting the ld-decode[r]
Projects
None yet
Development

No branches or pull requests

2 participants