-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
WIP: GS: Add Pre-Round Sprite hack #6553
base: master
Are you sure you want to change the base?
Conversation
8d11f2d
to
cf5a3d6
Compare
Works perfectly! Thank you for doing this :) |
Thanks :) there's still a lot to do, I want to make it less hacky and based more on hardware test results (confirming a theory on manhunt led to this) but we need more data, but it should provide some interesting testing. I should Also note if anybody is testing manhunt in hardware mode, you also have to enable auto flush. |
I don't know whether it's worth to report bugs at his point but here it is anyway. Maybe it will help somehow... |
Thanks, I mean it's worth checking those when we get more data to check, so it's appreciated. But as said in it's current state it WILL break stuff, I kind of expect it. |
cf5a3d6
to
7d7b6ec
Compare
I haven't checked the other dumps, but I think I'm on to something here, using some information gathered on how STQ works, and used some theory on UV's (I believe we've come across this problem before) Dark Cloud 2 seems to work in this, along with Manhunt and Beyond Good and Evil, so it feels a bit more promising. Edit: Wild Arms 3 no longer needs the Wild Arms hack, so that's nice? At least at the time of writing this. No doubt something else broke :D |
2a96497
to
30138f8
Compare
ALL GS dumps posted above are fine now. |
Thanks, but now Manhunt isn't fixed again xD Edit: Also noticed God of War doesn't like this also. |
30138f8
to
34f9bc6
Compare
Note: This PR is still NOT right, but it's working okay. Here's the current state of things that get fixed by this PR (unless it says "Not currently fixed" after it) 3-Nen B-Gumi Kinpachiu Sensei - sprite alignment for scene + character sprites (unknown if UV or ST, probably UV) Affected but not currently working: |
263a401
to
0029168
Compare
|
To a point, yes. They kinda do similar jobs, just at different points in the render. |
Dynasty Warriors 6 also suffers from the same lines that 5 do, so it's worth checking too. |
do you have Dynasty Warriors 6 to check? I do not, unfortunately |
I do. What do you a want me to test? Here's a gsdump showing the bug: |
Over the years, I gathered some info about DW 5 and 6. What I noticed is that, if you put half pixel offset to vertex, it would make the lines thinner, making it less obvious (but they are still there). But if you put round sprite to half (to fix the vertical lines when upscaling), it would make the lines wider again, which is curious. |
That is interesting, the HPO I understand as that corrects the upscaling issue, but of course it's wrong to begin with (As can be seen in native/software), so it kind of compounds the issue. So hopefully once I've finished with this PR, that it will correct that part of it, so only HPO is required for upscaling (as above)., maybe round too, i dunno.. |
86acd2c
to
eb46ab6
Compare
c44bbc6
to
da4b3b0
Compare
I mean in the new version which is currently building, it seems to be acting differently to you and link |
Sorry, I'm a dingus, apparently I was overriding the global settings with a per-game upscale and somehow didn't notice. Yes HW mode is fixed in native with that. |
there can be many reasons for this, the best thing to do would be to open an issue (if there isn't one) and post a GS dump with your issue, then we can look at it. Thanks |
da4b3b0
to
401e692
Compare
401e692
to
2d0dad6
Compare
2d0dad6
to
21eb889
Compare
21eb889
to
dddd08d
Compare
dddd08d
to
341df12
Compare
Fixes Genji's distorted image in the software renderer #4401. The hardware renderers show improvement, but still not quite right. |
341df12
to
dbedafc
Compare
Fixes I-Ninja black line in geometry. |
dbedafc
to
381a0b3
Compare
This attempts to preproduce hardware behaviour, but falls down in a bunch of cases, hence the hack.
381a0b3
to
97eaaa3
Compare
Description of Changes
This attempts to preproduce hardware behaviour of texel coordinates (0.5 centre), but falls down in a bunch of cases, hence the hack. But does fix a couple of long standing bugs.
Rationale behind Changes
In some cases, the PS2 will round down coordinates which PCSX2 does not, causing some issues with rendering. A prime example of this is Manhunt which draws a square over an area to hide lights, but uses a single pixel low alpha square in order to do so, and PCSX2 current misses this pixel by 1. This PR makes it pick the right pixel.
Suggested Testing Steps
Try games that have graphical errors (in software) and see if this resolves them. I've already checked Mercinaries, and Gran Turismo 4 also gets broken by this.
Fixes #257
Resolves the water effect mentioned here #1986
This PR kind of fixes the UI on Dark Cloud but causes some misaligned backgrounds.
Showcase of the two main fixes of this PR:
Beyond Good and Evil:
Manhunt: