Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider supporting logical cropping via Matroska header #556

Open
Marat-Tanalin opened this issue Aug 18, 2023 · 0 comments
Open

Consider supporting logical cropping via Matroska header #556

Marat-Tanalin opened this issue Aug 18, 2023 · 0 comments

Comments

@Marat-Tanalin
Copy link

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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant