Opened 4 years ago

Closed 3 years ago

#1148 closed defect (fixed)

Multiple bugs in preceding IRAP, GDR pictures and subpictures

Reported by: Vadim Owned by:
Priority: minor Milestone:
Component: VTM Version:
Keywords: Cc: ksuehring, XiangLi, fbossen, jvet@…

Description

  1. In m_prevGDRInSameLayerPOC{ MAX_INT }, m_pocCRA{ MAX_INT } only the first element is initialized to the specified value, others are 0. It means it is not correct for the second layer and onward. MAX_INT is not the correct value for IRAP either.
  1. Initializing preceding IRAP POC to MAX_INT or 0 is incorrect for the very first IRAP picture as some reference POCs if present can be negative and constraint check fails, it should be initialized with a large negative number, i.e. m_pocCRA{ -MAX_INT }, which actually should be std::fill_n(m_pocCRA[i], MAX_VPS_LAYERS, -MAX_INT);
  1. Same problem is for preceding IRAP subpicture, it should be m_prevIRAPSubpicPOC( -MAX_INT ), std::fill_n( m_prevIRAPSubpicPOC[i], MAX_NUM_SUB_PICS, -MAX_INT ).
  1. Very likely, it is the same problem for preceding GDR pictures and sub-pictures, it should be checked.

Change history (2)

comment:1 Changed 4 years ago by ZhipinDeng

Thanks for your suggestions! I submitted the following MR for it. I would appreciate if you could have a review.

https://vcgit.hhi.fraunhofer.de/jvet/VVCSoftware_VTM/-/merge_requests/1873

comment:2 Changed 3 years ago by fbossen

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.