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

s18: sub pixel mixing #727

Closed
jotego opened this issue Jul 12, 2024 · 6 comments
Closed

s18: sub pixel mixing #727

jotego opened this issue Jul 12, 2024 · 6 comments
Assignees
Labels
core bug Something isn't working

Comments

@jotego
Copy link
Owner

jotego commented Jul 12, 2024

There is a faint yellow line around the character portrait

image

Likely cause

This might be due to the vdp_sel being evaluated at the full clock rate. The vdp pixel is changing in between two pxl_cen pulses causing sub-pixel effects.

This should be visible in simulation waveforms, but maybe not in the simulation output image (as it is sampled at pxl_cen).

@jotego jotego added the core bug Something isn't working label Jul 12, 2024
@asturur
Copy link

asturur commented Jul 12, 2024

just a notice that the initial screen of moonwalker that is black and white is a great example of the issue ( clearly visible ) and is immediately after boot if that helps with simulation

@jotego
Copy link
Owner Author

jotego commented Jul 13, 2024

just a notice that the initial screen of moonwalker that is black and white is a great example of the issue ( clearly visible ) and is immediately after boot if that helps with simulation

Very good advice. Thank you.

@Zichio
Copy link

Zichio commented Jul 14, 2024

I think these might be the same issue. I'm seeing blue outlines on both Moonwalker and Shadow Dancer ("Shadow Dancer (World).mra" and "Michael Jackson's Moonwalker (World) (FD1094-8751 317-0159).mra"). Looks like a blue outline to the left of the text. Been like this since the first release, was waiting to see if it would be fixed before reporting it.

These blue pixels are not present in MAME (tested newest version 0.267). I also did not spot the issue in some direct pcb footage i watched (but that was really low quality and hard to tell 100%).

Attatching two cropped screenshots (direct screenshot capture on MiSTer).

Moonwalker start-screen after adding a credit:
Screenshot 2024-07-14 133904

Shadow Dancer, intro to the first level:
Screenshot 2024-07-14 134021

Thanks for the excellent work as always JT and Team!

@gyurco
Copy link
Collaborator

gyurco commented Jul 27, 2024

Likely cause

This might be due to the vdp_sel being evaluated at the full clock rate. The vdp pixel is changing in between two pxl_cen pulses causing sub-pixel effects.

Just a note: on the schematics, while vdp_sel is combinatorial inside the PAL, the signals coming from the tilemap chip are delayed by 2 pixels (and registered with the pixel clock), due to they goes through 2 D flip-flops.

@gyurco gyurco self-assigned this Jul 31, 2024
@gyurco
Copy link
Collaborator

gyurco commented Jul 31, 2024

The bug (and the fix) also affects Super Hang-On.

@jotego
Copy link
Owner Author

jotego commented Aug 3, 2024

Fixed in 9807462 by @gyurco. @jtmiki mark it as completed.

@jotego jotego closed this as completed Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core bug Something isn't working
Projects
Status: ✅ Done
Development

No branches or pull requests

5 participants