Ticket #277 (closed enhancement: fixed)

Opened 12 years ago

Last modified 11 years ago

Speed up imageproc

Reported by: rhl Owned by: rhl
Priority: normal Milestone:
Component: imageProcess Keywords:
Cc: Blocked By:
Blocking: Project: LSST
Version Number: 3704
How to repeat:

Run

examples/runImageSubtract.py

and see how long it takes

Description (last modified by rhl) (diff)

This ticket gives us somewhere to speed up imageproc (it also required changes on an fw/tickets/277 branch)

Change History

comment:1 Changed 12 years ago by rhl

  • Status changed from new to assigned

The first merge from this ticket (moving .cc files from include to src) is merged as [3814].

There are obvious code changes to speed this code up, and I'll make them over the next few days. I think that an FFT can be used to make the algorithm dramatically faster, but this needs to be confirmed.

comment:2 Changed 11 years ago by rhl

  • Description modified (diff)

I have restructured things in fw's Kernel classes to make the Kernel type available to the convolution routines, and provided partial specialisations for a new Kernel type, DeltaFunctionKernel. This lead to a 30-40% speedup.

This code isn't checked into the trunk (although I have done the merge) as it causes swig 1.3.33 to die with a bus error. The svn version of swig is OK, but I don't know that we want to deploy swig 1.3.34m1

comment:3 Changed 11 years ago by rhl

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

The work on this was reworked by Russell (particularly using DeltaFunction? Kernels) and is on the mainline.

comment:4 Changed 11 years ago by robyn

  • Milestone DC2 Benchmark Runs deleted

Milestone DC2 Benchmark Runs deleted

Note: See TracTickets for help on using tickets.