Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 14:24, Hendrik Leppkes wrote: If its entirely untested (due to being unsupported), it should not be made available, and instead be added once it is supported and has been confirmed to be working properly. There is also no reason to add it if we know it doesn't work, it would just be m

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 1:30 PM, Sven C. Dack wrote: > On 19/10/16 12:00, Hendrik Leppkes wrote: >> >> On Wed, Oct 19, 2016 at 12:20 PM, Sven C. Dack >> wrote: >>> >>> On 19/10/16 11:01, Yogender Kumar Gupta wrote: >> >> I've send in 3 patches regarding Nvidia stuff (1. check for temp aq,

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
>> Temporal AQ is implemented in CUDA, not in silicon. So far does the API (Video SDK 7.0.1) accept the flag without giving any errors. The output however remains identical currently (tested on a GTX 960). You'll have to ask Nvidia why they haven't implemented it yet (i'ts probably too soon). I am

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 12:00, Hendrik Leppkes wrote: On Wed, Oct 19, 2016 at 12:20 PM, Sven C. Dack wrote: On 19/10/16 11:01, Yogender Kumar Gupta wrote: I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but ther

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 12:20 PM, Sven C. Dack wrote: > On 19/10/16 11:01, Yogender Kumar Gupta wrote: I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. >> >> option for temp aq in hevc, 3. limitation work-around for scale npp), but >> there seems to be no maintaine

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Timo Rothenpieler
I'm not exactly maintaining libnpp, I think it was merged from libav, but the patch looks fine regardless. No idea why I never got these mails, must have mass-marked them as read and then deleted all unread mails. Will apply the old patches as this one does not show up on patchwork. __

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 11:01, Yogender Kumar Gupta wrote: I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but there seems to be no maintainer present at the moment to pick them up... Tempoal AQ HEVC not supported

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
>> This patch LGTM, minus the one spurious space at the end of the if. What is the procedure to submit, do I do a git push Thanks, Yogender ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
>> I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but there seems to be no maintainer present at the moment to pick them up... Tempoal AQ HEVC not supported yet. Thanks, Yogender ___

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 10:22, Timo Rothenpieler wrote: Am 19.10.2016 um 11:15 schrieb Sven C. Dack: On 19/10/16 09:34, Yogender Kumar Gupta wrote: Added capability check for Temporal AQ Thanks, Yogender I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Timo Rothenpieler
Am 19.10.2016 um 11:15 schrieb Sven C. Dack: > On 19/10/16 09:34, Yogender Kumar Gupta wrote: >> Added capability check for Temporal AQ >> >> Thanks, >> Yogender > I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. > option for temp aq in hevc, 3. limitation work-around for sca

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 09:34, Yogender Kumar Gupta wrote: Added capability check for Temporal AQ Thanks, Yogender I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but there seems to be no maintainer present at th

[FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
Added capability check for Temporal AQ Thanks, Yogender 0001-nvenc-Add-capability-check-for-Temporal-AQ.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel