Kodi Community Forum

To whom it may concern: This is a minor thing.

Not sure where this problem lays, but a movie or tv show that is "4K" 4:3 (1.33:1) shows with an 8K tag in the info screens.

Running under the nightly LibreElec Matrix build LibreELEC-Generic.x86_64-9.80-nightly-20201103-f0d82de.tar.

If this needs to be posted elsewhere, please advise.

Thank you,

(2020-11-04, 20:42)TheRealDeadMan Wrote: [ -> ]Not sure where this problem lays

Normally the video height is the indicator for what is today 2K, 4K or 8K.
4K usually ends with 2160 pixels in height, anything over that is ranked with the next step: 8K.

But, aspect ratio should be taken into consideration, and consequently the width in pixels.
It's not just a cosmetic problem, but one to be solved in the upcoming nightly release(s).

(2020-11-04, 20:42)TheRealDeadMan Wrote: [ -> ]To whom it may concern: This is a minor thing.

If you could create a Github ticket at Kodi's Bug Tracker (see link at the top menu), that'd be great.

Looks like we need to somehow use this -

https://www.wearethefirehouse.com/aspect...heat-sheet

(2020-11-05, 11:50)Hitcher Wrote: [ -> ]Looks like we need to somehow use this -

https://www.wearethefirehouse.com/aspect...heat-sheet

According to that all we need to is change iHeight <= 2160) to iHeight <= 3072) here https://github.com/xbmc/xbmc/blob/master...s.cpp#L575 because max height for 4K is the 4096 x 3072 resolution for 4:3.

@Hitcher Looks like you didn't have that chart at the time of https://github.com/xbmc/xbmc/pull/14742

Max 4K height is 3072 and Min 8K height is 3148 according to those charts so should be safe to change as doesn't appear to be any overlap between 4K and 8K ratio heights.

(2020-11-05, 10:16)Klojum Wrote: [ -> ]If you could create a Github ticket at Kodi's Bug Tracker (see link at the top menu), that'd be great.

Sure thing. Here is the tracker link.

Matrix: 4:3 4K TV/Movies showing 8K tags

Thanks for looking into everyone.

(2020-11-05, 12:38)jjd-uk Wrote: [ -> ]

(2020-11-05, 11:50)Hitcher Wrote: [ -> ]Looks like we need to somehow use this -

https://www.wearethefirehouse.com/aspect...heat-sheet

According to that all we need to is change iHeight <= 2160) to iHeight <= 3072) here https://github.com/xbmc/xbmc/blob/master...s.cpp#L575 because max height for 4K is the 4096 x 3072 resolution for 4:3.

@Hitcher Looks like you didn't have that chart at the time of https://github.com/xbmc/xbmc/pull/14742

Max 4K height is 3072 and Min 8K height is 3148 according to those charts so should be safe to change as doesn't appear to be any overlap between 4K and 8K ratio heights.

As I said in the comments it's all fritsch's work I just made the PR.

This should now be fixed for Matrix.

Thanks.

Thank you all.