koying / SPMC

fork of xbmc/kodi
Other
633 stars 256 forks source link

Android TV recommendations poster art is low rez, zoomed and cropped #434

Closed classicjazz closed 7 years ago

classicjazz commented 8 years ago

On my Shield Android TV, the recommended content included movies and TV show poster art. These are very low resolution including suffering from the "jaggies". By comparison, other recommendations such as TED and Live Channels is higher resolution (no jaggies).

There also appears to be a zoom function applied, which results in titles within the poster art being cropped, particularly at the top and bottom.

I am running at 4K resolution in Shield TV though I believe Android TV's GUI is only 1080px.

koying commented 8 years ago

How do they compare vs. the ones inside SPMC? they are the same...

classicjazz commented 8 years ago

How do they compare vs. the ones inside SPMC?

I am running the GUI in 4K.

Poster art source images are 3000x2000px.

Initial (thumbnail) cache should be 2160px with the advancedsettings.xml mod but there appears to be a bug because actual cached size is 720x1080px. Cached images are resized with lanczos.

`2160

2160 lanczos` Even though poster art is smaller than desired, it still looks very good with the FTV skin in SPMC. > they are the same... I assume you are saying that they are pulled from the same thumbnail cache. They are obviously downscaled and cropped when rendered by Leanback Launcher. What scaling algorithm are you using to do this?
koying commented 8 years ago

Plain Android standard resize. I have not paid attention on how it's actually done behind the scenes... Mind posting some screenshots to make the issue more obvious?

classicjazz commented 7 years ago

It's harder to see from these images but SPMC thumbs are low resolution. You can see how blocky the text is and how it is off screen. And one thumb didn't render properly at all. Compare the image quality from the TED broadcast screen grabs.

img_5689 img_5690 img_5691

classicjazz commented 7 years ago

Appears to be fixed in 16.5RC