ralfbiedert / openh264-rs

Idiomatic Rust wrappers around OpenH264.
68 stars 35 forks source link

Allow encoding 4K vertical videos #62

Closed balins closed 1 week ago

balins commented 1 week ago

This PR modifies the resolution check, so that 2160x3840 is supported. OpenH264 has a 4K resolution limit, but it can be either horizontal or vertical. I've tested the solution by successfully encoding a video from Pixel 3A with 1080x2220 resolution – previously it would error out since the height (2220) exceeds 2160px.

ralfbiedert commented 1 week ago

Thanks!