Opened 4 years ago

Closed 4 years ago

#1290 closed defect (fixed)

Issue with ISP when nTbH = 1

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

Description

In Section 8.7.4.1, when nTbH is equal to 1, it appears that numbered Step (1.) and Step (2.) are bypassed.

"1. When nTbH is greater than 1, each (vertical) column..."

"2. When nTbH and nTbW are both greater than 1, ..."

If so, the input d[][] is never used, and nothing is assigned into e[][] or g[][]. By numbered Step (3.), g[][] will be empty and uninitialized.

"3. When nTbW is greater than 1, each (horizontal) row of the resulting array g[ x ][ y ] ..."

It seems like a step is missing where d[][] is copied directly into g[][] when nTbH is equal to 1. Essentially, the same thing as Step (4.), but for nTbH=1 instead of nTbW=1.

Change history (2)

comment:1 Changed 4 years ago by deluxan

I agree with the reported bug. The extra step between Step 2 and Step 3 could be

When nTbH is equal to 1, g[ x ][ y ] is set equal to d[ x ][ y ] for x = 0..nTbW − 1, y = 0

And then Step 4 could be simplified (no need to especify the range of x, given that nTBw is 1) as

When nTbW is equal to 1, r[ x ][ y ] is set equal to e[ x ][ y ] for x = 0, y = 0..nTbH − 1.

comment:2 Changed 4 years ago by jlchen

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

Thanks for the reporting the issue. The suggested solution is good to me.

The fix will be included in S2001-vH release.

Note: See TracTickets for help on using tickets.