Opened 6 years ago
Closed 6 years ago
#237 closed defect (fixed)
N0413 – DPCM Reference Sample Filtering
Reported by: | bheng | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | VTM-5.0 |
Component: | VTM | Version: | VTM-4.2 |
Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description
It appears that N0413 DPCM implementation intends to use unfiltered luma reference samples. For example, the following code seems to try to implement this:
IntraPrediction::useFilteredIntraRefSamples( )
{
...
#if JVET_N0413_RDPCM
if( isLuma( chType ) && pu.cu->bdpcmMode ) { return false; }
#endif
...
}
However, it appears that the above function “useFilteredIntraRefSamples” is never actually called anywhere. Therefore, the above code is never executed and has no effect.
Instead, in the following function, the DPCM mode is treated as PLANAR and the reference filtering can be enabled (or not) depending on the block size.
If this is not the intended behavior, I believe the following function should be fixed to add a DPCM condition.
IntraPrediction::initPredIntraParams( )
{
...
else if (dirMode == PLANAR_IDX) Planar intra prediction
{
m_ipaParam.refFilterFlag = puSize.width * puSize.height > 32 ? true : false;
}
...
}
Change history (1)
comment:1 Changed 6 years ago by XiangLi
- Milestone set to VTM-5.0
- Resolution set to fixed
- Status changed from new to closed
Fixed in https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/514