Nevcairiel / LAVFilters

LAV Filters - Open-Source DirectShow Media Splitter and Decoders
GNU General Public License v2.0
7.19k stars 786 forks source link

Consider supporting logical cropping via Matroska header #556

Open Marat-Tanalin opened 10 months ago

Marat-Tanalin commented 10 months ago

Matroska (MKV) container format supports a header that specifies amount of pixels that should be cropped-out. This would be useful for setting proper cropping for videos that contain black bars hard-encoded together with true video data, with no need for re-encoding the video that is both time-consuming and leading to extra quality-loss.

In MKVToolNix (official Matroska muxing tool), the header is specified via “Properties” → “Video properties” → “Cropping” in properties panel of the video track.

View MKVToolNix screenshot ![](https://user-images.githubusercontent.com/1833563/261628481-42f86cf5-3b91-4fa9-bc34-163802d80629.png)

Previously requested this for MPC-HC player, its developer said the following:

Must be done in LAV Filters.
LAV Filters must implement it.

Not sure whether “Must” in both replies is supposed to mean that the feature is already implemented or that MPC-HC developer believes the feature is better suitable for implementing (in future) on LAV-Filters side. Probably the latter. Thanks.