Opened 6 years ago

Closed 6 years ago

#70 closed defect (invalid)

Typos in VVC D2 v4

Reported by: vdrugeon Owned by:
Priority: minor Milestone:
Component: 360Lib Version: VVC D2 v4
Keywords: Cc: vzakharc, yuwenhe, jvet@…

Description

Here is a list of some more typos and small issues that I found:

In section 8.2.4.2.1, when generating the reference samples p, there is a typo in step 1 "the reference samples refUnfilt[ x ][ y ] with x = −1, y = −1 refH − 1 and x = 0..refW − 1, y = −1 as output" where the ".." are missing for y=-1..refH-1.

In section 8.2.4.2.2 "The refW + refH − 1 neighbouring samples refUnfilt[ x ][ y ] that are constructed samples prior to the in-loop filter process, with x = −1, y = −1..refH − 1 and x = 0..refW − 1, y = −1": there are refW + refH + 1 samples.

In section 8.2.4.2.3, the outputs are currently listed as one of the inputs. They should be listed separately.

In section 8.2.4.2.3 step 3 "the value of refUnfilt[ x ][ y ] is set equal to p the value of refUnfilt[ x − 1 ][ y ]", the "p" before the "the value of" should be removed.

In section 8.2.4.2.4 at the end when p[x][y] is derived, equation (8-23) sets pF[x][-1] instead of p[x][-1].

In section 8.2.2, cbWidth and cbHeight should be added as inputs to the process.

General question about the order in which inputs to a process are listed (it may not be important):
In some processes, an array is needed as input together with its size, as for example in process 8.2.4.2.3:
"
Inputs to this process are:
– reference samples refUnfilt[ x ][ y ] with x = −1, y = −1..refH − 1 and x = 0..refW − 1, y = −1 for intra sample prediction,
– a variable refW specifying the reference samples width,
– a variable refH specifying the reference samples height,
"
Would it make more sense to list the size of the array BEFORE listing the array? Otherwise the first input contains two variables (in the example above refH and refW) that are defined afterwards.

Change history (1)

comment:1 Changed 6 years ago by vdrugeon

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

Really sorry, it was meant as a "spec" ticket, not as a "360Lib" ticket

Note: See TracTickets for help on using tickets.