Opened 13 years ago Closed 12 years ago #302 closed defect (fixed)Proofreading issues
Description
A few minor proofreading issues I found in WD9.
Section 8.4.2.1.6 (p.127): incorrect references: it refers to 8.4.2.1.6, 8.4.2.1.7, and 8.4.2.1.10; it should say 8.4.2.1.7, 8.4.2.1.8, and 8.4.2.1.9
Variable yA1 is defined differently in 8.4.2.1.2 and 8.4.2.1.7 (as yP+nPSH-1 and yP+nPSH-MinPuSize); is there a practical difference between the two definitions?
8.4.2.2.2 (8-95) and (8-96) should be referring to mvCLX instead of mvLX
8.4.3.1 (page 141) if InterTUSplitDirection is equal to 2, yB1 and xB2 are undefined
8.4.3.1 (page 142) and 8.4.3.2 (page 143):
Should be:
"– The variable log2TrafoWidth1 is set equal to log2TrafoWidth – 2 * InterTUSplitDirection. Change History (5)comment:1 Changed 13 years ago by DefaultCC Plugin
comment:2 Changed 13 years ago by eugene.kuznetsovcomment:3 Changed 13 years ago by eugene.kuznetsov
8.6.1.4 item 2 (p.152) "of size (2)x(nD)x(nD)," -> "of size (2)x(2)x(nD)x(nD), "
8.6.3 (p.163-164): references to 8.6.2.1, 8.6.2.2, 8.6.2.3, and 8.6.2.4 should be 8.6.3.2, 8.6.3.3, 8.6.3.4, and 8.6.3.5.
8.6.3.2-8.6.3.5: derivation of filter coefficients in 8.6.3.2 seems fundamentally inconsistent with filtering process in 8.6.3.3-8.6.3.5. For example, the sum in (8-476) always has at most 17 terms (judging by the figure 8-6 and from symmetry considerations, it appears that it is intended to have 17 elements at all times), but the array c_L constructed in 8.6.3.2 may have as many as 80 elements. Incomplete merge? As far as I can tell, sections 8.6.3.3-8.6.3.5 were mostly replaced in WD5 d5, but 8.6.3.2 and corresponding syntax sections (7.3.3.5 and 7.4.3.5) still describe the previous algorithm.
8.6.3.4 (p.169): alf_filter_shape_flag is undefined
8.6.3.5 (p.170): alf_filter_shape_chroma_flag is undefined comment:4 Changed 13 years ago by eugene.kuznetsov
Some of the issues mentioned above are still present in the CD:
"RefPicListListA"/"RefPicListListB" instead of "RefPicListLA"/"RefPicListLB" in 3 places
8.5.3.1 (page 134): if InterTUSplitDirection is equal to 2, yB1 and xB2 are undefined
8.5.3.1 (page 135), 8.5.3.2 (page 136), 8.7.1.1 (page 144):
"– The variable log2TrafoWidth1 is set equal to (log2TrafoWidth – 2) * InterTUSplitDirection.
Should be:
"– The variable log2TrafoWidth1 is set equal to log2TrafoWidth – 2 * InterTUSplitDirection.
8.7.1.4 (p.146) "ae" -> "are" comment:5 Changed 12 years ago by bbross
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
|
Some more nitpicking.
The term "coding unit" may need to be defined in section 3. (It is currently defined in section 6.3.)
In section 3, is "may" synonymous with "can"?
In section 8.6.1, in the description of the order in which edges are deblocked within each treeblock, clarify that the right edge and the bottom edge of each coding unit are excluded from filtering.
In section 8.6.1, replace "coinside" with "which coincide" in two places.
Both changes mentioned above for 8.4.3.1 also apply to 8.6.1.1.
8.6.1.3 seems to be saying that, whenever we find ourselves at the intersection of a horizontal edge and a vertical edge, we only apply the deblocking filter to the horizontal edge. Surely this is incorrect?