Jump to content

Wrong vertical resolution


Recommended Posts

Posted

Hi,

I've noticed that the vertical resolution displayed on the video details page is incorrect when it's below 480p: even if the actual height is 294 pixels, Emby displays it as 480p.

Isn't it possible to display the actual height?

image.png.60d4916ae6f1951168f84fbe1a48fa77.png   In this instance, the actual resolution of this video is 712x296, therefore lower than 480p

Thanks in advance.

Regards.

GrimReaper
Posted (edited)

Been a long time since this was mentioned, lemme see if I still remember those values correctly:

700-1270-1900-3800

@ebrdoes that sound about right? 

Anyway, IIRC Emby uses Width/horizontal value, not Height/vertical value for resolution, with delimiters being those stated above, i.e.:

SD<700px, 480p<1270px, 720p<1900px, 1080p<3800px, 4K>3800px width. 

In your example, it's 712px width, hence 480p; if it were few px less (<700), it'd be marked SD res.

Edited by GrimReaper
Posted

Hi, what is the resolution?

Happy2Play
Posted (edited)
7 minutes ago, Luke said:

Hi, what is the resolution?

 

1 hour ago, Damsdu77 said:

 In this instance, the actual resolution of this video is 712x296, therefore lower than 480p

But similar topics have come up many times over the years.

Edited by Happy2Play
Happy2Play
Posted (edited)

From Old topics so might have slightly changed over time but basically the same as what Grim posted above.

  

On 10/3/2017 at 1:29 PM, speechles said:

https://github.com/MediaBrowser/Emby/blob/master/MediaBrowser.Controller/Entities/Video.cs#L783-L808

            if (videoStream != null)
            {
                if (videoStream.Width.HasValue)
                {
                    if (videoStream.Width.Value >= 3800)
                    {
                        terms.Add("4K");
                    }
                    else if (videoStream.Width.Value >= 1900)
                    {
                        terms.Add("1080P");
                    }
                    else if (videoStream.Width.Value >= 1270)
                    {
                        terms.Add("720P");
                    }
                    else if (videoStream.Width.Value >= 700)
                    {
                        terms.Add("480P");
                    }
                    else
                    {
                        terms.Add("SD");
                    }
                }
            }

Understand now? Perhaps this part of the code should be refactored to make use of videoStream.Height.Value in these evaluations too?

 

Edited by Happy2Play
Posted
51 minutes ago, GrimReaper said:

Been a long time since this was mentioned, lemme see if I still remember those values correctly:

700-1270-1900-3800

@ebrdoes that sound about right? 

Anyway, IIRC Emby uses Width/horizontal value, not Height/vertical value for resolution, with delimiters being those stated above, i.e.:

SD<700px, 480p<1270px, 720p<1900px, 1080p<3800px, 4K>3800px width. 

In your example, it's 712px width, hence 480p; if it were few px less (<700), it'd be marked SD res.

If that's indeed the logic that was applied in Emby, then I question its basis, because generally when we talk about 1080p or 720p, we're referring to the number of vertical lines, not the number of horizontal pixels. Why would it be different for 480p? But perhaps I misunderstood your argument. 🤔

Sources: https://en.wikipedia.org/wiki/480phttps://en.wikipedia.org/wiki/720phttps://en.wikipedia.org/wiki/1080p

PS: Sorry for posting in the wrong section
PPS: Version in question: not Windows but Linux (on a Raspberry Pi) 😉

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...