Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-03-01 Thread Ganapathy Raman Kasi
: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Philip Langdale Sent: Tuesday, February 28, 2017 5:54 AM To: Miroslav Slugeň Cc: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr On Tue, 28 Feb 2017 11:54

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Timo Rothenpieler
> Patch looks good to me too. Timo, I can push it if you don't mind not > testing it yourself. I have some stilistic nits, otherwise it's ok. It leaves the fall through comment dangling, even though there is no fall through anymore. And it could use fallthrough to in the new location of RC_(2_PASS

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Philip Langdale
On Tue, 28 Feb 2017 11:54:32 +0100 Miroslav Slugeň wrote: > Dne 28.2.2017 v 10:44 Timo Rothenpieler napsal(a): > > Am 28.02.2017 um 01:24 schrieb Ganapathy Raman Kasi: > >> Can someone please help review patch. Thanks. > > I have seen an marked it, but I don't have time to review and test > >

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Miroslav Slugeň
Dne 28.2.2017 v 10:44 Timo Rothenpieler napsal(a): Am 28.02.2017 um 01:24 schrieb Ganapathy Raman Kasi: Can someone please help review patch. Thanks. I have seen an marked it, but I don't have time to review and test stuff right now. Will get to it eventually. __

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-28 Thread Timo Rothenpieler
Am 28.02.2017 um 01:24 schrieb Ganapathy Raman Kasi: > Can someone please help review patch. Thanks. I have seen an marked it, but I don't have time to review and test stuff right now. Will get to it eventually. ___ ffmpeg-devel mailing list ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-27 Thread Ganapathy Raman Kasi
Can someone please help review patch. Thanks. Ganapathy -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Ganapathy Raman Kasi Sent: Thursday, February 23, 2017 5:42 PM To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PATC

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-23 Thread Ganapathy Raman Kasi
: Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr Adding on to the previous mail, To test this patch, please try : i) ffmpeg -y -i $INPUT -vcodec h264_nvenc -preset medium $OUT_medium.mp4 ii) ffmpeg -y -i $INPUT -vcodec h264_nvenc -preset slow $OUT_slow.mp4 Without patch

Re: [FFmpeg-devel] [PATCH] Remove qmin and qmax constraints for nvenc vbr

2017-02-23 Thread Ganapathy Raman Kasi
Adding on to the previous mail, To test this patch, please try : i) ffmpeg -y -i $INPUT -vcodec h264_nvenc -preset medium $OUT_medium.mp4 ii) ffmpeg -y -i $INPUT -vcodec h264_nvenc -preset medium $OUT_slow.mp4 Without patch : $OUT_medium and $OUT_slow will be binary identical since both use NV_E