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

GS/HW: Improve Round Sprite upscaling fix to cause less problems [TEST] #10278

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

refractionpcsx2
Copy link
Member

Description of Changes

Attempt to reduce the problems associated with Round Sprite (Especially Full Round Sprite)

Rationale behind Changes

It was pretty awful and caused a whole bunch of problems, this tries to mitigate the worst ones.

Suggested Testing Steps

Test any game which uses round sprite, or needed it but it caused problems.

@bigol83
Copy link

bigol83 commented Nov 15, 2023

Couple of games tested

This greatly improves Dragon Quest VIII menu highlight

PR

Dragon Quest VIII - Journey of the Cursed King_SLES-53974_20231115202801

Master

Dragon Quest VIII - Journey of the Cursed King_SLES-53974_20231115203056

But it makes text in Final Fantasy X slightly worse

PR

Final Fantasy X_SCES-50493_20231115210506

Master

Final Fantasy X_SCES-50493_20231115210430

Dragon Quest VIII uses Half Round Sprite to fix text
Final Fantasy X uses Full Round Sprite to fix text

@refractionpcsx2
Copy link
Member Author

yeah, there's not a lot in that, it looks wrong to begin with :P

@cyanea-bt
Copy link
Contributor

cyanea-bt commented Nov 19, 2023

Hi, here's some comparisons using Full Round Sprite on MGS2 and MGS3. First MGS2:
Text artifacts on some UI elements look about the same on both versions (vertical lines next to some letters):

old:
life_Master
new:
life_newVersion

old:
timer_Master
new:
timer_newVersion

But one big improvement on this new version - it seems like it doesn't break DoF like the old version used to do. Awesome!
old:
raiden_Master
new:
raiden_newVersion


Now for MGS3, which doesn't use roundSprite at the moment but imo could benefit from it. For example this upscaling artifact on the right and bottom sides of videos that play on the codec screen:

without roundSprite:
codec_video_noRoundSprite
old roundSprite:
codec_video_Master
new roundSprite (looks the same as old version here):
codec_video_newVersion

But sadly in MGS3 this new version still breaks DoF, just like the old version did. Maybe it doesn't play nicely with the Renderfix (#8991) that MGS3 needs for it's DoF effect? Or idk what else could be the reason for it working in MGS2 but not in MGS3. I added the roundSprite fix to MGS3 via its entry in GameIndex.yaml to ensure that all other fixes it needs stay intact. Just in case, this is what the modified entry looks like:

SLUS-21359:
  name: "Metal Gear Solid 3 - Subsistence [Disc 1 of 3]"
  region: "NTSC-U"
  compat: 5
  gameFixes:
    - BlitInternalFPSHack # Fixes internal FPS detection.
    - InstantDMAHack # Fixes missing letters in text such as E.
  gsHWFixes:
    recommendedBlendingLevel: 3 # Fixes skin colour and banding.
    autoFlush: 2 # Fixes lens flare.
    halfPixelOffset: 2 # Fixes blurriness.
    roundSprite: 2
    getSkipCount: "GSC_MetalGearSolid3" # Fixes depth of field blur.

old:
dof_Master
new:
dof_newVersion

@refractionpcsx2
Copy link
Member Author

yeah unfortunately this isn't going to revolutionise round sprite, but I hope to reduce the amount of times it screws things up.

Thanks for testing!

@refractionpcsx2
Copy link
Member Author

Actually can you provide a GS dump of the slicey Depth of Field effect? Maybe we can improve that too.

@cyanea-bt
Copy link
Contributor

Sure thing, I'll attach 2 GS dumps from that cutscene. If you need any more or if I did them wrong, just let me know.

MGS3_roundSprite_slices_1.zip
MGS3_roundSprite_slices_2.zip

@refractionpcsx2
Copy link
Member Author

Sure thing, I'll attach 2 GS dumps from that cutscene. If you need any more or if I did them wrong, just let me know.

That's perfect, thanks :)

@bigol83
Copy link

bigol83 commented Nov 19, 2023

This PR also improves post processing being broken in Prince of Persia Sands of Time when using Round Sprite Full to improve post processing misalignment due to upscaling.

Master
Prince of Persia - Le Sabbie del Tempo_SLES-51918_20231119134626

PR
Prince of Persia - The Sands of Time_SLES-51918_20231119195605

Look at the effect around the sun. It's not completely fixed but it's much better. Upscaling is at 3x.

@refractionpcsx2
Copy link
Member Author

yeah that was actually one of my targets. lol

@bigol83
Copy link

bigol83 commented Mar 16, 2024

It breaks Valkyrie Profile while using Round Sprite both Half and Full

Valkyrie Profile 2 - Silmeria_SLES-54647_20240317003508
Valkyrie Profile 2 - Silmeria_SLES-54647_20240317003508.gs.zip

Notice those horizontal dotted lines, that doesn't happen on Master.

I wonder though, what is Round Sprite used for in Valkyrie Profile? When it's off cutscenes look clean.

@refractionpcsx2
Copy link
Member Author

It breaks Valkyrie Profile while using Round Sprite both Half and Full
I wonder though, what is Round Sprite used for in Valkyrie Profile? When it's off cutscenes look clean.

"Fixes area transition vertical lines and lessens red forest vertical lines." apparently.

@bigol83
Copy link

bigol83 commented Mar 16, 2024

It breaks Valkyrie Profile while using Round Sprite both Half and Full
I wonder though, what is Round Sprite used for in Valkyrie Profile? When it's off cutscenes look clean.

"Fixes area transition vertical lines and lessens red forest vertical lines." apparently.

Yeah, i checked ingame and with Round Sprite OFF those lines on area transition are distracting. It's a shame because without it cutscenes look so beautiful.

@refractionpcsx2
Copy link
Member Author

It seems to be the combination of Special Texture, Unscaled Palette and Half Round Sprite

image

If I change to align to native it looks better

image

however the forest squares look a little worse (not that they were okay before)

Also to note, if I do master with half round sprite and special texture but no unscale palette, that looks pretty bad too, so I suspect some coincidental reason it doesn't work properly

Master with no unscale palette
image

@bigol83
Copy link

bigol83 commented Mar 17, 2024

With Align to Native and Round Sprite cutscenes look a lot better, vertical lines area transitions are not visible, but the top and left side have some thin lines (those could be cropped).

@refractionpcsx2
Copy link
Member Author

That's always the problem with upscaling, isn't it :P

@bigol83
Copy link

bigol83 commented Mar 17, 2024

That's always the problem with upscaling, isn't it :P

I also checked two other games that need Round Sprite, DQ VIII and Bully. Dragon Quest VIII menu box orange highlight has improved like i reported back in November, with no visible side effects. Bully doesn't show any difference compared to Master.

@refractionpcsx2
Copy link
Member Author

I resolved the Valkyrie Profile issue for those not on the discord. We were running round sprite on native palette draws, which probably wasn't the best.

@F0bes F0bes marked this pull request as draft July 29, 2024 23:01
@lightningterror lightningterror added this to the Release 2.2 milestone Aug 23, 2024
@lightningterror lightningterror removed this from the Release 2.2 milestone Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants