Ticket #361 (closed: fixed)

Opened 12 years ago

Last modified 5 years ago

Speed up Cylinder attenuation correction

Reported by: Nick Draper Owned by: Laurent Chapon
Priority: major Milestone: Iteration 16
Component: Keywords:
Cc: Blocked By:
Blocking: Tester:

Description

This is to cover to speedups

  • investigate replacing exponential with a table lookup (talk to Laurent about this otherwise Laurent may well duplicate it anyway)
  • Add a parameter to define how many time bins should be used(100% = all, 10% use 10% of time bins and interpolate).
  • Interpolation should be linear or quadratic whichever is better.

Change History

comment:1 Changed 12 years ago by Nick Draper

  • Owner changed from Russell Taylor to Laurent Chapon

comment:2 Changed 12 years ago by Laurent Chapon

(In [2056]) re#361 Added a fast approximation to the exp function as alternative method. Added the possibility to calculate absorption only for a subset of all bins. The rest is linearly interpolated. Corrected a bug occuring when the neutron track hits the cylinder at two different positions( bottom or top plane and cylinder).

comment:3 Changed 12 years ago by Laurent Chapon

(In [2058]) re#361 Clean-UP

comment:4 Changed 12 years ago by Laurent Chapon

(In [2062]) re#361 Perform some clean-up at the end of algorithm

comment:5 Changed 12 years ago by Laurent Chapon

(In [2064]) re#361 Change test to define new parameter

comment:6 Changed 12 years ago by Laurent Chapon

(In [2065]) re#361 Declare union as static

comment:7 Changed 12 years ago by Laurent Chapon

(In [2066]) re#361 Changed bounded validator

comment:8 Changed 12 years ago by Laurent Chapon

(In [2067]) re#361 Perform some cleanup operations for vectors

comment:9 Changed 12 years ago by Russell Taylor

(In [2070]) Fix problem that caused CorrectForAttenuation test to crash. Re #361.

comment:10 Changed 12 years ago by Nick Draper

  • Milestone changed from Iteration 15 to Iteration 16

Batch move uncompleted tickes to next iteration

comment:11 Changed 12 years ago by Laurent Chapon

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

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​1209

Note: See TracTickets for help on using tickets.