[x265] [x265 patch] Adaptive Frame Duplication
Pradeep Ramachandran
pradeep at multicorewareinc.com
Fri Oct 4 11:48:45 CEST 2019
On Thu, Oct 3, 2019 at 12:35 PM Akil <akil at multicorewareinc.com> wrote:
> # HG changeset patch
> # User Akil Ayyappan<akil at multicorewareinc.com>
> # Date 1568370446 -19800
> # Fri Sep 13 15:57:26 2019 +0530
> # Node ID fa0d04affb8a4405ac548841ad12cfa2dab87ef2
> # Parent c4b098f973e6b0ee4aee3bf0d7b54da4e2734d42
> Adaptive Frame duplication
>
> This patch does the following.
> 1. Replaces 2-3 near-identical frames with one frame and sets pic_struct
> based on frame doubling / tripling.
> 2. Add option "--frame-dup" and "--dup-threshold' to enable frame
> duplication and to set threshold for frame similarity (optional).
>
> diff -r c4b098f973e6 -r fa0d04affb8a doc/reST/cli.rst
> --- a/doc/reST/cli.rst Tue Aug 13 10:51:21 2019 +0530
> +++ b/doc/reST/cli.rst Fri Sep 13 15:57:26 2019 +0530
> @@ -501,6 +501,17 @@
> second. The decoder must re-combine the fields in their correct
> orientation for display.
>
> +.. option:: --frame-dup, --no-frame-dup
> +
> + Enable Adaptive Frame duplication. Replaces 2-3 near-identical frames
> with one
> + frame and sets pic_struct based on frame doubling / tripling.
> + Default disabled.
> +
> +.. option:: --dup-threshold <integer>
> +
> + Frame similarity threshold can vary between 1 and 99. This requires
> Adaptive
> + Frame Duplication to be enabled. Default 70.
> +
> .. option:: --seek <integer>
>
> Number of frames to skip at start of input file. Default 0
> diff -r c4b098f973e6 -r fa0d04affb8a source/CMakeLists.txt
> --- a/source/CMakeLists.txt Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/CMakeLists.txt Fri Sep 13 15:57:26 2019 +0530
> @@ -29,7 +29,7 @@
> option(STATIC_LINK_CRT "Statically link C runtime for release builds" OFF)
> mark_as_advanced(FPROFILE_USE FPROFILE_GENERATE NATIVE_BUILD)
> # X265_BUILD must be incremented each time the public API is changed
> -set(X265_BUILD 179)
> +set(X265_BUILD 180)
> configure_file("${PROJECT_SOURCE_DIR}/x265.def.in"
> "${PROJECT_BINARY_DIR}/x265.def")
> configure_file("${PROJECT_SOURCE_DIR}/x265_config.h.in"
> diff -r c4b098f973e6 -r fa0d04affb8a source/common/frame.cpp
> --- a/source/common/frame.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/common/frame.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -57,6 +57,7 @@
> m_addOnPrevChange = NULL;
> m_classifyFrame = false;
> m_fieldNum = 0;
> + m_picStruct = 0;
> }
>
> bool Frame::create(x265_param *param, float* quantOffsets)
> diff -r c4b098f973e6 -r fa0d04affb8a source/common/frame.h
> --- a/source/common/frame.h Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/common/frame.h Fri Sep 13 15:57:26 2019 +0530
> @@ -98,6 +98,7 @@
>
> float* m_quantOffsets; // points to
> quantOffsets in x265_picture
> x265_sei m_userSEI;
> + uint32_t m_picStruct; // picture structure SEI
> message
> x265_dolby_vision_rpu m_rpu;
>
> /* Frame Parallelism - notification between FrameEncoders of
> available motion reference rows */
> diff -r c4b098f973e6 -r fa0d04affb8a source/common/param.cpp
> --- a/source/common/param.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/common/param.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -135,6 +135,7 @@
>
> /* Source specifications */
> param->internalBitDepth = X265_DEPTH;
> + param->sourceBitDepth = 8;
> param->internalCsp = X265_CSP_I420;
> param->levelIdc = 0; //Auto-detect level
> param->uhdBluray = 0;
> @@ -338,6 +339,9 @@
> param->pictureStructure = -1;
> param->bEmitCLL = 1;
>
> + param->bEnableFrameDuplication = 0;
> + param->dupThreshold = 70;
> +
> /* SVT Hevc Encoder specific params */
> param->bEnableSvtHevc = 0;
> param->svtHevcParam = NULL;
> @@ -1294,6 +1298,8 @@
> OPT("fades") p->bEnableFades = atobool(value);
> OPT("field") p->bField = atobool( value );
> OPT("cll") p->bEmitCLL = atobool(value);
> + OPT("frame-dup") p->bEnableFrameDuplication = atobool(value);
> + OPT("dup-threshold") p->dupThreshold = atoi(value);
> OPT("hme") p->bEnableHME = atobool(value);
> OPT("hme-search")
> {
> @@ -1680,6 +1686,8 @@
> "Supported factor for controlling max AU size is from 0.5 to 1");
> CHECK((param->dolbyProfile != 0) && (param->dolbyProfile != 50) &&
> (param->dolbyProfile != 81) && (param->dolbyProfile != 82),
> "Unsupported Dolby Vision profile, only profile 5, profile 8.1
> and profile 8.2 enabled");
> + CHECK(param->dupThreshold < 1 || 99 < param->dupThreshold,
> + "Invalid frame-duplication threshold. Value must be between 1 and
> 99.");
> if (param->dolbyProfile)
> {
> CHECK((param->rc.vbvMaxBitrate <= 0 || param->rc.vbvBufferSize <=
> 0), "Dolby Vision requires VBV settings to enable HRD.\n");
> @@ -1972,6 +1980,9 @@
> s += sprintf(s, " subme=%d", p->subpelRefine);
> s += sprintf(s, " merange=%d", p->searchRange);
> BOOL(p->bEnableTemporalMvp, "temporal-mvp");
> + BOOL(p->bEnableFrameDuplication, "frame-dup");
> + if(p->bEnableFrameDuplication)
> + s += sprintf(s, " dup-threshold=%d", p->dupThreshold);
> BOOL(p->bEnableHME, "hme");
> if (p->bEnableHME)
> s += sprintf(s, " Level 0,1,2=%d,%d,%d", p->hmeSearchMethod[0],
> p->hmeSearchMethod[1], p->hmeSearchMethod[2]);
> @@ -2209,6 +2220,7 @@
> if (src->csvfn) dst->csvfn = strdup(src->csvfn);
> else dst->csvfn = NULL;
> dst->internalBitDepth = src->internalBitDepth;
> + dst->sourceBitDepth = src->sourceBitDepth;
> dst->internalCsp = src->internalCsp;
> dst->fpsNum = src->fpsNum;
> dst->fpsDenom = src->fpsDenom;
> @@ -2263,6 +2275,8 @@
> dst->subpelRefine = src->subpelRefine;
> dst->searchRange = src->searchRange;
> dst->bEnableTemporalMvp = src->bEnableTemporalMvp;
> + dst->bEnableFrameDuplication = src->bEnableFrameDuplication;
> + dst->dupThreshold = src->dupThreshold;
> dst->bEnableHME = src->bEnableHME;
> if (src->bEnableHME)
> {
> diff -r c4b098f973e6 -r fa0d04affb8a source/encoder/api.cpp
> --- a/source/encoder/api.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/encoder/api.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -923,6 +923,7 @@
> pic->userSEI.numPayloads = 0;
> pic->rpu.payloadSize = 0;
> pic->rpu.payload = NULL;
> + pic->picStruct = 0;
>
> if ((param->analysisSave || param->analysisLoad) ||
> (param->bAnalysisType == AVC_INFO))
> {
> diff -r c4b098f973e6 -r fa0d04affb8a source/encoder/encoder.cpp
> --- a/source/encoder/encoder.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/encoder/encoder.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -117,6 +117,8 @@
> m_cR = 1.0;
> for (int i = 0; i < X265_MAX_FRAME_THREADS; i++)
> m_frameEncoder[i] = NULL;
> + for (uint32_t i = 0; i < DUP_BUFFER; i++)
> + m_dupBuffer[i] = NULL;
> MotionEstimate::initScales();
>
> #if ENABLE_HDR10_PLUS
> @@ -160,6 +162,53 @@
> int rows = (p->sourceHeight + p->maxCUSize - 1) >>
> g_log2Size[p->maxCUSize];
> int cols = (p->sourceWidth + p->maxCUSize - 1) >>
> g_log2Size[p->maxCUSize];
>
> + if (m_param->bEnableFrameDuplication)
> + {
> + size_t framesize = 0;
> + int pixelbytes = p->sourceBitDepth > 8 ? 2 : 1;
> + for (int i = 0; i < x265_cli_csps[p->internalCsp].planes; i++)
> + {
> + int stride = (p->sourceWidth >>
> x265_cli_csps[p->internalCsp].width[i]) * pixelbytes;
> + framesize += (stride * (p->sourceHeight >>
> x265_cli_csps[p->internalCsp].height[i]));
> + }
> +
> + //Sets the picture structure and emits it in the picture timing
> SEI message
> + m_param->pictureStructure = 0;
> +
> + for (uint32_t i = 0; i < DUP_BUFFER; i++)
> + {
> + m_dupBuffer[i] =
> (AdaptiveFrameDuplication*)x265_malloc(sizeof(AdaptiveFrameDuplication));
> + m_dupBuffer[i]->dupPic = NULL;
> + m_dupBuffer[i]->dupPic = x265_picture_alloc();
> + x265_picture_init(p, m_dupBuffer[i]->dupPic);
> + m_dupBuffer[i]->dupPlane = NULL;
> + m_dupBuffer[i]->dupPlane = X265_MALLOC(char, framesize);
> + m_dupBuffer[i]->dupPic->planes[0] = m_dupBuffer[i]->dupPlane;
> + m_dupBuffer[i]->bOccupied = false;
> + m_dupBuffer[i]->bDup = false;
> + }
> +
> + if (!(p->sourceBitDepth == 8 && p->internalBitDepth == 8))
> + {
> + int size = p->sourceWidth * p->sourceHeight;
> + int hshift = CHROMA_H_SHIFT(p->internalCsp);
> + int vshift = CHROMA_V_SHIFT(p->internalCsp);
> + int widthC = p->sourceWidth >> hshift;
> + int heightC = p->sourceHeight >> vshift;
> +
> + m_dupPicOne[0] = X265_MALLOC(pixel, size);
> + m_dupPicTwo[0] = X265_MALLOC(pixel, size);
> + if (p->internalCsp != X265_CSP_I400)
> + {
> + for (int k = 1; k < 3; k++)
> + {
> + m_dupPicOne[k] = X265_MALLOC(pixel, widthC * heightC);
> + m_dupPicTwo[k] = X265_MALLOC(pixel, widthC * heightC);
> + }
> + }
> + }
> + }
> +
> // Do not allow WPP if only one row or fewer than 3 columns, it is
> pointless and unstable
> if (rows == 1 || cols < 3)
> {
> @@ -771,6 +820,33 @@
> m_exportedPic = NULL;
> }
>
> + if (m_param->bEnableFrameDuplication)
> + {
> + for (uint32_t i = 0; i < DUP_BUFFER; i++)
> + {
> + X265_FREE(m_dupBuffer[i]->dupPlane);
> + x265_picture_free(m_dupBuffer[i]->dupPic);
> + X265_FREE(m_dupBuffer[i]);
> + }
> +
> + if (!(m_param->sourceBitDepth == 8 && m_param->internalBitDepth
> == 8))
> + {
> + for (int k = 0; k < 3; k++)
> + {
> + if (k == 0)
> + {
> + X265_FREE(m_dupPicOne[k]);
> + X265_FREE(m_dupPicTwo[k]);
> + }
> + else if(k >= 1 && m_param->internalCsp != X265_CSP_I400)
> + {
> + X265_FREE(m_dupPicOne[k]);
> + X265_FREE(m_dupPicTwo[k]);
> + }
> + }
> + }
> + }
> +
> for (int i = 0; i < m_param->frameNumThreads; i++)
> {
> if (m_frameEncoder[i])
> @@ -981,6 +1057,250 @@
> }
> }
>
> +//Find Sum of Squared Difference (SSD) between two pictures
> +uint64_t Encoder::computeSSD(pixel *fenc, pixel *rec, intptr_t stride,
> uint32_t width, uint32_t height, x265_param *param)
> +{
> + uint64_t ssd = 0;
> +
> + if (!param->bEnableFrameDuplication || (width & 3))
> + {
> + if ((width | height) & 3)
> + {
> + /* Slow Path */
> + for (uint32_t y = 0; y < height; y++)
> + {
> + for (uint32_t x = 0; x < width; x++)
> + {
> + int diff = (int)(fenc[x] - rec[x]);
> + ssd += diff * diff;
> + }
> +
> + fenc += stride;
> + rec += stride;
> + }
> +
> + return ssd;
> + }
> + }
> +
> + uint32_t y = 0;
> +
> + /* Consume rows in ever narrower chunks of height */
> + for (int size = BLOCK_64x64; size >= BLOCK_4x4 && y < height; size--)
> + {
> + uint32_t rowHeight = 1 << (size + 2);
> +
> + for (; y + rowHeight <= height; y += rowHeight)
> + {
> + uint32_t y1, x = 0;
> +
> + /* Consume each row using the largest square blocks possible
> */
> + if (size == BLOCK_64x64 && !(stride & 31))
> + for (; x + 64 <= width; x += 64)
> + ssd += primitives.cu[BLOCK_64x64].sse_pp(fenc + x,
> stride, rec + x, stride);
> +
> + if (size >= BLOCK_32x32 && !(stride & 15))
> + for (; x + 32 <= width; x += 32)
> + for (y1 = 0; y1 + 32 <= rowHeight; y1 += 32)
> + ssd += primitives.cu[BLOCK_32x32].sse_pp(fenc +
> y1 * stride + x, stride, rec + y1 * stride + x, stride);
> +
> + if (size >= BLOCK_16x16)
> + for (; x + 16 <= width; x += 16)
> + for (y1 = 0; y1 + 16 <= rowHeight; y1 += 16)
> + ssd += primitives.cu[BLOCK_16x16].sse_pp(fenc +
> y1 * stride + x, stride, rec + y1 * stride + x, stride);
> +
> + if (size >= BLOCK_8x8)
> + for (; x + 8 <= width; x += 8)
> + for (y1 = 0; y1 + 8 <= rowHeight; y1 += 8)
> + ssd += primitives.cu[BLOCK_8x8].sse_pp(fenc + y1
> * stride + x, stride, rec + y1 * stride + x, stride);
> +
> + for (; x + 4 <= width; x += 4)
> + for (y1 = 0; y1 + 4 <= rowHeight; y1 += 4)
> + ssd += primitives.cu[BLOCK_4x4].sse_pp(fenc + y1 *
> stride + x, stride, rec + y1 * stride + x, stride);
> +
> + fenc += stride * rowHeight;
> + rec += stride * rowHeight;
> + }
> + }
> +
> + /* Handle last few rows of frames for videos
> + with height not divisble by 4 */
> + uint32_t h = height % y;
> + if (param->bEnableFrameDuplication && h)
> + {
> + for (uint32_t i = 0; i < h; i++)
> + {
> + for (uint32_t j = 0; j < width; j++)
> + {
> + int diff = (int)(fenc[j] - rec[j]);
> + ssd += diff * diff;
> + }
> +
> + fenc += stride;
> + rec += stride;
> + }
> + }
> +
> + return ssd;
> +}
> +
> +//Compute the PSNR weightage between two pictures
> +double Encoder::ComputePSNR(x265_picture *firstPic, x265_picture *secPic,
> x265_param *param)
> +{
> + uint64_t ssdY = 0, ssdU = 0, ssdV = 0;
> + intptr_t strideL, strideC;
> + uint32_t widthL, heightL, widthC, heightC;
> + double psnrY = 0, psnrU = 0, psnrV = 0, psnrWeight = 0;
> + int width = firstPic->width;
> + int height = firstPic->height;
> + int hshift = CHROMA_H_SHIFT(firstPic->colorSpace);
> + int vshift = CHROMA_V_SHIFT(firstPic->colorSpace);
> + pixel *yFirstPic = NULL, *ySecPic = NULL;
> + pixel *uFirstPic = NULL, *uSecPic = NULL;
> + pixel *vFirstPic = NULL, *vSecPic = NULL;
> +
> + strideL = widthL = width;
> + heightL = height;
> +
> + strideC = widthC = widthL >> hshift;
> + heightC = heightL >> vshift;
> +
> + int size = width * height;
> + int maxvalY = 255 << (X265_DEPTH - 8);
> + int maxvalC = 255 << (X265_DEPTH - 8);
> + double refValueY = (double)maxvalY * maxvalY * size;
> + double refValueC = (double)maxvalC * maxvalC * size / 4.0;
> +
> + if (firstPic->bitDepth == 8 && X265_DEPTH == 8)
> + {
> + yFirstPic = (pixel*)firstPic->planes[0];
> + ySecPic = (pixel*)secPic->planes[0];
> + if (param->internalCsp != X265_CSP_I400)
> + {
> + uFirstPic = (pixel*)firstPic->planes[1];
> + uSecPic = (pixel*)secPic->planes[1];
> + vFirstPic = (pixel*)firstPic->planes[2];
> + vSecPic = (pixel*)secPic->planes[2];
> + }
> + }
> + else if (firstPic->bitDepth == 8 && X265_DEPTH > 8)
> + {
> + int shift = (X265_DEPTH - 8);
> + uint8_t *yChar1, *yChar2, *uChar1, *uChar2, *vChar1, *vChar2;
> +
> + yChar1 = (uint8_t*)firstPic->planes[0];
> + yChar2 = (uint8_t*)secPic->planes[0];
> +
> + primitives.planecopy_cp(yChar1, firstPic->stride[0] /
> sizeof(*yChar1), m_dupPicOne[0], firstPic->stride[0] / sizeof(*yChar1),
> width, height, shift);
> + primitives.planecopy_cp(yChar2, secPic->stride[0] /
> sizeof(*yChar2), m_dupPicTwo[0], secPic->stride[0] / sizeof(*yChar2),
> width, height, shift);
> +
> + if (param->internalCsp != X265_CSP_I400)
> + {
> + uChar1 = (uint8_t*)firstPic->planes[1];
> + uChar2 = (uint8_t*)secPic->planes[1];
> + vChar1 = (uint8_t*)firstPic->planes[2];
> + vChar2 = (uint8_t*)secPic->planes[2];
> +
> + primitives.planecopy_cp(uChar1, firstPic->stride[1] /
> sizeof(*uChar1), m_dupPicOne[1], firstPic->stride[1] / sizeof(*uChar1),
> widthC, heightC, shift);
> + primitives.planecopy_cp(uChar2, secPic->stride[1] /
> sizeof(*uChar2), m_dupPicTwo[1], secPic->stride[1] / sizeof(*uChar2),
> widthC, heightC, shift);
> +
> + primitives.planecopy_cp(vChar1, firstPic->stride[2] /
> sizeof(*vChar1), m_dupPicOne[2], firstPic->stride[2] / sizeof(*vChar1),
> widthC, heightC, shift);
> + primitives.planecopy_cp(vChar2, secPic->stride[2] /
> sizeof(*vChar2), m_dupPicTwo[2], secPic->stride[2] / sizeof(*vChar2),
> widthC, heightC, shift);
> + }
> + }
> + else
> + {
> + uint16_t *yShort1, *yShort2, *uShort1, *uShort2, *vShort1,
> *vShort2;
> + /* defensive programming, mask off bits that are supposed to be
> zero */
> + uint16_t mask = (1 << X265_DEPTH) - 1;
> + int shift = abs(firstPic->bitDepth - X265_DEPTH);
> +
> + yShort1 = (uint16_t*)firstPic->planes[0];
> + yShort2 = (uint16_t*)secPic->planes[0];
> +
> + if (firstPic->bitDepth > X265_DEPTH)
> + {
> + /* shift right and mask pixels to final size */
> + primitives.planecopy_sp(yShort1, firstPic->stride[0] /
> sizeof(*yShort1), m_dupPicOne[0], firstPic->stride[0] / sizeof(*yShort1),
> width, height, shift, mask);
> + primitives.planecopy_sp(yShort2, secPic->stride[0] /
> sizeof(*yShort2), m_dupPicTwo[0], secPic->stride[0] / sizeof(*yShort2),
> width, height, shift, mask);
> + }
> + else /* Case for (pic.bitDepth <= X265_DEPTH) */
> + {
> + /* shift left and mask pixels to final size */
> + primitives.planecopy_sp_shl(yShort1, firstPic->stride[0] /
> sizeof(*yShort1), m_dupPicOne[0], firstPic->stride[0] / sizeof(*yShort1),
> width, height, shift, mask);
> + primitives.planecopy_sp_shl(yShort2, secPic->stride[0] /
> sizeof(*yShort2), m_dupPicTwo[0], secPic->stride[0] / sizeof(*yShort2),
> width, height, shift, mask);
> + }
> +
> + if (param->internalCsp != X265_CSP_I400)
> + {
> + uShort1 = (uint16_t*)firstPic->planes[1];
> + uShort2 = (uint16_t*)secPic->planes[1];
> + vShort1 = (uint16_t*)firstPic->planes[2];
> + vShort2 = (uint16_t*)secPic->planes[2];
> +
> + if (firstPic->bitDepth > X265_DEPTH)
> + {
> + primitives.planecopy_sp(uShort1, firstPic->stride[1] /
> sizeof(*uShort1), m_dupPicOne[1], firstPic->stride[1] / sizeof(*uShort1),
> widthC, heightC, shift, mask);
> + primitives.planecopy_sp(uShort2, secPic->stride[1] /
> sizeof(*uShort2), m_dupPicTwo[1], secPic->stride[1] / sizeof(*uShort2),
> widthC, heightC, shift, mask);
> +
> + primitives.planecopy_sp(vShort1, firstPic->stride[2] /
> sizeof(*vShort1), m_dupPicOne[2], firstPic->stride[2] / sizeof(*vShort1),
> widthC, heightC, shift, mask);
> + primitives.planecopy_sp(vShort2, secPic->stride[2] /
> sizeof(*vShort2), m_dupPicTwo[2], secPic->stride[2] / sizeof(*vShort2),
> widthC, heightC, shift, mask);
> + }
> + else /* Case for (pic.bitDepth <= X265_DEPTH) */
> + {
> + primitives.planecopy_sp_shl(uShort1, firstPic->stride[1]
> / sizeof(*uShort1), m_dupPicOne[1], firstPic->stride[1] / sizeof(*uShort1),
> widthC, heightC, shift, mask);
> + primitives.planecopy_sp_shl(uShort2, secPic->stride[1] /
> sizeof(*uShort2), m_dupPicTwo[1], secPic->stride[1] / sizeof(*uShort2),
> widthC, heightC, shift, mask);
> +
> + primitives.planecopy_sp_shl(vShort1, firstPic->stride[2]
> / sizeof(*vShort1), m_dupPicOne[2], firstPic->stride[2] / sizeof(*vShort1),
> widthC, heightC, shift, mask);
> + primitives.planecopy_sp_shl(vShort2, secPic->stride[2] /
> sizeof(*vShort2), m_dupPicTwo[2], secPic->stride[2] / sizeof(*vShort2),
> widthC, heightC, shift, mask);
> + }
> + }
> + }
> +
> + if (!(firstPic->bitDepth == 8 && X265_DEPTH == 8))
> + {
> + yFirstPic = m_dupPicOne[0]; ySecPic = m_dupPicTwo[0];
> + uFirstPic = m_dupPicOne[1]; uSecPic = m_dupPicTwo[1];
> + vFirstPic = m_dupPicOne[2]; vSecPic = m_dupPicTwo[2];
> + }
> +
> + //Compute SSD
> + ssdY = computeSSD(yFirstPic, ySecPic, strideL, widthL, heightL,
> param);
> + psnrY = (ssdY ? 10.0 * log10(refValueY / (double)ssdY) : 99.99);
> +
> + if (param->internalCsp != X265_CSP_I400)
> + {
> + ssdU = computeSSD(uFirstPic, uSecPic, strideC, widthC, heightC,
> param);
> + ssdV = computeSSD(vFirstPic, vSecPic, strideC, widthC, heightC,
> param);
> + psnrU = (ssdU ? 10.0 * log10(refValueC / (double)ssdU) : 99.99);
> + psnrV = (ssdV ? 10.0 * log10(refValueC / (double)ssdV) : 99.99);
> + }
> +
> + //Compute PSNR(picN,pic(N+1))
> + return psnrWeight = (psnrY * 6 + psnrU + psnrV) / 8;
> +}
> +
> +void Encoder::copyPicture(x265_picture *dest, const x265_picture *src)
> +{
> + dest->poc = src->poc;
> + dest->pts = src->pts;
> + dest->userSEI = src->userSEI;
> + dest->bitDepth = src->bitDepth;
> + dest->framesize = src->framesize;
> + dest->height = src->height;
> + dest->width = src->width;
> + dest->colorSpace = src->colorSpace;
> + dest->userSEI = src->userSEI;
> + dest->rpu.payload = src->rpu.payload;
> + dest->picStruct = src->picStruct;
> + dest->stride[0] = src->stride[0];
> + dest->stride[1] = src->stride[1];
> + dest->stride[2] = src->stride[2];
> + memcpy(dest->planes[0], src->planes[0], src->framesize *
> sizeof(char));
> + dest->planes[1] = (char*)dest->planes[0] + src->stride[0] *
> src->height;
> + dest->planes[2] = (char*)dest->planes[1] + src->stride[1] *
> (src->height >> x265_cli_csps[src->colorSpace].height[1]);
> +}
> +
> /**
> * Feed one new input frame into the encoder, get one frame out. If
> pic_in is
> * NULL, a flush condition is implied and pic_in must be NULL for all
> subsequent
> @@ -1004,6 +1324,10 @@
> if (m_aborted)
> return -1;
>
> + const x265_picture* inputPic = NULL;
> + static int written = 0, read = 0;
> + bool dontRead = false;
> +
> if (m_exportedPic)
> {
> if (!m_param->bUseAnalysisFile && m_param->analysisSave)
> @@ -1012,25 +1336,84 @@
> m_exportedPic = NULL;
> m_dpb->recycleUnreferenced();
> }
> - if (pic_in && (!m_param->chunkEnd || (m_encodedFrameNum <
> m_param->chunkEnd)))
> - {
> - if (m_latestParam->forceFlush == 1)
> + if ((pic_in && (!m_param->chunkEnd || (m_encodedFrameNum <
> m_param->chunkEnd))) || (m_param->bEnableFrameDuplication && !pic_in &&
> (read < written)))
> + {
> + if ((m_param->bEnableFrameDuplication && !pic_in && (read <
> written)))
> + dontRead = true;
> + else
> {
> - m_lookahead->setLookaheadQueue();
> - m_latestParam->forceFlush = 0;
> + if (m_latestParam->forceFlush == 1)
> + {
> + m_lookahead->setLookaheadQueue();
> + m_latestParam->forceFlush = 0;
> + }
> + if (m_latestParam->forceFlush == 2)
> + {
> + m_lookahead->m_filled = false;
> + m_latestParam->forceFlush = 0;
> + }
> +
> + if (pic_in->bitDepth < 8 || pic_in->bitDepth > 16)
> + {
> + x265_log(m_param, X265_LOG_ERROR, "Input bit depth (%d)
> must be between 8 and 16\n",
> + pic_in->bitDepth);
> + return -1;
> + }
> }
> - if (m_latestParam->forceFlush == 2)
> +
> + if (m_param->bEnableFrameDuplication)
> {
> - m_lookahead->m_filled = false;
> - m_latestParam->forceFlush = 0;
> + double psnrWeight = 0;
> +
> + if (!dontRead)
> + {
> + if (!m_dupBuffer[0]->bOccupied)
> + {
> + copyPicture(m_dupBuffer[0]->dupPic, pic_in);
> + m_dupBuffer[0]->bOccupied = true;
> + written++;
> + return 0;
> + }
> + else if (!m_dupBuffer[1]->bOccupied)
> + {
> + copyPicture(m_dupBuffer[1]->dupPic, pic_in);
> + m_dupBuffer[1]->bOccupied = true;
> + written++;
> + }
> +
> + psnrWeight = ComputePSNR(m_dupBuffer[0]->dupPic,
> m_dupBuffer[1]->dupPic, m_param);
> +
> + if (psnrWeight >= m_param->dupThreshold)
> + {
> + if (m_dupBuffer[0]->bDup)
> + {
> + m_dupBuffer[0]->dupPic->picStruct = tripling;
> + m_dupBuffer[0]->bDup = false;
> + read++;
> + }
> + else
> + {
> + m_dupBuffer[0]->dupPic->picStruct = doubling;
> + m_dupBuffer[0]->bDup = true;
> + m_dupBuffer[1]->bOccupied = false;
> + read++;
> + return 0;
> + }
> + }
> + else if (m_dupBuffer[0]->bDup)
> + m_dupBuffer[0]->bDup = false;
> + else
> + m_dupBuffer[0]->dupPic->picStruct = 0;
> + }
> +
> + if (read < written)
> + {
> + inputPic = m_dupBuffer[0]->dupPic;
> + read++;
> + }
> }
> -
> - if (pic_in->bitDepth < 8 || pic_in->bitDepth > 16)
> - {
> - x265_log(m_param, X265_LOG_ERROR, "Input bit depth (%d) must
> be between 8 and 16\n",
> - pic_in->bitDepth);
> - return -1;
> - }
> + else
> + inputPic = pic_in;
>
> Frame *inFrame;
> x265_param* p = (m_reconfigure || m_reconfigureRc) ?
> m_latestParam : m_param;
> @@ -1038,7 +1421,7 @@
> {
> inFrame = new Frame;
> inFrame->m_encodeStartTime = x265_mdate();
> - if (inFrame->create(p, pic_in->quantOffsets))
> + if (inFrame->create(p, inputPic->quantOffsets))
> {
> /* the first PicYuv created is asked to generate the CU
> and block unit offset
> * arrays which are then shared with all subsequent
> PicYuv (orig and recon)
> @@ -1098,34 +1481,35 @@
> }
>
> /* Copy input picture into a Frame and PicYuv, send to lookahead
> */
> - inFrame->m_fencPic->copyFromPicture(*pic_in, *m_param,
> m_sps.conformanceWindow.rightOffset, m_sps.conformanceWindow.bottomOffset);
> + inFrame->m_fencPic->copyFromPicture(*inputPic, *m_param,
> m_sps.conformanceWindow.rightOffset, m_sps.conformanceWindow.bottomOffset);
>
> inFrame->m_poc = ++m_pocLast;
> - inFrame->m_userData = pic_in->userData;
> - inFrame->m_pts = pic_in->pts;
> - inFrame->m_forceqp = pic_in->forceqp;
> + inFrame->m_userData = inputPic->userData;
> + inFrame->m_pts = inputPic->pts;
> + inFrame->m_forceqp = inputPic->forceqp;
> inFrame->m_param = (m_reconfigure || m_reconfigureRc) ?
> m_latestParam : m_param;
> + inFrame->m_picStruct = inputPic->picStruct;
> if (m_param->bField && m_param->interlaceMode)
> - inFrame->m_fieldNum = pic_in->fieldNum;
> -
> - copyUserSEIMessages(inFrame, pic_in);
> -
> - /*Copy Dolby Vision RPU from pic_in to frame*/
> - if (pic_in->rpu.payloadSize)
> + inFrame->m_fieldNum = inputPic->fieldNum;
> +
> + copyUserSEIMessages(inFrame, inputPic);
> +
> + /*Copy Dolby Vision RPU from inputPic to frame*/
> + if (inputPic->rpu.payloadSize)
> {
> - inFrame->m_rpu.payloadSize = pic_in->rpu.payloadSize;
> - inFrame->m_rpu.payload = new uint8_t[pic_in->rpu.payloadSize];
> - memcpy(inFrame->m_rpu.payload, pic_in->rpu.payload,
> pic_in->rpu.payloadSize);
> + inFrame->m_rpu.payloadSize = inputPic->rpu.payloadSize;
> + inFrame->m_rpu.payload = new
> uint8_t[inputPic->rpu.payloadSize];
> + memcpy(inFrame->m_rpu.payload, inputPic->rpu.payload,
> inputPic->rpu.payloadSize);
> }
>
> - if (pic_in->quantOffsets != NULL)
> + if (inputPic->quantOffsets != NULL)
> {
> int cuCount;
> if (m_param->rc.qgSize == 8)
> cuCount = inFrame->m_lowres.maxBlocksInRowFullRes *
> inFrame->m_lowres.maxBlocksInColFullRes;
> else
> cuCount = inFrame->m_lowres.maxBlocksInRow *
> inFrame->m_lowres.maxBlocksInCol;
> - memcpy(inFrame->m_quantOffsets, pic_in->quantOffsets, cuCount
> * sizeof(float));
> + memcpy(inFrame->m_quantOffsets, inputPic->quantOffsets,
> cuCount * sizeof(float));
> }
>
> if (m_pocLast == 0)
> @@ -1147,9 +1531,9 @@
> }
>
> /* Use the frame types from the first pass, if available */
> - int sliceType = (m_param->rc.bStatRead) ?
> m_rateControl->rateControlSliceType(inFrame->m_poc) : pic_in->sliceType;
> -
> - /* In analysisSave mode, x265_analysis_data is allocated in
> pic_in and inFrame points to this */
> + int sliceType = (m_param->rc.bStatRead) ?
> m_rateControl->rateControlSliceType(inFrame->m_poc) : inputPic->sliceType;
> +
> + /* In analysisSave mode, x265_analysis_data is allocated in
> inputPic and inFrame points to this */
> /* Load analysis data before lookahead->addPicture, since
> sliceType has been decided */
> if (m_param->analysisLoad)
> {
> @@ -1157,7 +1541,7 @@
> static int paramBytes = 0;
> if (!inFrame->m_poc && m_param->bAnalysisType != HEVC_INFO)
> {
> - x265_analysis_data analysisData = pic_in->analysisData;
> + x265_analysis_data analysisData = inputPic->analysisData;
> paramBytes = validateAnalysisData(&analysisData, 0);
> if (paramBytes == -1)
> {
> @@ -1178,10 +1562,10 @@
> uint32_t outOfBoundaryLowresH = extendedHeight -
> m_param->sourceHeight / 2;
> if (outOfBoundaryLowresH * 2 >= m_param->maxCUSize)
> cuLocInFrame.skipHeight = true;
> - readAnalysisFile(&inFrame->m_analysisData,
> inFrame->m_poc, pic_in, paramBytes, cuLocInFrame);
> + readAnalysisFile(&inFrame->m_analysisData,
> inFrame->m_poc, inputPic, paramBytes, cuLocInFrame);
> }
> else
> - readAnalysisFile(&inFrame->m_analysisData,
> inFrame->m_poc, pic_in, paramBytes);
> + readAnalysisFile(&inFrame->m_analysisData,
> inFrame->m_poc, inputPic, paramBytes);
> inFrame->m_poc = inFrame->m_analysisData.poc;
> sliceType = inFrame->m_analysisData.sliceType;
> inFrame->m_lowres.bScenecut =
> !!inFrame->m_analysisData.bScenecut;
> @@ -1202,9 +1586,9 @@
> }
> }
> }
> - if (m_param->bUseRcStats && pic_in->rcData)
> + if (m_param->bUseRcStats && inputPic->rcData)
> {
> - RcStats* rc = (RcStats*)pic_in->rcData;
> + RcStats* rc = (RcStats*)inputPic->rcData;
> m_rateControl->m_accumPQp = rc->cumulativePQp;
> m_rateControl->m_accumPNorm = rc->cumulativePNorm;
> m_rateControl->m_isNextGop = true;
> @@ -1228,6 +1612,18 @@
> }
> m_param->bUseRcStats = 0;
> }
> +
> + if (m_param->bEnableFrameDuplication && ((read < written) ||
> (m_dupBuffer[0]->dupPic->picStruct == tripling && (read <= written))))
> + {
> + if (m_dupBuffer[0]->dupPic->picStruct == tripling)
> + m_dupBuffer[0]->bOccupied = m_dupBuffer[1]->bOccupied =
> false;
> + else
> + {
> + copyPicture(m_dupBuffer[0]->dupPic,
> m_dupBuffer[1]->dupPic);
> + m_dupBuffer[1]->bOccupied = false;
> + }
> + }
> +
> if (m_reconfigureRc)
> inFrame->m_reconfigureRc = true;
>
> @@ -1262,7 +1658,7 @@
> Slice *slice = outFrame->m_encData->m_slice;
> x265_frame_stats* frameData = NULL;
>
> - /* Free up pic_in->analysisData since it has already been
> used */
> + /* Free up inputPic->analysisData since it has already been
> used */
> if ((m_param->analysisLoad && !m_param->analysisSave) ||
> ((m_param->bAnalysisType == AVC_INFO) && slice->m_sliceType != I_SLICE))
> x265_free_analysis_data(m_param,
> &outFrame->m_analysisData);
>
> @@ -3174,6 +3570,30 @@
> p->dynamicRd = 0;
> x265_log(p, X265_LOG_WARNING, "Dynamic-rd disabled, requires RD
> <= 4, VBV and aq-mode enabled\n");
> }
> +
> + if (!p->bEnableFrameDuplication && p->dupThreshold && p->dupThreshold
> != 70)
> + {
> + x265_log(p, X265_LOG_WARNING, "Frame-duplication threshold works
> only with frame-duplication enabled. Enabling frame-duplication.\n");
> + p->bEnableFrameDuplication = 1;
> + }
> +
> + if (p->bEnableFrameDuplication && p->interlaceMode)
> + {
> + x265_log(p, X265_LOG_WARNING, "Frame-duplication does not support
> interlace mode. Disabling Frame Duplication.\n");
> + p->bEnableFrameDuplication = 0;
> + }
> +
> + if (p->bEnableFrameDuplication && p->pictureStructure != 0 &&
> p->pictureStructure != -1)
> + {
> + x265_log(p, X265_LOG_WARNING, "Frame-duplication works only with
> pic_struct = 0. Setting pic-struct = 0.\n");
> + p->pictureStructure = 0;
> + }
> +
> + if (m_param->bEnableFrameDuplication && (!bIsVbv ||
> !m_param->bEmitHRDSEI))
> + {
> + x265_log(m_param, X265_LOG_WARNING, "Frame-duplication require
> NAL HRD and VBV parameters. Disabling frame duplication\n");
> + m_param->bEnableFrameDuplication = 0;
> + }
> #ifdef ENABLE_HDR10_PLUS
> if (m_param->bDhdr10opt && m_param->toneMapFile == NULL)
> {
> diff -r c4b098f973e6 -r fa0d04affb8a source/encoder/encoder.h
> --- a/source/encoder/encoder.h Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/encoder/encoder.h Fri Sep 13 15:57:26 2019 +0530
> @@ -88,6 +88,9 @@
> };
>
> #define MAX_NUM_REF_IDX 64
> +#define DUP_BUFFER 2
> +#define doubling 7
> +#define tripling 8
>
> struct RefIdxLastGOP
> {
> @@ -141,6 +144,18 @@
> }
> };
>
> +struct AdaptiveFrameDuplication
> +{
> + x265_picture* dupPic;
> + char* dupPlane;
> +
> + //Flag to denote the availability of the picture buffer.
> + bool bOccupied;
> +
> + //Flag to check whether the picture has duplicated.
> + bool bDup;
> +};
> +
>
> class FrameEncoder;
> class DPB;
> @@ -189,6 +204,10 @@
> x265_param* m_latestParam; // Holds latest param during a
> reconfigure
> RateControl* m_rateControl;
> Lookahead* m_lookahead;
> + AdaptiveFrameDuplication* m_dupBuffer[DUP_BUFFER]; // picture
> buffer of size 2
> + /*Frame duplication: Two pictures used to compute PSNR */
> + pixel* m_dupPicOne[3];
> + pixel* m_dupPicTwo[3];
>
> bool m_externalFlush;
> /* Collect statistics globally */
> @@ -324,6 +343,12 @@
>
> void calcRefreshInterval(Frame* frameEnc);
>
> + uint64_t computeSSD(pixel *fenc, pixel *rec, intptr_t stride,
> uint32_t width, uint32_t height, x265_param *param);
> +
> + double ComputePSNR(x265_picture *firstPic, x265_picture *secPic,
> x265_param *param);
> +
> + void copyPicture(x265_picture *dest, const x265_picture *src);
> +
> void initRefIdx();
> void analyseRefIdx(int *numRefIdx);
> void updateRefIdx();
> diff -r c4b098f973e6 -r fa0d04affb8a source/encoder/frameencoder.cpp
> --- a/source/encoder/frameencoder.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/encoder/frameencoder.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -713,6 +713,8 @@
> sei->m_picStruct = (poc & 1) ? 2 /* bottom */ : 1
> /* top */;
> }
> }
> + else if (m_param->bEnableFrameDuplication)
> + sei->m_picStruct = m_frame->m_picStruct;
> else
> sei->m_picStruct = m_param->pictureStructure;
>
> diff -r c4b098f973e6 -r fa0d04affb8a source/encoder/framefilter.cpp
> --- a/source/encoder/framefilter.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/encoder/framefilter.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -32,7 +32,6 @@
>
> using namespace X265_NS;
>
> -static uint64_t computeSSD(pixel *fenc, pixel *rec, intptr_t stride,
> uint32_t width, uint32_t height);
> static float calculateSSIM(pixel *pix1, intptr_t stride1, pixel *pix2,
> intptr_t stride2, uint32_t width, uint32_t height, void *buf, uint32_t&
> cnt);
>
> namespace X265_NS
> @@ -673,7 +672,7 @@
> uint32_t width = reconPic->m_picWidth - m_pad[0];
> uint32_t height = m_parallelFilter[row].getCUHeight();
>
> - uint64_t ssdY = computeSSD(fencPic->getLumaAddr(cuAddr),
> reconPic->getLumaAddr(cuAddr), stride, width, height);
> + uint64_t ssdY =
> m_frameEncoder->m_top->computeSSD(fencPic->getLumaAddr(cuAddr),
> reconPic->getLumaAddr(cuAddr), stride, width, height, m_param);
> m_frameEncoder->m_SSDY += ssdY;
>
> if (m_param->internalCsp != X265_CSP_I400)
> @@ -682,8 +681,8 @@
> width >>= m_hChromaShift;
> stride = reconPic->m_strideC;
>
> - uint64_t ssdU = computeSSD(fencPic->getCbAddr(cuAddr),
> reconPic->getCbAddr(cuAddr), stride, width, height);
> - uint64_t ssdV = computeSSD(fencPic->getCrAddr(cuAddr),
> reconPic->getCrAddr(cuAddr), stride, width, height);
> + uint64_t ssdU =
> m_frameEncoder->m_top->computeSSD(fencPic->getCbAddr(cuAddr),
> reconPic->getCbAddr(cuAddr), stride, width, height, m_param);
> + uint64_t ssdV =
> m_frameEncoder->m_top->computeSSD(fencPic->getCrAddr(cuAddr),
> reconPic->getCrAddr(cuAddr), stride, width, height, m_param);
>
> m_frameEncoder->m_SSDU += ssdU;
> m_frameEncoder->m_SSDV += ssdV;
> @@ -825,71 +824,6 @@
> }
> }
>
> -static uint64_t computeSSD(pixel *fenc, pixel *rec, intptr_t stride,
> uint32_t width, uint32_t height)
> -{
> - uint64_t ssd = 0;
> -
> - if ((width | height) & 3)
> - {
> - /* Slow Path */
> - for (uint32_t y = 0; y < height; y++)
> - {
> - for (uint32_t x = 0; x < width; x++)
> - {
> - int diff = (int)(fenc[x] - rec[x]);
> - ssd += diff * diff;
> - }
> -
> - fenc += stride;
> - rec += stride;
> - }
> -
> - return ssd;
> - }
> -
> - uint32_t y = 0;
> -
> - /* Consume rows in ever narrower chunks of height */
> - for (int size = BLOCK_64x64; size >= BLOCK_4x4 && y < height; size--)
> - {
> - uint32_t rowHeight = 1 << (size + 2);
> -
> - for (; y + rowHeight <= height; y += rowHeight)
> - {
> - uint32_t y1, x = 0;
> -
> - /* Consume each row using the largest square blocks possible
> */
> - if (size == BLOCK_64x64 && !(stride & 31))
> - for (; x + 64 <= width; x += 64)
> - ssd += primitives.cu[BLOCK_64x64].sse_pp(fenc + x,
> stride, rec + x, stride);
> -
> - if (size >= BLOCK_32x32 && !(stride & 15))
> - for (; x + 32 <= width; x += 32)
> - for (y1 = 0; y1 + 32 <= rowHeight; y1 += 32)
> - ssd += primitives.cu[BLOCK_32x32].sse_pp(fenc +
> y1 * stride + x, stride, rec + y1 * stride + x, stride);
> -
> - if (size >= BLOCK_16x16)
> - for (; x + 16 <= width; x += 16)
> - for (y1 = 0; y1 + 16 <= rowHeight; y1 += 16)
> - ssd += primitives.cu[BLOCK_16x16].sse_pp(fenc +
> y1 * stride + x, stride, rec + y1 * stride + x, stride);
> -
> - if (size >= BLOCK_8x8)
> - for (; x + 8 <= width; x += 8)
> - for (y1 = 0; y1 + 8 <= rowHeight; y1 += 8)
> - ssd += primitives.cu[BLOCK_8x8].sse_pp(fenc + y1
> * stride + x, stride, rec + y1 * stride + x, stride);
> -
> - for (; x + 4 <= width; x += 4)
> - for (y1 = 0; y1 + 4 <= rowHeight; y1 += 4)
> - ssd += primitives.cu[BLOCK_4x4].sse_pp(fenc + y1 *
> stride + x, stride, rec + y1 * stride + x, stride);
> -
> - fenc += stride * rowHeight;
> - rec += stride * rowHeight;
> - }
> - }
> -
> - return ssd;
> -}
> -
> /* Function to calculate SSIM for each row */
> static float calculateSSIM(pixel *pix1, intptr_t stride1, pixel *pix2,
> intptr_t stride2, uint32_t width, uint32_t height, void *buf, uint32_t& cnt)
> {
> diff -r c4b098f973e6 -r fa0d04affb8a source/input/y4m.cpp
> --- a/source/input/y4m.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/input/y4m.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -388,6 +388,7 @@
> pic.bitDepth = depth;
> pic.framesize = framesize;
> pic.height = height;
> + pic.width = width;
> pic.colorSpace = colorSpace;
> pic.stride[0] = width * pixelbytes;
> pic.stride[1] = pic.stride[0] >>
> x265_cli_csps[colorSpace].width[1];
> diff -r c4b098f973e6 -r fa0d04affb8a source/input/yuv.cpp
> --- a/source/input/yuv.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/input/yuv.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -204,6 +204,7 @@
> pic.bitDepth = depth;
> pic.framesize = framesize;
> pic.height = height;
> + pic.width = width;
> pic.stride[0] = width * pixelbytes;
> pic.stride[1] = pic.stride[0] >>
> x265_cli_csps[colorSpace].width[1];
> pic.stride[2] = pic.stride[0] >>
> x265_cli_csps[colorSpace].width[2];
> diff -r c4b098f973e6 -r fa0d04affb8a source/test/regression-tests.txt
> --- a/source/test/regression-tests.txt Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/test/regression-tests.txt Fri Sep 13 15:57:26 2019 +0530
> @@ -156,6 +156,7 @@
> 720p50_parkrun_ter.y4m,--preset medium --bitrate 400 --hme
> ducks_take_off_420_1_720p50.y4m,--preset medium --aq-mode 4 --crf 22
> --no-cutree
> ducks_take_off_420_1_720p50.y4m,--preset medium --selective-sao 4 --sao
> --crf 20
> +Traffic_4096x2048_30p.y4m, --preset medium --frame-dup --dup-threshold 60
> --hrd --bitrate 10000 --vbv-bufsize 15000 --vbv-maxrate 12000
>
> # Main12 intraCost overflow bug test
> 720p50_parkrun_ter.y4m,--preset medium
> diff -r c4b098f973e6 -r fa0d04affb8a source/x265.cpp
> --- a/source/x265.cpp Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/x265.cpp Fri Sep 13 15:57:26 2019 +0530
> @@ -541,10 +541,11 @@
> return true;
> }
>
> - /* Unconditionally accept height/width/csp from file info */
> + /* Unconditionally accept height/width/csp/bitDepth from file info */
> param->sourceWidth = info.width;
> param->sourceHeight = info.height;
> param->internalCsp = info.csp;
> + param->sourceBitDepth = info.depth;
>
> /* Accept fps and sar from file info if not specified by user */
> if (param->fpsDenom == 0 || param->fpsNum == 0)
> diff -r c4b098f973e6 -r fa0d04affb8a source/x265.h
> --- a/source/x265.h Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/x265.h Fri Sep 13 15:57:26 2019 +0530
> @@ -455,16 +455,21 @@
> * multi pass ratecontrol mode. */
> void* rcData;
>
> - uint64_t framesize;
> + size_t framesize;
>
> int height;
>
> + int width;
> +
>
[PR] Please add new fields in an externally-exposed structure only to the
end to ensure backwards compatibility with applications that cannot / don't
want to recompile.
> // pts is reordered in the order of encoding.
> int64_t reorderedPts;
>
> //Dolby Vision RPU metadata
> x265_dolby_vision_rpu rpu;
> -
> +
> + //SEI picture structure message
> + uint32_t picStruct;
> +
> int fieldNum;
> } x265_picture;
>
> @@ -844,6 +849,9 @@
> * Future builds may support 12bit pixels. */
> int internalBitDepth;
>
> + /*Input sequence bit depth. It can be either 8bit, 10bit or 12bit.*/
> + int sourceBitDepth;
> +
>
[PR] Please add new fields in an externally-exposed structure only to the
end to ensure backwards compatibility with applications that cannot / don't
want to recompile.
> /* Color space of internal pictures, must match color space of input
> * pictures */
> int internalCsp;
> @@ -1327,6 +1335,19 @@
> * */
> int pictureStructure;
>
> + /*
> + * Signals picture structure SEI timing message for every frame
> + * picture structure 7 is signalled for frame doubling
> + * picture structure 8 is signalled for frame tripling
> + * */
> + int bEnableFrameDuplication;
> +
> + /*
> + * For adaptive frame duplication, a threshold is set above which the
> frames are similar.
> + * User can set a variable threshold. Default 70.
> + * */
> + int dupThreshold;
> +
>
[PR] Please add new fields in an externally-exposed structure only to the
end to ensure backwards compatibility with applications that cannot / don't
want to recompile.
> struct
> {
> /* Explicit mode of rate-control, necessary for API users. It must
> diff -r c4b098f973e6 -r fa0d04affb8a source/x265cli.h
> --- a/source/x265cli.h Tue Aug 13 10:51:21 2019 +0530
> +++ b/source/x265cli.h Fri Sep 13 15:57:26 2019 +0530
> @@ -321,6 +321,9 @@
> { "hevc-aq", no_argument, NULL, 0 },
> { "no-hevc-aq", no_argument, NULL, 0 },
> { "qp-adaptation-range", required_argument, NULL, 0 },
> + { "frame-dup", no_argument, NULL, 0 },
> + { "no-frame-dup", no_argument, NULL, 0 },
> + { "dup-threshold", required_argument, NULL, 0 },
> #ifdef SVT_HEVC
> { "svt", no_argument, NULL, 0 },
> { "no-svt", no_argument, NULL, 0 },
> @@ -638,6 +641,8 @@
> H1(" --recon-depth <integer> Bit-depth of reconstructed raw
> image file. Defaults to input bit depth, or 8 if Y4M\n");
> H1(" --recon-y4m-exec <string> pipe reconstructed frames to Y4M
> viewer, ex:\"ffplay -i pipe:0 -autoexit\"\n");
> H0(" --lowpass-dct Use low-pass subband dct
> approximation. Default %s\n", OPT(param->bLowPassDct));
> + H0(" --[no-]frame-dup Enable Frame duplication.
> Default %s\n", OPT(param->bEnableFrameDuplication));
> + H0(" --dup-threshold <integer> PSNR threshold for Frame
> duplication. Default %d\n", param->dupThreshold);
> #ifdef SVT_HEVC
> H0(" --[no]svt Enable SVT HEVC encoder %s\n",
> OPT(param->bEnableSvtHevc));
> H0(" --[no-]svt-hme Enable Hierarchial motion
> estimation(HME) in SVT HEVC encoder \n");
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20191004/6e723915/attachment-0001.html>
More information about the x265-devel
mailing list