Dudemanguy
commented
8 months ago @kasper93 probably won't like this but maybe there should be an option for when the crop gets applied.
Open vslmc opened 8 months ago
Dudemanguy
commented
8 months ago @kasper93 probably won't like this but maybe there should be an option for when the crop gets applied.
kasper93
commented
8 months ago 720x576 SAR > 702x570 Crop > 736x570
Can you explain how you arrived at this? Post the log and retest with --no-config.
vslmc
commented
8 months ago 720x576 SAR > 702x570 Crop > 736x570
Can you explain how you arrived at this? Post the log and retest with
--no-config.
Testing it , I can say there is no problem with no-config. Also after some investigation I found the problem is this line in my config file:
vf=format=fmt=nv24:primaries=dci-p3:convert=yes
Removing this line results in the correct output of 748x570
(I will add the log anyway) log.zip
Important Information
Provide following Information:
Reproduction steps
Add mkv container crop values on e.g. 4:3 PAR videos (Crop = 10,2,8,4)
Expected behavior
Cropping after aspect ratio correction is applied
720x576 SAR > 768x576 PAR > 750x570 Crop
Actual behavior
720x576 SAR > 702x570 Crop > 736x570
Addional mpv reports a crop of X=10, Y=2 instead of 10,2,8,4 (but this is only a ui problem)
Sample files
Original
Bug
Correct
Sample Video video.zip