RPCS3 / rpcs3

PlayStation 3 emulator and debugger
https://rpcs3.net/
GNU General Public License v2.0
15.66k stars 1.93k forks source link

High-Resolution Game Background for loading #16285

Closed elad335 closed 3 weeks ago

elad335 commented 3 weeks ago

Some games are missing PIC1.PNG (the previous default) and instead were resorting to ICON0.PNG which is a low-resolution image. Do something else entirely for image picking - pick the image path between ICON0.PNG, PIC0.PNG, PIC1.PNG, PIC2.PNG and PIC3.PNG that exists and it is the largest in disk size. (assume it is the highest resolution one)

Before: image

After: image

bigol83 commented 3 weeks ago

Gran Turismo 6 still takes icon0 png while it should take PIC0.

Icon0 is 12 kb Pic0 is 20 kb on disk

Immagine 2024-11-04 120229

EDIT I see you have a PR to fix this

Megamouse commented 3 weeks ago

This completely ignores the existing logic of prefering the update path. Also, please stop merging shit like this without any review or discussion. This is not your personal project

elad335 commented 3 weeks ago

This completely ignores the existing logic of prefering the update path. Also, please stop merging shit like this without any review or discussion. This is not your personal project

Fixed in #16286

Kazeira commented 3 weeks ago

I don't think it's necessary to check the file sizes, the expected resolution are PIC1=PIC3>PIC0>PIC2>ICON0. By the way, the menus don't show the correct icons for Disc games + HDD data. On real hardware every disc game still use the files (ICON/SND0/PIC...) from the disc, the files in the hard drive are only used inside the 'Game Data Utility' folder.

So in the menu, Disc files should have priority over HDD files for Disc games. And checking the background loading should be more like Disc game: PIC1/PIC3(Disc) > PIC1/PIC3(HDD) > PIC0(Disc) > PIC0(HDD) > PIC2(Disc) > PIC2(HDD) > ICON0(Disc) HDD game: PIC1/PIC3 > PIC0 > PIC2 > ICON0 (all in the HDD of course)

I also want to add that using the regional files depending of the console language selected could be great (ICON0_02.PNG... for french for example), and use the default files (ICON0.PNG...) only when missing, like a real PS3. But yeah I know, combined with the background loading, it makes a lot of checks.

Megamouse commented 3 weeks ago

This is not the PS3 menu. This is the RPCS3 interface. We are not trying to do a 1:1 copy of the PS3 XMB

Megamouse commented 3 weeks ago

Since the console language is not a UI setting, we don't really care for the icon localization at the moment. It's something we'd have to integrate properly first.

Megamouse commented 3 weeks ago

We could probably add it for the background image already though.

Kazeira commented 3 weeks ago

Of course I understand, my point about the languages was just a bonus, these kinds of things are low priority since having a working emulator is always better than a fancy UI. I only wanted to point out that:

elad335 commented 3 weeks ago

Even if resolution is the same, bigger file size also indicates more detail or vibrant colors usually, so from RPCS3 perspective it is a pretty good measure for image quality that even the developers may not have thought through. (unless of course, they did some odd things with image compression ratio lol)

If you know any exceptions keep us posted.