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

Field sequence mismatch errors on MIT Discursions (both sides, all areas) #565

Closed
happycube opened this issue Oct 25, 2020 · 1 comment
Closed
Assignees
Labels
bug regression bug directly caused by fixing another issue
Milestone

Comments

@happycube
Copy link
Owner

happycube commented Oct 25, 2020

https://www.lddb.com/laserdisc/55520/MIT/Discursions:-Architecture-Machine-Group

(probably a regression) - looks like field14 detection is bad through the disk.

./ld-decode ../mit_discursions_side1_2020-10-24_12-42-20.ldf mit -s 30000 -l 10 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30001 CAV frame 29669 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30002 CAV frame 29670 file frame 30003 CAV frame 29671 WARNING:root:Field phaseID sequence mismatch (4->3) (player may be paused) WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30004 CAV frame 29672 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30005 CAV frame 29673 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30006 CAV frame 29674 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30007 CAV frame 29675 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30008 CAV frame 29676 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30009 CAV frame 29677 WARNING:root:Field phaseID sequence mismatch (3->2) (player may be paused) file frame 30010 CAV frame 29678 saving JSON and exiting

@happycube happycube added the bug label Oct 25, 2020
@happycube happycube self-assigned this Oct 25, 2020
@happycube happycube added this to the Revision 7 milestone Oct 25, 2020
@happycube happycube changed the title Getting phase-tracking errors on MIT Discursions Field sequence mismatch errors on MIT Discursions (both sides, all areas) Oct 25, 2020
@happycube happycube added the regression bug directly caused by fixing another issue label Oct 25, 2020
@happycube
Copy link
Owner Author

Confirmed regression - the phase offset is about 90 degrees and the new code doesn't know what to do with it. Will probably apply to the National Gallery of Art disk too.

happycube added a commit that referenced this issue Oct 25, 2020
Fix Issue #565: re-add 90degree burst phase adjustments for NTSC
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug regression bug directly caused by fixing another issue
Projects
None yet
Development

No branches or pull requests

1 participant