Opened 4 years ago

Closed 4 years ago

#614 closed enhancement (fixed)

Editorial comments on 8.8.3.6.7 Filtering process for a luma sample using long filters

Reported by: vdrugeon Owned by:
Priority: minor Milestone:
Component: spec Version:
Keywords: Cc: ksuehring, bbross, XiangLi, fbossen, jvet@…

Description

In 8.8.3.6.7 Filtering process for a luma sample using long filters, the following inputs and outputs are listed:
"Inputs to this process are:

– the variables maxFilterLengthP and maxFilterLengthQ,
– the sample values pi and qj with i = 0..maxFilterLengthP and j = 0..maxFilterLengthQ,
– the locations of pi and qj, ( xPi, yPi ) and ( xQj, yQj ) with i = 0..maxFilterLengthP − 1 and j = 0..maxFilterLengthQ − 1,
– a variable tC.

Outputs of this process are:
– the filtered sample values pi′ and qj′ with i = 0..maxFilterLengthP − 1, j = 0..maxFilterLenghtQ − 1."

When this process is invoked in 8.8.3.6.2 Filtering process for luma block edges, the inputs are listed in the wrong order (maxFilterLengthP and maxFilterLengthQ are not listed first).
The locations of pi and qj seem not to be used in the process at all. Are these inputs necessary?
Is it intended that the number of filtered sample values that are output is different from the number of input sample values?

Change history (2)

comment:1 Changed 4 years ago by Kenneth

Thanks for your comments.

  • I agree that the order of input parameters should be corrected.
  • The locations are not needed inside 8.8.3.6.7. Same applies to 8.8.3.6.6. I suspect this is a left over from when PCM was removed from the specification.
  • It is indended that the number of filtered samples is less than the number of input samples.

comment:2 Changed 4 years ago by bbross

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

Thanks for reporting.

This will be fixed JVET-P2001-vC.

Note: See TracTickets for help on using tickets.