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

Solid red areas on NTSC can have herringbone effect #206

Open
Gamnn opened this issue May 13, 2019 · 11 comments
Open

Solid red areas on NTSC can have herringbone effect #206

Gamnn opened this issue May 13, 2019 · 11 comments
Assignees
Labels
enhancement ld-decode An issue only affecting the ld-decode[r]

Comments

@Gamnn
Copy link
Contributor

Gamnn commented May 13, 2019

Possibly related to #190, merge if so.

Tested on rev4, issue exists there as well.

ntscherring.lds.zip

Frame 3 ld-decode output:
frame-2

s-video capture:
vlcsnap-2019-05-13-19h19m21s735

@happycube happycube self-assigned this May 31, 2019
@happycube happycube added this to the ld-decode rev5.1 milestone May 31, 2019
@happycube
Copy link
Owner

It's regressed and now vrooping

@Gamnn
Copy link
Contributor Author

Gamnn commented May 31, 2019

vroop regression caused by ee8a5c1.

@happycube
Copy link
Owner

oopsie. backed out unintentional commit.

@happycube
Copy link
Owner

happycube commented Jun 6, 2019

I'm playing around with RF decoding parameters and they help a lot. So a new option for less agressive/sharp settings is a good idea...

ldd.rf.DecoderParams['video_bpf'] = (3500000, 12500000)
ldd.rf.DecoderParams['video_bpf_order'] = 4

ldd.rf.DecoderParams['video_lpf_freq'] = (4200000)
ldd.rf.DecoderParams['video_lpf_order'] = 6

happycube added a commit that referenced this issue Jun 8, 2019
@happycube
Copy link
Owner

Added a new --video-bpf-high parameter (in mhz) to ld-decode.py just now. "--video_bpf_high 12.2" squelches it on this disk.

@simoninns
Copy link
Collaborator

It would be good to generate a wiki page for all these additional options (and explain briefly what they are for as well as the range of usable settings)

@happycube
Copy link
Owner

Added new page "Fine tuning parameters" but there should be a troubleshooting page as well.

@happycube
Copy link
Owner

Closing this, at least for now :)

@happycube happycube reopened this Dec 30, 2019
@Gamnn
Copy link
Contributor Author

Gamnn commented Dec 30, 2019

On some sources, adjusting the bpf doesn't clear up the wibble before considerably weakening the chroma and washing out colors.

Attaching some more samples across a variety of discs and players.

akowibble-a-x9.zip
akowibble-b-x9.zip
akowibble-c-x9.zip
akowibble-eb-dvl919.zip
akowibble-e-dvl919.zip
akowibble-r-clds315.zip

@happycube
Copy link
Owner

happycube commented Dec 30, 2019

Still experimenting - the below RFParams block helps with akowibble-a-x9.ldf and the majo x9 sample but doesn't eliminate the wibble.

In the case of majo there's a spurious signal at the burst frequency and a harmonic around the main wave at 13.8?mhz. ld-decode's defaults are sharp enough to pick this up. This is not like the PAL fault though, since it does not occur in the VITS area.

RFParams_NTSC = {
    # The audio notch filters are important with DD v3.0+ boards
    'audio_notchwidth': 350000,
    'audio_notchorder': 2,

    'video_deemp': (120*.32, 320*.32),

    # This BPF is similar but not *quite* identical to what Pioneer did
    'video_bpf_low': 4000000, 
    'video_bpf_high': 13000000,
    'video_bpf_order': 4,

    # This can easily be pushed up to 4.5mhz or even a bit higher. 
    # A sharp 4.8-5.0 is probably the maximum before the audio carriers bleed into 0IRE.
    'video_lpf_freq': 4200000,   # in mhz
    'video_lpf_order': 6, # butterworth filter order

    # MTF filter
    'MTF_basemult': .4, # general ** level of the MTF filter for frame 0.
    'MTF_poledist': .9,
    'MTF_freq': 12.2, # in mhz

    # used to detect rot
    'video_hpf_freq': 10000000,
    'video_hpf_order': 4,

    'audio_filterwidth': 150000,
    'audio_filterorder': 800,
}

@simoninns simoninns added enhancement ld-decode An issue only affecting the ld-decode[r] labels Jan 1, 2020
@happycube happycube removed this from the ld-decode rev5.2 milestone Jan 7, 2020
@Gamnn
Copy link
Contributor Author

Gamnn commented Feb 10, 2020

Between --WibbleRemover and --lowband, and the manual high/low pass filters, I think this can be closed now. The wiki could use an explanation of the options and when/how to use them however.

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

No branches or pull requests

3 participants