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

Misaligned fields on Clerks side 2 #783

Open
atsampson opened this issue Aug 27, 2022 · 4 comments
Open

Misaligned fields on Clerks side 2 #783

atsampson opened this issue Aug 27, 2022 · 4 comments
Assignees
Labels
becausetechnidisc (or other dodgy mastering issues) deficiency ld-decode An issue only affecting the ld-decode[r]
Milestone

Comments

@atsampson
Copy link
Collaborator

(This is probably an after-rev7 kind of problem.)

Using current master, with a Technidisc-pressed copy of Clerks (3618 AS), side 1 decodes happily, but side 2 - which is quite long, at 59m49s - produces errors throughout:

Frame 1/110000: File Frame 1: CLV Timecode CLV Timecode 0:00.00 frame 0         
At field #2, Field phaseID sequence mismatch (2->1) (player may be paused)
At field #3, Field phaseID sequence mismatch (1->4) (player may be paused)
Frame 2/110000: File Frame 2: CLV Timecode CLV Timecode 0:00.01 frame 1         
At field #4, Field phaseID sequence mismatch (4->3) (player may be paused)
At field #5, Field phaseID sequence mismatch (3->2) (player may be paused)
[...]

Looking at the TBC, this is because some fields are being offset to the right by 2 samples. The video signal appears to have fairly ringy transitions on the edges of the sync pulses, so I wonder if the TBC is locking to an incorrect edge...

clerks-ring

I've put samples of both sides on the SFTP under /sftp/adamsampson/Faults/issue783.

@atsampson atsampson added deficiency ld-decode An issue only affecting the ld-decode[r] labels Aug 27, 2022
@Gamnn
Copy link
Contributor

Gamnn commented Aug 28, 2022

Similar issue on this sample, worked around with --video_lpf 4.7
@happycube asked me to note it here.

/sftp/chadpage/issues/i783-phase.ldf

@happycube happycube added the becausetechnidisc (or other dodgy mastering issues) label Aug 28, 2022
@happycube happycube added this to the Revision 8 milestone Aug 28, 2022
@atsampson
Copy link
Collaborator Author

It's all OK if I decode with --video_lpf 4.0 -- if I go much higher than that then I get increasing numbers of misaligned fields.

@happycube
Copy link
Owner

4.0 is very low - I was able to use 4.7 on the sample you posted, can you post a more difficult one?

@atsampson
Copy link
Collaborator Author

I've put up a couple more clips - 3618as-side2-middle1.ldf needs 4.4, 3618as-side2-start.ldf needs 4.2, and 3618as-side2-middle2.ldf needs 4.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
becausetechnidisc (or other dodgy mastering issues) deficiency ld-decode An issue only affecting the ld-decode[r]
Projects
None yet
Development

No branches or pull requests

3 participants