Opened 9 years ago

Closed 9 years ago

#930 closed defect (fixed)

"short_term_ref_pic_set_idx" is undefined when Ceil(Log2(num_short_term_ref_pic_sets)) is equal to 0.

Reported by: tajime Owned by: bbross
Priority: minor Milestone:
Component: Text Version: D9 (K1003) v13
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

In the WD, "short_term_ref_pic_set_idx" is undefined when Ceil(Log2(num_short_term_ref_pic_sets)) is equal to 0.
I suggest to append "When not present, the value of short_term_ref_pic_set_idx is inferred to be equal to 0."
to the end of its defnition.

For reference, in HM9.1 TDecCavlc::parseSliceHeader(), "short_term_ref_pic_set_idx" is defined bellow;

if (numBits > 0)
{

READ_CODE( numBits, uiCode, "short_term_ref_pic_set_idx");}

else
{

uiCode = 0;

}

Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by yk

This looks like a bug indeed. The same bug applies to the syntax element lt_idx_sps[ i ].

I think the right fixes should be similar to what was agreed at the Shanghai meeting for sending of list_entry_l0[ i ] and list_entry_l1[ i ] only when NumPocTotalCurr > 1.

Thus, the syntax should be changed to:

...
if( !short_term_ref_pic_set_sps_flag )

short_term_ref_pic_set( num_short_term_ref_pic_sets )

else if( num_short_term_ref_pic_sets > 1 )

short_term_ref_pic_set_idx

...

if( i < num_long_term_sps && num_long_term_ref_pics_sps > 1 )

lt_idx_sps[ i ]

...

And in the semantics, add what was suggested by tajime, for both short_term_ref_pic_set_idx and lt_idx_sps[ i ].

comment:3 Changed 9 years ago by bbross

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

Fixed in JCTVC-L0030.

Note: See TracTickets for help on using tickets.

This list contains all users that will be notified about changes made to this ticket.

These roles will be notified: Reporter, Owner, Subscriber, Participant

  • Benjamin Bross(Owner, Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • tajime(Reporter)
  • Woo-Jin Han(Subscriber)
  • Ye-Kui Wang(Participant)