Opened 14 years ago

Closed 14 years ago

#92 closed enhancement (fixed)

Encoder Speed for SIFO

Reported by: rahul Owned by:
Priority: minor Milestone:
Component: HM Version:
Keywords: Cc: bossen@…, peisongc@…, rpanchal@…, martak@…, fbossen, ksuehring, davidf, jct-vc@…

Description

While running TE12 tests it is noticed that SIFO is slower compared to other interpolation filters.
Basically increase in speed is because of multiple ME for different offsets in SIFO.
It can be speedup up by a very small code change in encoder only. (No syntax changes)
After speedup SIFO speed will be same as other interpolation method.

The macro to fix this issue is "SIFO_ENC_SPEEDUP"

Attachments (1)

setFirstPassSubpelOffset.doc​ (38.5 KB) - added by rahul 14 years ago.
modified setFirstPassSubpelOffset() function in TEncSIFO.cpp

Download all attachments as: .zip

Change History (8)

Changed 14 years ago by rahul

modified setFirstPassSubpelOffset() function in TEncSIFO.cpp

comment:1 Changed 14 years ago by secret9th

Dear Rahul,

Personally, I am curious about performance change after encoder speed-up. Do you have any data to check the amount of perforamnce change?

comment:2 follow-up: Changed 14 years ago by rahul

Summary of Performance change is reported below:

Random access
All -0.18(Y BD-rate) -0.16(U BD-rate) -0.23(V BD-rate)
Enc Time[%] 85%

Random access LoCo
All -0.17(Y BD-rate) -0.21(U BD-rate) -0.27(V BD-rate)
Enc Time[%] 77%

Low delay
All -0.13(Y BD-rate) -0.37(U BD-rate) -0.21(V BD-rate)
Enc Time[%] 92%

Low delay LoCo
All -0.04(Y BD-rate) -0.02(U BD-rate) -0.15(V BD-rate)
Enc Time[%] 87%

comment:3 in reply to: ↑ 2 Changed 14 years ago by secret9th

I am seeing the max 23% encoder time saving (Rancom access LoCo) with additional bit saving from your results(I guess minus means bit saving). Good job.

Thank you for your efforts.

Replying to rahul:

Summary of Performance change is reported below:

Random access
All -0.18(Y BD-rate) -0.16(U BD-rate) -0.23(V BD-rate)
Enc Time[%] 85%

Random access LoCo
All -0.17(Y BD-rate) -0.21(U BD-rate) -0.27(V BD-rate)
Enc Time[%] 77%

Low delay
All -0.13(Y BD-rate) -0.37(U BD-rate) -0.21(V BD-rate)
Enc Time[%] 92%

Low delay LoCo
All -0.04(Y BD-rate) -0.02(U BD-rate) -0.15(V BD-rate)
Enc Time[%] 87%

comment:4 Changed 14 years ago by fbossen

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

Fixed in r253

comment:5 Changed 12 years ago by davidf

  • Cc changed from bossen@docomolabs-usa.com,peisongc@qualcomm.com,rpanchal@qualcomm.com,martak@qualcomm.com to bossen@docomolabs-usa.com, peisongc@qualcomm.com, rpanchal@qualcomm.com, martak@qualcomm.com
  • Component set to HM

Updating component after adding WD (Text) tickets

comment:6 Changed 12 years ago by davidf

  • Cc fbossen ksuehring davidf added

comment:7 Changed 12 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

  • bossen@…(Subscriber)
  • David Flynn(Subscriber, Participant)
  • Frank Bossen(Subscriber, Participant)
  • Il-Koo Kim(Participant)
  • jct-vc@…(Subscriber)
  • karl.sharman@…(Always)
  • Karsten Suehring(Subscriber, Always)
  • martak@…(Subscriber)
  • peisongc@…(Subscriber)
  • Rahul Panchal(Reporter, Subscriber, Participant)