[x265] [PATCH] noiseReduction: make noiseReduction deterministic for a given number of frameEncoders

Steve Borho steve at borho.org
Wed Oct 15 21:06:01 CEST 2014


On 10/15, Deepthi Nandakumar wrote:
> Aah, you' re right. In my older patch, the encoder was initialising
> threadLocalData.nr . Somehow, I missed that while making the new patch.

I had a small epiphany today. If we moved the noise reduction array from
ThreadLocalData into Quant itself:

class Quant {

   NoiseReduction* m_frameNR; // instance per frame encoder, if NR enabled
   NoiseReduction* m_curNR;   // current NR instance, if enabled
   ...
}

setQPforQuant(ctu) could do this:

if (m_frameNr)
   m_curNr = m_frameNr + ctu->m_frame.frameEncoderID;

Then we wouldn't need the ugly setup code in processRowEncoder()
or parallelModeAnalysis()

-- 
Steve Borho


More information about the x265-devel mailing list