Opened 13 years ago Closed 13 years ago #533 closed defect (invalid)Incorrect derivation of ColumnWidth and RowHeight with uniform_spacing_flag
Description
This following report is based on I0030.
When uniform_spacing_flag is used, the pseudocode for deriving ColumnWidth and RowHeight does not work properly (see equations 7-20 and 7-21). As an example, for a picture width of 23 CTBs and with three tiles, three tiles of 8 CTBs in width will be allocated.
The following pseudocode is suggested as a replacement for ColumnWidth and RowHeight derivations respectively:
if( uniform_spacing_flag ) {
} else {
}
if( uniform_spacing_flag ) {
} else {
} Change History (8)comment:1 Changed 13 years ago by DefaultCC Plugin
comment:2 follow-up: ↓ 3 Changed 13 years ago by shilin.xucomment:3 in reply to: ↑ 2 Changed 13 years ago by stworrall
Replying to shilin.xu:
Dear Shilin,
If we consider the more general case, 7-20 can be simplified as follows:
ColumnWidth[ i ] =
ColumnWidth[ i ] = ( ( i + 1 - i ) * PicWidthInCtbs ) / ( num_tile_columns_minus1 + 1 )
ColumnWidth[ i ] = PicWidthInCtbs / ( num_tile_columns_minus1 + 1 )
This means that all of the ColumnWidth array values will always be the same. The same is true for the RowHeight derivation. So in the previous example (picture width of 23 CTB, three vertical tiles), all of the three tiles will have a width of 8.
Stewart. comment:4 follow-up: ↓ 5 Changed 13 years ago by shilin.xu
Dear Stewart,
Actually the "/" operator in JCTVC-H1003_dK specifically means integer division with truncation of the result toward zero (please refer to section 5.1). Consequently we can not simplify 7-20 in your way due to the truncation.
Shilin comment:5 in reply to: ↑ 4 Changed 13 years ago by stworrall
Replying to shilin.xu:
Dear Shilin,
Ok, thanks for the clarification. I agree that the existing spec works.
Regards, comment:6 follow-up: ↓ 7 Changed 13 years ago by bbross
Can the ticket be closed then? comment:7 in reply to: ↑ 6 Changed 13 years ago by stworrall
Replying to bbross:
Yes, please feel free to close it.
Regards, comment:8 Changed 13 years ago by ksuehring
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
| ||||||||||||||||
Powered by Trac 1.0.18
By Edgewall Software.
Visit the Trac open source project at
http://trac.edgewall.org/
By using the same example (a picture width of 23 CTBs and with three vertical tiles), the equations 7-20 and 7-21 in I0030 should generate three tiles whose widths are 7, 8, and 8, respectively, rather than 8, 8 and 8. Could you please tell me how you get the 8, 8 and 8 result? Thank you very much.
Shilin