> -----Original Message-----
> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of
> Pedro Arthur
> Sent: Friday, July 26, 2019 10:36 PM
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 1/2] dnn: add layer pad which is equivalent
> to tf.pad
>
> Hi,
>
> Em seg, 1 de jul de 2019 às 05:10, Guo, Yejun <yejun....@intel.com>
> escreveu:
> >
> > + // handle the first dimension
> > + before_paddings = params->paddings[0][0];
> > + after_paddings = params->paddings[0][1];
> > + for (int n = 0; n < before_paddings; n++) {
> > + float *dst = output + n * new_hwc_stride;
> > + if (params->mode == LPMP_CONSTANT) {
> > + for (int i = 0; i < new_hwc_stride; i++) {
> > + *dst = params->constant_values;
> I suppose it is missing a pointer increase in dst ptr?
> I found this same pattern in all loops below.
thanks, nice catch.
I just checked my unit test for LPMP_CONSTANT and found it happened to test the
fourth dimension which does not need the pointer increasing.
will send out a new patch together with unit test improved.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
