Opened 6 years ago
Closed 6 years ago
#235 closed defect (fixed)
N0280 – Transform Skip Residual Coding
| Reported by: | bheng | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | VTM | Version: | VTM-4.2 |
| Keywords: | Cc: | ksuehring, XiangLi, fbossen, jvet@… |
Description (last modified by ksuehring)
1.) The following context derivation for the TS significant coefficient group flag seems to be off by 1.
unsigned sigLeft = unsigned( int( CGPosX - 1 ) > 0 ? ... unsigned sigAbove = unsigned( int( CGPosY - 1 ) > 0 ? ...
Specifically, ( CGPosX - 1 ) > 0 and ( CGPosY - 1 ) > 0. Shouldn’t these just be CGPos > 0? As it is now, this code ignores the first row/column of context.
2.) The following quantization selection logic (dependent quant vs. regular quant) for mtsIdx ==1 should only apply to luma, not chroma.
if( tu.cs->slice->getDepQuantEnabledFlag() && tu.mtsIdx != 1 )
Change history (2)
comment:1 Changed 6 years ago by ksuehring
- Description modified (diff)
comment:2 Changed 6 years ago by bheng
- Resolution set to fixed
- Status changed from new to closed
Note: See TracTickets for help on using tickets.
Fixed with merge request:
https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/512