Skip to content

Video with 1920x800 resolution (2.40:1 aspect ratio) incorrectly matched as 720p #10

@Bastien42

Description

@Bastien42

Issue Description
Videos with a resolution of 1920x800 (2.40:1 aspect ratio) are incorrectly classified as 720p instead of 1080p.

Expected Behavior
Videos with a width of 1920 pixels should be classified as 1080p, as the horizontal resolution defines the standard (e.g., 1920x800 is 1080p). Height should not be the sole determining factor for resolution classification.

Current Behavior
The plugin classifies 1920x800 videos as 720p, likely because the height is below 1080 pixels, despite the width meeting the 1080p requirement.

Steps to Reproduce

  1. Use a video file with a resolution of 1920x800 (2.40:1 aspect ratio).
  2. Observe that the plugin categorizes it as 720p rather than 1080p.

Suggested Fix
Modify the resolution detection logic to prioritize width for classification. If the width is 1920 pixels or higher, classify the video as 1080p, regardless of height. This approach aligns with industry standards, where horizontal resolution determines the resolution tier (e.g., 1920x800 is still 1080p).

Additional Context

  • Resolution: 1920x800
  • Aspect Ratio: 2.40:1
  • Expected Classification: 1080p
  • Current Classification: 720p

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions