[RESOLVED] CLIP + cropping
Re: [RESOLVED] CLIP + cropping
That's great feedback, renols, thank you. I will release the fix now.
Re: [RESOLVED] CLIP + cropping
Here are
https://dropmefiles.com/IRyST
Coded size is 1280x544, I don't get why. Video is 1280x536. Sar is not set (default setting for hd x264 encodes).
Re: [RESOLVED] CLIP + cropping
Thank you for the stream sample. The .264 elementary stream declares a coded size of 544 and a display area of 536, so it is being reported correctly. 544 is mod 16 and 536 is not. The coded size must be an integral number of macroblocks. One thing I can do is to populate the display size in the Info dialog. It's absence could be a bug, I'll check.
Re: CLIP + cropping
I agree 1088 is autocrop to 1080 but once you set a crop value it will be based on 1088.
BTW something else is more frustating:
- on a 1088 steam with coded display area at 1080
- don't set any crop value
- just set both NVresize's values ("rh" & "rv")
Check the result with editor on the bottom of the frame.
The resize is done with the full frame size at 1088 and not the autocropped at 1080 !!! Causing frame distorsion.
Is this process normal ?
[Edit] using build 168
Re: [RESOLVED] CLIP + cropping
No, that is not normal. I will sort this out once and for all. Thanks for pointing it out.
Re: [RESOLVED] CLIP + cropping
OK, I'm going to bite the bullet and make cropping relative to the display size. The coded size is just a complication with no user relevance. If anyone objects speak now as I am about to light the afterburner.
Re: [RESOLVED] CLIP + cropping
If your solution could satisfy most users, your choice make sense.
Re: [RESOLVED] CLIP + cropping
I have DGIndexNV cropping converted to work relative to the display size. It wasn't as hard as I expected. Tomorrow I will convert DGDecodeNV and give y'all a test version.
Re: [RESOLVED] CLIP + cropping
Here is a 64-bit test version of DGIndexNV/DGDecodeNV. Everything is relative to the display area. So if you have a coded 1088 with display area 1080, you do not need a fixed bottom crop of 8. I tested it quite a bit and it seems to be fine, but I will await your feedback before I revise the documents and do a slipstream.
http://rationalqm.us/misc/DGDecNV_taz742.zip
http://rationalqm.us/misc/DGDecNV_taz742.zip
Re: [RESOLVED] CLIP + cropping
Thank you I'll test it tomorrow but before I've a question: is it problematic to use your dgdecode 64bit version with a 32bit version of avisynth+ ?
Re: [RESOLVED] CLIP + cropping
Yes, it is problematic. I have added the 32-bit binaries to the ZIP file.
Re: [RESOLVED] CLIP + cropping
Your fixed version work like a charm for me.
Tested on a sample coded 1088 with display area 1080:
- Cropping only -> OK
- Resize only-> OK
- Cropping with Resize -> OK
Tested on a sample coded 1088 with display area 1080:
- Cropping only -> OK
- Resize only-> OK
- Cropping with Resize -> OK
Re: [RESOLVED] CLIP + cropping
Great news! Thank you for your testing, and thank you again for the gentle push to make this change. If you get a chance, please also test Save BMP and Copy BMP in DGIndexNV to see if they properly honor the display area and cropping. Should be OK but it never hurts to have a second pair of eyes.
Re: [RESOLVED] CLIP + cropping
I confirm that [Save BMP] and [Copy BMP] in DGIndexNV properly honor the display area and cropping.
Re: [RESOLVED] CLIP + cropping
Great. Thank you, taz742. I will make a slipstream today.