Opened 9 years ago

Closed 9 years ago

#591 closed defect (fixed)

Issue of SAO edgeIdx derivation

Reported by: chihming.fu Owned by: bbross
Priority: minor Milestone: D7
Component: Text Version: D7 (I1003) d4
Keywords: Cc: bbross, wjhan, jct-vc@…

Description

The Sign(x) function in the text is as follows,
if (x>=0), Sign(x)= 1,
otherwise if (x<0), Sign(x)= -1

However, the Sign(x) function for edgeIdx should be
if (x>0), Sign(x)= 1,
otherwise if (x==0), Sign(x)= 0
otherwise if (x<0), Sign(x)= -1

Therefore, we should define a new Sign1(x) function for edgeIdx,

edgeIdx = 2 + sum_k( Sign1( recPicture[ xC + i, yC + j ] –

recPicture[ xC + i + hPos[ k ], yC + j + vPos[ k ] ] ) ) with k = 0..1 (8-350)

where Sign1(x) function is as follows,
if (x>0), Sign1(x)= 0,
otherwise if (x==0), Sign1(x)= 0
otherwise if (x<0 ), Sign1(x)= -1


Change History (3)

comment:1 Changed 9 years ago by DefaultCC Plugin

  • Cc bbross wjhan jct-vc@… added

comment:2 Changed 9 years ago by chihming.fu

Typo fix:
if (x>0), Sign1(x)= 1,
otherwise if (x==0), Sign1(x)= 0
otherwise if (x<0 ), Sign1(x)= -1

comment:3 Changed 9 years ago by bbross

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

This is fixed in JCTVC-I1003 d5 (v6).

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

  • Benjamin Bross(Owner, Subscriber, Participant)
  • Chih-Ming Fu(Reporter, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Always)
  • Woo-Jin Han(Subscriber)