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

Letterboxing still present in MGS2 & MGS3 - 1440p #2

Closed
ShizCalev opened this issue Oct 25, 2023 · 17 comments
Closed

Letterboxing still present in MGS2 & MGS3 - 1440p #2

ShizCalev opened this issue Oct 25, 2023 · 17 comments

Comments

@ShizCalev
Copy link
Contributor

ShizCalev commented Oct 25, 2023

2023-10-25_19-39-38-METAL_GEAR_SOLID2-METAL_GEAR_SOLID_2_SONS_OF_LIBERTY
MGSHDFix.log

2023-10-25_19-32-22-METAL_GEAR_SOLID3-METAL_GEAR_SOLID_3_SNAKE_EATER
2023-10-25_19-30-07-METAL_GEAR_SOLID3-METAL_GEAR_SOLID_3_SNAKE_EATER
MGSHDFix.log

Noticed someone else reported the same issue on this forum too
https://www.resetera.com/threads/metal-gear-solid-master-collection-vol-1-pc-performance-thread.777830/page-5#post-113930219

@ShizCalev ShizCalev changed the title Letterboxing still present in MGS3 - 1440p Letterboxing still present in MGS2 & MGS3 - 1440p Oct 25, 2023
@Lyall
Copy link
Owner

Lyall commented Oct 26, 2023

Hmm, I'm not getting that issue on my end. I think there's some odd things going on with the framebuffer setup. Or the game is displaying incorrectly in fullscreen. You could try running SpecialK with the game, it might let it display correctly. Either way I'll look into this further when I can.

Here's a 1440p shot on my end.
1440p

@emoose
Copy link
Contributor

emoose commented Oct 26, 2023

The game does a strange thing where it checks the resolutions your display supports, then seems to scale the 1280x720 width/height to match the ratio of the largest resolution reported from it, which caused black bars for 4K users before: https://steamcommunity.com/app/2131650/discussions/0/3878221733380149058/

It might be a similar issue here too, I added a patch for that in the generic resolution mod, maybe worth testing if it goes away with that: https://steamcommunity.com/sharedfiles/filedetails/?id=3059974819
(happy for that to be added here too, or maybe it's worth looking into a better way for it)

@Lyall
Copy link
Owner

Lyall commented Oct 26, 2023

The game does a strange thing where it checks the resolutions your display supports, then seems to scale the 1280x720 width/height to match the ratio of the largest resolution reported from it, which caused black bars for 4K users before: https://steamcommunity.com/app/2131650/discussions/0/3878221733380149058/

It might be a similar issue here too, I added a patch for that in the generic resolution mod, maybe worth testing if it goes away with that: https://steamcommunity.com/sharedfiles/filedetails/?id=3059974819 (happy for that to be added here too, or maybe it's worth looking into a better way for it)

That's some great info, thanks @emoose. I wonder if at 4K it's maybe users with TVs that have a max resolution of 4096x2160.

When looking further on my end, when I set my desktop res to 2560x1440 and run the game at that forced resolution, the framebuffer resolution reads as 3440x1440 (my max resolution) in SpecialK. With the instruction from your mod NOPd, the framebuffer resolution in SpecialK reports as 2560x1440.

@ShizCalev
Copy link
Contributor Author

ShizCalev commented Oct 26, 2023

The game does a strange thing where it checks the resolutions your display supports, then seems to scale the 1280x720 width/height to match the ratio of the largest resolution reported from it, which caused black bars for 4K users before: https://steamcommunity.com/app/2131650/discussions/0/3878221733380149058/

It might be a similar issue here too, I added a patch for that in the generic resolution mod, maybe worth testing if it goes away with that: https://steamcommunity.com/sharedfiles/filedetails/?id=3059974819 (happy for that to be added here too, or maybe it's worth looking into a better way for it)

You ABSOLUTE chad, that fixed the issue. SpecialK was reporting that my framebuffer resolution was 1365x720. Looks like yours works fine for MG1/MG2 & MGS3, but MGS2 crashes when it's applied.

Yes, I do have a TV, and yes, its max res is 4096x2160 (17:9) while I'm set to 2560x1440(16:9). Hit the nail on the head.
image
I rock it in 1440 though for 120Hz refresh rate vs 60Hz at 4k.
image

@ShizCalev
Copy link
Contributor Author

To note, in addition to MGS2 & MGS3, the issue also affects MG1/MG2SS (the MSX games.)

MGS1 and the NES games do NOT have this issue.

@curryking3
Copy link

I have the same issue.

The game is likely picking the aspect ratio based on the highest supported resolution aspect ratio, in the case of my 4K TVs with native aspect ratio of 16:9 and 3840x2160p native resolution, the highest resolution supported is actually above this with a slightly wider aspect ratio with 4096x2160p.

image

@ShizCalev
Copy link
Contributor Author

ShizCalev commented Oct 26, 2023

I have the same issue.

The game is likely picking the aspect ratio based on the highest supported resolution aspect ratio, in the case of my 4K TVs with native aspect ratio of 16:9 and 3840x2160p native resolution, the highest resolution supported is actually above this with a slightly wider aspect ratio with 4096x2160p.

Someone shared this steam guide over on Reddit to override what resolutions your monitor reports to the display drivers, it fixed the issue outright for me and also corrected issues in a few other games I have. Now I can go back to using MGSHDFix without any other patches needed. :)
https://steamcommunity.com/sharedfiles/filedetails/806615157

@curryking3
Copy link

I have the same issue.
The game is likely picking the aspect ratio based on the highest supported resolution aspect ratio, in the case of my 4K TVs with native aspect ratio of 16:9 and 3840x2160p native resolution, the highest resolution supported is actually above this with a slightly wider aspect ratio with 4096x2160p.

Someone shared this steam guide over on Reddit to override what resolutions your monitor reports to the display drivers, it fixed the issue outright for me and also corrected issues in a few other games I have. Now I can go back to using MGSHDFix without any other patches needed. :) https://steamcommunity.com/sharedfiles/filedetails/806615157

It is a good possible fix and I'm sure works on most, but CRU sometimes causes display problems too, at least it just did for me, I had to reset the changes to even get my display to show anything again!

Just having a forced aspect ratio in the mod would be best really, that way it wouldn't require playing with EDID of displays.

Lyall added a commit that referenced this issue Oct 27, 2023
MGS 2 | MGS 3: Added windowed mode toggle and option to disable mouse cursor.  Thanks @emoose ! #4
@Lyall
Copy link
Owner

Lyall commented Oct 27, 2023

Okay I've added a framebuffer fix thanks to @emoose kindly sharing their hard work. I'll be doing a bit more testing with it and then the fix for this issue should come in the next build. It also addresses issue #6.

@Lyall
Copy link
Owner

Lyall commented Oct 27, 2023

Since this is related to the other issue can @ShizCalev and/or @curryking3 try out this test build of v0.6? It includes @emoose's fixes and should theoretically address the issue.

@Reonu
Copy link

Reonu commented Oct 27, 2023

@Lyall Not sure where to ask this: Am I supposed to enable "Fix Aspect Ratio" and "Fix FMVs" if playing on a non-ultrawide resolution, such as 2560x1440?

@ShizCalev
Copy link
Contributor Author

Since this is related to the other issue can @ShizCalev and/or @curryking3 try out this test build of v0.6? It includes @emoose's fixes and should theoretically address the issue.

Confirming that it is fixed in 0.6. Thanks for implementing that!

@Lyall
Copy link
Owner

Lyall commented Oct 27, 2023

@Lyall Not sure where to ask this: Am I supposed to enable "Fix Aspect Ratio" and "Fix FMVs" if playing on a non-ultrawide resolution, such as 2560x1440?

In the next build they will be automatically disabled if the fix detects that you're running at 16:9. So basically you won't need to worry about editing those settings.

@ShizCalev
Copy link
Contributor Author

Resolved by ffbfbb2

@curryking3
Copy link

Working great, appreciate your work, nice fix!

20231027190829_1
20231027190921_1
20231027190932_1
20231027190934_1

@ShizCalev
Copy link
Contributor Author

ShizCalev commented Oct 28, 2023

@Lyall If it's not too much of a bother, MG1 / MG2 (the MSX games) also have the same exact aspect ratio issue - it also does benefit from having a high resolution patch (rendering at native resolution works with MGSResolutionPatch currently), would love to see a fix implemented for it too with your mod so I can just symlink the files in a single folder to update all 3 games' ASI's at a single time. ♥

@curryking3
Copy link

curryking3 commented Oct 28, 2023

Tried to play this on 16:10, looking really good, I think the aspect ratio is preserved with the full 16:10 window, the only thing that is stretched out to 16:10 are the UI elements and HUD and subtitles and credits for example, overall it looks excellent, but would be cool if there were 3 different possible options:

  1. preserved 16:9 viewport (no fix needed for UI elements)
  2. 16:10 viewport with 16:9 scaled HUD
  3. 16:10 viewport with 16:10 stretched HUD (what is currently present)

Lastly probably is more of a feature request, but a custom internal resolution setting would be amazing, high end PCs can probably run this at 8K internal resolution or higher 😎

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

No branches or pull requests

5 participants