Opened 10 years ago

Closed 9 years ago

#224 closed defect (wontfix)

Bug regarding the tiles parameter coding in PPS & SPS

Reported by: shilin.xu Owned by: ksuehring
Priority: minor Milestone: HM-5.2
Component: HM Version: HM-4.0
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

The implementation of the PPS and SPS syntax for tiles does not correctly reflect the syntax adopted in JCTVC-F335.

Attachments (1)

tiles_SPS_PPS_bugfix.patch (13.9 KB) - added by shilin.xu 10 years ago.
This is the patch for this bug

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by shilin.xu

The implementation of the PPS and SPS syntax in HM4.0-dev for tiles does not correctly reflect the syntax adopted in JCTVC-F335. Below, we show the difference between the HM code and the correct syntax from JCTVC-F335 for the SPS. The syntax differences for the PPS are exactly the same.

A code fragment from SPS in the HM:

uniform_spacing_idc
tile_boundary_independence_idc
num_tile_columns_minus1
num_tile_rows_minus1
if (uniform_spacing_idc == 0) {

for (i=0; i<num_tile_columns_minus1 ; i++)

column_width[i]

for (i=0; i <num_tile_rows_minus1; i++)

row_height[i]

}


The syntax defined in JCTVC-F335 is:

num_tile_columns_minus1
num_tile_rows_minus1
if (num_tile_columns_minus1 != 0 ll num_tile_rows_minus1 != 0) {

tile_boundary_independence_idc
uniform_spacing_idc
if (uniform_spacing_idc != 1) {

for (i=0; i<num_tile_columns_minus1 ; i++)

column_width[i]

for (i=0; i <num_tile_rows_minus1; i++)

row_height[i]

}

}

Changed 10 years ago by shilin.xu

This is the patch for this bug

comment:2 Changed 10 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:3 Changed 10 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:4 Changed 10 years ago by davidf

  • Cc jct-vc@… added

comment:5 Changed 10 years ago by ksuehring

  • Milestone set to HM-5.2
  • Owner set to ksuehring
  • Status changed from new to assigned

comment:6 Changed 9 years ago by ksuehring

  • Resolution set to wontfix
  • Status changed from assigned to closed

It seems we forgot this patch. The syntax has been changed in HM 6.1 and will be revised in 7.x.

I guess the issues will be fixed with that.

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

  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Owner, Subscriber, Participant, Always)
  • Shilin Xu(Reporter, Participant)