Opened 14 years ago

Closed 14 years ago

#51 closed defect (fixed)

Bug in edge based prediction for low complexity

Reported by: virginie.drugeon@… Owned by: fbossen
Priority: minor Milestone:
Component: HM Version:
Keywords: Cc: fbossen, ksuehring, davidf, jct-vc@…

Description

Edge based complexity is currently never used in the low complexity configuration.

The following bug fix solves the problem:
In the file TComEdgeBased.cpp, the function TComEdgeBased::shift_right_round should be:

inline int TComEdgeBased::shift_right_round(int val, int b)
{

if(b <= 0)

return val;

else

return (val + (1 << (b-1))) >> b;

}

Change History (6)

comment:1 Changed 14 years ago by anonymous

Edge based prediction is currently never used in the low complexity configuration.

The following bug fix solves the problem:
In the file TComEdgeBased.cpp, the function TComEdgeBased::shift_right_round should be:

inline int TComEdgeBased::shift_right_round(int val, int b)
{

if(b <= 0)

return val;

else

return (val + (1 << (b-1))) >> b;

}

comment:2 Changed 14 years ago by fbossen

  • Owner set to fbossen
  • Status changed from new to accepted

comment:3 Changed 14 years ago by fbossen

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

Fixed in r161

comment:4 Changed 13 years ago by davidf

  • Component set to HM

Updating component after adding WD (Text) tickets

comment:5 Changed 13 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:6 Changed 13 years ago by davidf

  • Cc jct-vc@… added
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

  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Owner, Subscriber, Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)
  • virginie.drugeon@…(Reporter)