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

Improve video quality on Wii #65

Merged
merged 1 commit into from Oct 2, 2019
Merged

Improve video quality on Wii #65

merged 1 commit into from Oct 2, 2019

Conversation

Extrems
Copy link
Contributor

@Extrems Extrems commented Apr 15, 2019

No description provided.

@endrift
Copy link
Contributor

endrift commented May 5, 2019

Got around to testing this (albeit on component cables). Looks good to me.

@gingerbeardman
Copy link

This looks good to me also

@gingerbeardman
Copy link

gingerbeardman commented Jun 3, 2019

Here are some photos of my 640x480p LCD displaying Mario Kart Wii both with and without the 480p fix. This is great because the 1:1 pixel output allows you to see the differences at a sub-pixel resolution.

Places to look for differences

  1. e of Channel
  2. inside the a of Back
  3. glasses frame on my Mii
  4. Ms of Main Menu

Conclusion

  • fixed: graphics are pixel aligned horizontally
  • original: graphics are not pixel aligned horizontally

cropped photo album
https://imgur.com/a/Ef02mmz

full size photo album
https://imgur.com/a/R0k3igo

fixed: single pixel wide glasses frame
IMG_6337

original: double pixel wide glasses frame
IMG_6341

fixed: m starts on pixel boundary
IMG_6338

original: m starts half way through a pixel
IMG_6342

Original (before/after) checkerboard image (click to view 1:1):
480p-comparison

@WinterMute
Copy link
Contributor

This should at least have some explanation of what the mode parameter actually does. It would be good if there were named parameters rather than just random numbers.

@WinterMute WinterMute merged commit b90c941 into devkitPro:master Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants