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

GachiTora! Abarenbou Kyoushi in High School graphical issue #14146

Open
Panderner opened this issue Feb 15, 2021 · 20 comments
Open

GachiTora! Abarenbou Kyoushi in High School graphical issue #14146

Panderner opened this issue Feb 15, 2021 · 20 comments
Labels
Guardband / Range Culling Involves vertices outside fustrum. Vulkan
Milestone

Comments

@Panderner
Copy link
Contributor

Panderner commented Feb 15, 2021

Reported on #14136 (comment)

This graphical issue for GachiTora is only occurs for Vulkan.

OpenGL:
Screenshot_2021-02-15-17-00-43-429_org ppsspp ppsspp

Vulkan:
Screenshot_2021-02-15-17-00-55-158_org ppsspp ppsspp

GE dump:
NPJH50409_0001.ppdmp.zip

@hrydgard hrydgard modified the milestones: v1.11.2, v1.12.0 Feb 15, 2021
@unknownbrackets
Copy link
Collaborator

The prims between 3311-3322/4961 draw incorrectly from depth clamp, so this is another depth clamp related issue. It's not a depth skew issue, because minz/maxz are the full range.

The vert depth values are pretty crazy, so this is likely something we've got wrong with the clipping depth clamp performs...

-[Unknown]

@hrydgard hrydgard modified the milestones: v1.12.0, Future-Prio Aug 29, 2021
@unknownbrackets unknownbrackets added the Guardband / Range Culling Involves vertices outside fustrum. label Jan 9, 2022
@unknownbrackets
Copy link
Collaborator

I think this has been fixed by the cull/clip changes - it looks fine on Vulkan for me now. Notably, that means it's still an issue on Mali.

Mentioning this for #12058 so it doesn't get lost.

-[Unknown]

@ghost
Copy link

ghost commented Jan 9, 2022

Mali-G52 MC2 Vulkan RN9

DisableAccurateDepth=TRUE
Screenshot_2022-01-09-12-48-56-801_org ppsspp ppsspp

DisableAccurateDepth=FALSE
Screenshot_2022-01-09-12-49-19-189_org ppsspp ppsspp

Similar to #15016 (comment)

@ghost
Copy link

ghost commented Aug 1, 2022

How about this now?

@ghost
Copy link

ghost commented Sep 20, 2022

This is ok on my oppo a3s adreno 506 gpu vulkan.
Screenshot_2022-09-20-17-56-36-62

Might be fixed via #16043

@hrydgard
Copy link
Owner

OK. Let's assume it's working on mali too now then :) If not, let's re-open.

@unknownbrackets
Copy link
Collaborator

unknownbrackets commented Sep 20, 2022

I think this is the neg z thing and not fixed on Mali (no clip/culldistance support), but we have multiple issues for that so I think it's okay to close this one.

-[Unknown]

@unknownbrackets unknownbrackets modified the milestones: Future-Prio, v1.13.0 Sep 20, 2022
@IrfanH495
Copy link

Samsung m20
Exynos 7904
Mali-G71
Screenshot_20221003-161745_PPSSPP

@ghost
Copy link

ghost commented Oct 3, 2022

Samsung m20 Exynos 7904 Mali-G71 Screenshot_20221003-161745_PPSSPP

Add the game id to disableaccuratedepth

@IrfanH495
Copy link

@Gamemulatorer ok thanks
Screenshot_20221003-183017_PPSSPP

@unknownbrackets
Copy link
Collaborator

That's a hack and might result in other issues, because it's not how the PSP renders depth. Does it work to use the new geometry shader support in recent git builds? Although I think this one requires clipping.

-[Unknown]

@IrfanH495
Copy link

version I'm using now
v1.13.2-1346-gbc1975bc9
Screenshot_20221003-211454_PPSSPP

I will try the latest version.

@IrfanH495
Copy link

v1.13.2-1366-g360eab321
Screenshot_20221003-212600_PPSSPP

@ghost
Copy link

ghost commented Oct 3, 2022

Mali GPU is very problematic gpu in terms of emulator 🤔

@ghost
Copy link

ghost commented Oct 4, 2022

Might be good to reopen this @IrfanH495 can still reproduce this on his MALI GPU phone with geo shader enable.

@hrydgard hrydgard reopened this Oct 4, 2022
@hrydgard
Copy link
Owner

hrydgard commented Oct 4, 2022

Although if it requires clipping, it not working yet is expected. It'll have to wait until someone gets around to implementing software clipping in the geo shader or fragment shader.

@IrfanH495
Copy link

ppsspp-v1.13.2-1393-g7dd93f906-android issue has been fixed
Screenshot_20221006-164052_PPSSPP
thanks very much.

@unknownbrackets
Copy link
Collaborator

For the remaining devices, it might be possible to adjust the Z min/max values to clip using standard Z clip, but you'd have to be careful to adjust the output Z to this range and account for it in any depth rounding. I think most modern devices do support shader clip distance.

Alternatively, could discard in the fragment shader, as in:
master...unknownbrackets:simulate-clip-range

-[Unknown]

@unknownbrackets unknownbrackets modified the milestones: v1.13.0, Future-Prio Oct 20, 2022
@IrfanH495
Copy link

sorry, may I share how to use compat.ini on my channel because some people have asked about it.

@IrfanH495
Copy link

sorry, this is the same issue
geometry shader culling ON
Screenshot_20230318-120458_PPSSPP
geometry shader culling OFF
Screenshot_20230318-120446_PPSSPP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Guardband / Range Culling Involves vertices outside fustrum. Vulkan
Projects
None yet
Development

No branches or pull requests

4 participants