media_management_scripts icon indicating copy to clipboard operation
media_management_scripts copied to clipboard

Vertical video resolution name is wrong

Open raydouglass opened this issue 2 years ago • 0 comments

If a regular resolution video is made in vertical format, it could be classified with the wrong resolution name.

For example, a regular 1920x1080 video is HIGH_DEF, but the vertical equivalent (1080x1920) is parsed as ULTRA_HIGH_DEF.

Caused by https://github.com/raydouglass/media_management_scripts/blob/main/media_management_scripts/support/encoding.py#L68

Maybe this function should consider max(width, height)? Or some other method.

raydouglass avatar Dec 03 '23 03:12 raydouglass