ppsspp icon indicating copy to clipboard operation
ppsspp copied to clipboard

GachiTora! Abarenbou Kyoushi in High School graphical issue

Open Panderner opened this issue 4 years ago • 19 comments

Reported on https://github.com/hrydgard/ppsspp/issues/14136#issuecomment-778784080

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

Panderner avatar Feb 15 '21 09:02 Panderner

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]

unknownbrackets avatar Feb 20 '21 05:02 unknownbrackets

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]

unknownbrackets avatar Jan 09 '22 03:01 unknownbrackets

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 https://github.com/hrydgard/ppsspp/issues/15016#issuecomment-949245823

ghost avatar Jan 09 '22 04:01 ghost

How about this now?

ghost avatar Aug 01 '22 09:08 ghost

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

Might be fixed via https://github.com/hrydgard/ppsspp/pull/16043

ghost avatar Sep 20 '22 09:09 ghost

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

hrydgard avatar Sep 20 '22 11:09 hrydgard

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 avatar Sep 20 '22 13:09 unknownbrackets

Samsung m20 Exynos 7904 Mali-G71 Screenshot_20221003-161745_PPSSPP

IrfanH495 avatar Oct 03 '22 09:10 IrfanH495

Samsung m20 Exynos 7904 Mali-G71 Screenshot_20221003-161745_PPSSPP

Add the game id to disableaccuratedepth

ghost avatar Oct 03 '22 09:10 ghost

@Gamemulatorer ok thanks Screenshot_20221003-183017_PPSSPP

IrfanH495 avatar Oct 03 '22 13:10 IrfanH495

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]

unknownbrackets avatar Oct 03 '22 14:10 unknownbrackets

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

I will try the latest version.

IrfanH495 avatar Oct 03 '22 14:10 IrfanH495

v1.13.2-1366-g360eab321 Screenshot_20221003-212600_PPSSPP

IrfanH495 avatar Oct 03 '22 14:10 IrfanH495

Mali GPU is very problematic gpu in terms of emulator 🤔

ghost avatar Oct 03 '22 15:10 ghost

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

ghost avatar Oct 04 '22 17:10 ghost

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.

hrydgard avatar Oct 04 '22 18:10 hrydgard

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

IrfanH495 avatar Oct 06 '22 09:10 IrfanH495

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: https://github.com/hrydgard/ppsspp/compare/master...unknownbrackets:simulate-clip-range

-[Unknown]

unknownbrackets avatar Oct 07 '22 01:10 unknownbrackets

This can be close now issue is finally fix.

Gamemulatorer avatar Oct 07 '22 17:10 Gamemulatorer

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

IrfanH495 avatar Mar 18 '23 04:03 IrfanH495

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

IrfanH495 avatar Mar 18 '23 05:03 IrfanH495