Opened 5 years ago

Closed 5 years ago

#400 closed defect (fixed)

Mismatch between spec and vtm on signHidden derivation

Reported by: forayr Owned by:
Priority: minor Milestone:
Component: spec Version: VVC D6 vE
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

There is a mismatch on firstSigScanPosSb and lastSigScanPosSb derivation, which are used to derive signHidden flag.
After parsing dec_abs_level[n], if AbsLevel[ xC ][ yC ] > 0, the specification only updates firstSigScanPosSb, and vtm only updates lastNZPos (which corresponds to lastSigScanPosSb).
I think both should be updated.

The spec should become:

...
  dec_abs_level[ n ]
  if(AbsLevel[ xC ][ yC ] > 0 ) {
    if( lastSigScanPosSb  = =  −1 )
      lastSigScanPosSb = n
    firstSigScanPosSb = n
  }
...

Change history (3)

comment:1 Changed 5 years ago by forayr

MR for the part of this issue concerning the vtm:
https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/merge_requests/755

comment:2 Changed 5 years ago by bbross

  • Version changed from VVC D6 vD to VVC D6 vE

comment:3 Changed 5 years ago by bbross

  • Resolution set to fixed
  • Status changed from new to closed

Thanks for reporting.

This will be fixed JVET-P2001-v7.

Note: See TracTickets for help on using tickets.