hrydgard / ppsspp

A PSP emulator for Android, Windows, Mac and Linux, written in C++. Want to contribute? Join us on Discord at https://discord.gg/5NJB6dD or just send pull requests / issues. For discussion use the forums at forums.ppsspp.org.
https://www.ppsspp.org
Other
11.18k stars 2.17k forks source link

Texture scaling breaks some textures from recent color conversion changes. #7364

Closed LunaMoo closed 9 years ago

LunaMoo commented 9 years ago

The issue I got while testing the hack in #6192 was actually caused by some of the recent changes:]. It actually causes alot of broken stuff in some games, like for example Parasite Eve 3 which I was also testing against the hack: ules01513_00001

Happens with texture scaling with both backends for texture type 5551(the one from GE2 transitioning screen which I posted in the bloom issue): ge2 texture

However that weirdness in PE3 happens only in OGL, shadows are: pe3

and snowflakes: pe3-1

Edited the title, since the clut 4 / 4444 texture type issue actually happened in v0.9.9.1-1580-gf1b57da as well, while the texture type 5551 issue is just from v0.9.9.1-1582-g08eaa6e which was supposed to be a fix for it?

Anyway to sum it up since it might be hard to notice between screenshots:

Also just to mention this http://forums.ppsspp.org/showthread.php?tid=14430 is same issue as with PE3, same texture type.

LunaMoo commented 9 years ago

All the problematic commits were reverted, so closing:3.

hrydgard commented 9 years ago

Will do it again later, more carefully :) Then we will be in a good position to speed these up, although I don't think they're usually big bottlenecks anyway..

unknownbrackets commented 9 years ago

Well, optimizations can matter in games with a lot of texture turnover (changes) or for load times in some cases.

-[Unknown]

hrydgard commented 9 years ago

Yeah it will help a bit and probably noticeably in some cases, just meant it's not likely to revolutionize speed :)