Subclause 8.5 bugs, wording, typos, cross references
Subclause 8.5:
- Step 2.: The decoding process for the residual signal of coding units in inter prediction mode is Subclause 8.5.3 instead of 8.5.2.2.3.2
Subclause 8.5.1:
- wording for inputs: add 'the' between 'specifying' and 'prediction'. (2x)
- PartIdx is a local variable and should therefore be written as partIdx.
- PartMode is a local variable and should therefore be written as partMode.
- For the steps depending on partMode: If the prediction unit size is not the coding unit size,
the decoding process for prediction units in inter prediction mode (subclause 8.5.2) should return
_modified_ arrays predSample{L,Cb,Cr} since the same array is filled step-by-step.
Subclause 8.5.2:
- specification of process output: wording/typo semicolon in "inter prediction samples (predSamples)/;/ which are ..."
Subclause 8.5.2.1:
- Local variables should start lower case in second sentence after Input, Output spec: PredFlagL0, PredFlagL1.
- Step 3 (after eq (8-94)): The derivation process for luma motion vector prediction is subclause 8.5.2.1.5 (instead of 8.5.2.1.3)
(also reported in #340)
Subclause 8.5.2.1.1:
- In step 3, the size of the current PU (nPSW,nPSH) is missing in the parameter set passed to subclause 8.5.2.1.7
The text in subclause 8.5.2.1.7 indicates the refIdx of the 'current prediction unit partition', while it is the refIdx of A1 or 0.
- In step 7, the variable numNscaleMergeCand is not specified
Subclause 8.5.2.1.2:
- Eqs (8-103)--(8-105) miss a "(" after the "=". Same comment for Eqs (8-141)--(8-143).
- Harmonize expression for the location of A1 in 8.5.2.1.2 and 8.5.2.1.6 (top/bottom right pixel of PU, respectively)
Subclause 8.5.2.1.{3,4}:
- Typo "dervied" should be "derived" before eq.(8-106), (8-115), (8-124)
Subclause 8.5.2.1.4:
- Remove round brackets in statement "If (slice_type is equal to P)"
Subclause 8.5.2.1.5:
- Step 2.: replace "subclause 5" by subclause "8.5.2.1.7".
Subclause 8.5.2.1.6:
- Figure 8-3 should be marked informative. The depicted positions of A1,B1,B2 correspond to the specification of A1,B1,B2
in 8.5.2.1.2, not to specification in this subclause.
- Step 1 for mvLXA: Remove the sentence "The set of sample locations ( xAk, yAk ) represent the sample
locations immediately to the left side of the left partition boundary and it’s extended line."
- Step 2 for mvLXA: rephrase, remove 'initially'.
- Steps 4,5 for mvLXA: repeated specification of yA1, remove this.
Subclause 8.5.2.1.{6,7}:
- scaling motion vectors according to POC is specified three times, around eqs. (8-137),(8-145), and (8-155). --> Put into separate process.
- Eq.(8-138,146,156): Fractional maximum MV length 8191.75? all other values are integer. (also ticket #341)
Subclause 8.5.2.2.1 "Reference picture selection process" is not specified.
Subclause 8.5.2.2.2:
- remove 'general' in "These variables are used only inside this subclause for specifying
general fractional-sample locations inside the reference sample arrays ..."
- Replace "0<=xL<nPSW" by "xL=0..nPSW", same for yL, xC, yC.
Subclause 8.5.2.2.2.1:
- The sentence after eq.8-187) "The positions labelled with lower-case letters within un-shaded blocks represent
luma samples at quarter-pel sample fractional locations" should be moved behind the sentence referencing Figure 8-4.
- In the text above eq.(8-173), the term 'rules' should be replaced by 'equations'.
Subclause 8.5.2.2.2.2:
- In the text above eq.(8-188), Figure 8-5 should be referenced instead of Figure 8-4.
- The sentence after eq.8-187) "The positions labelled with lower-case letters within un-shaded blocks represent chroma
samples at eighth-pel sample fractional locations." should be moved behind the sentence referencing (corrected) Figure 8-5.
- In the text above eq.(8-173), the term 'rules' should be replaced by 'equations'.
Subclause 8.5.2.2.3:
- replace "as described in subclause ..." with "as specified in subclause ..."
- The wording "with the same inputs and outputs as ..." when calling a process is not used elsewhere
The same applies to subclauses 8.5.2.2.3.{1,2}.
- The shift and offset values are derived here, but used in the sub-processes 8.5.2.2.3.{1,2} without passing them.
Subclause 8.5.2.2.3.2:
The organization of the subclause seems to be broken.
- the variables for weighted prediction logWDC, logWDC (doubled), w0C, w1C, o0C, o1C are not passed as input to the process in
subclause 8.5.2.2.3 but are then derived after eq.(8-220). The derivation might be moved after input/output and before
the derivation of predSamples.
- The variable name should be modified to log2WDC to indicate the base of the logarithm.
Question: is it specified somewhere that arrays like mvLX[][], refIdxLX[][], etc.,
represent their respective values for each pixel position of a CU/PU? E.g. in subclause 8.5.2.1,
different pixel positions are used for addressing a PU (top left, bottom right, top right pixel).
Change History (6)
Changed 13 years ago by DefaultCC Plugin
-
Cc
bbross wjhan jct-vc@… added
-
Milestone
changed from WD6 to D7
-
Resolution
set to fixed
-
Status
changed from new to closed
| 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)
- Isabelle Perroux(Participant)
- jct-vc@…(Subscriber)
- karl.sharman@…(Always)
- Karsten Suehring(Always)
- Mathias Wien(Reporter)
- Woo-Jin Han(Subscriber)
|
Additionnaly in 8.5.2.1.7: the equation number for RefPicOrderCnt(picX, refIdx, LX)(8 141) is broken. It should be replaced by field codes and display (8-149) instead of (8 141).