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

[Nintendo Switch] Image resample for low resolution novels #32

Open
felixsrg opened this issue Feb 18, 2021 · 7 comments
Open

[Nintendo Switch] Image resample for low resolution novels #32

felixsrg opened this issue Feb 18, 2021 · 7 comments
Labels
enhancement New feature or request

Comments

@felixsrg
Copy link

Please do a quick search before making a feature request to see if someone else has already made a similar one!

Is your feature request related to a problem? Please describe.
Not really a problem, most novels have a 640x480 resolution, on Switch they look really pixelated.

Describe the solution you'd like
Maybe it's possible to apply a image filtering or resampling to make the novels look a little bit better?

Describe alternatives you've considered
I was able to resize all "background" and "foreground" pictures to 1280x720 and it looks better, but that would have to be applied per novel.

@felixsrg felixsrg added the enhancement New feature or request label Feb 18, 2021
@ajusa
Copy link
Owner

ajusa commented Feb 20, 2021

From my understanding, you are talking about rescaling the image, correct? An earlier version of the project did actually rescale all of the images, but then someone pointed out that they looked stretched. I'll add an option in the settings menu that will allow people to change whether the image is stretched, or if the original aspect ratio should be preserved.

image

@felixsrg
Copy link
Author

Not really rescaling to fit the whole screen. I was thinking maybe keeping the original aspect ratio but applying a linear or bicubic interpolation. This for the image to look a little bit smoother when novels have a lower resolution compared to the console's. I don't know if it's possible, asking just in case.

@ajusa
Copy link
Owner

ajusa commented Feb 20, 2021

By default I believe there should be a linear interpolation being applied, I'll double check and see if that's the case.

@ajusa
Copy link
Owner

ajusa commented Feb 20, 2021

Nearest Neighbor:
image
Linear (default)
image
Note that these images were taken on my computer, rather than on the Switch directly. There's a chance that it might be doing nearest neighbor on the switch rather than linear, but I haven't tested that yet.

It honestly might just be that the original images are pretty low resolution to begin with. I did consider looking into projects like Waifu2x or Anime4k for dynamic upscaling, but those are too complex and resource hungry to embed for now.

@felixsrg
Copy link
Author

I see, it's just that on my Nintendo Switch, the images look more like if they were using the Nearest Neighbor method, here are a screenshot and the original picture for comparisson:

Original novel background file with resolution of 640x480:
FGNY02A
Here's the screenshot taken from the Switch at 1280x720:
2021022014373800-DA63280140B1530CD17755515D814CFE

I'm really sorry if this is just how it's supposed to look, just wanted to confirm.

@ajusa
Copy link
Owner

ajusa commented Feb 20, 2021

That definitely looks like an issue then, thanks for bringing it to my attention (and for the screenshot)! This is blocked by #30, so I won't be able to try fixing it until I can sort out that bug. Hopefully it doesn't take too long!

@felixsrg
Copy link
Author

I think I kind of understand understand why this is blocked by the other issue. Well, hope it can be fixed soon too. Thank you very much for bringing us this wonderful project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants