[x265] [PATCH] psyrd: use psyrdcost for intra for rdLevels 5

2014-05-22 Thread sumalatha
# HG changeset patch # User Sumalatha Polureddysumala...@multicorewareinc.com # Date 1400752845 -19800 # Node ID fc400f71de9dcbaeda5c0669edabd27e288cdf2c # Parent f39484bb3eecc8cfca0448c63f16fe8dacc54d7f psyrd: use psyrdcost for intra for rdLevels 5 diff -r f39484bb3eec -r fc400f71de9d

Re: [x265] [PATCH] psyrd: use psyrdcost for intra for rdLevels 5

2014-05-22 Thread Deepthi Nandakumar
psy-rd is enabled only in full-rdo decisions (rdLevel = 5). Your patch calculates intra-psy rd cost for rd=4 and lower, which is where xEncodeIntrainInter is called. On Thu, May 22, 2014 at 3:31 PM, sumala...@multicorewareinc.com wrote: # HG changeset patch # User Sumalatha

Re: [x265] [PATCH] psyrd: use psyrdcost for intra for rdLevels 5

2014-05-22 Thread Sumalatha Polureddy
psy-rd is enabled for full-rdo decision(rd=5) and for rd= 4 m_rdCost.setPsyRdScale(top-param-rdLevel = 4 ? top-param-psyRd : 0); so this patch is required for rd=4 where inter uses psyrdcost and intra was using rdcost. For other lower rd levels 4, m_psyRdScale = 0, so psycost will not have much

Re: [x265] [PATCH] psyrd: use psyrdcost for intra for rdLevels 5

2014-05-22 Thread Deepthi Nandakumar
On Thu, May 22, 2014 at 4:23 PM, Sumalatha Polureddy sumala...@multicorewareinc.com wrote: psy-rd is enabled for full-rdo decision(rd=5) and for rd= 4 This is a bug - psy-rd should be enabled only for rd=5. Fixing now. m_rdCost.setPsyRdScale(top-param-rdLevel = 4 ? top-param-psyRd : 0);