Ticket #2244 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

CompressEvents: New algorithm to reduce event workspace memory use

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 27
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

Look through each event list, and lump together similar events (events with the same or possible nearly the same TOF/X value); ignore pulse-time (assume that event filtering by time has been completed).

Create a new Weighted event list and if 2 events matched, make one event with Weight = 2 and error = sqrt(2).

Possibly use "rebin"-style parameters to determine how close you consider events to be. It would be wise to use very small bins here otherwise you'd introduce inaccuracies.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

Additional note: for more precision, when you lump together N events into one, use the average X value (average TOF) as its new X, rather than the center of the bin provided. This will improve accuracy if it happens that 10 events are in the bin, but all close to one end, for example.

comment:2 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:3 Changed 10 years ago by Janik Zikovsky

(In [8833]) Refs #2244: CompressEvents algorithm added to reduce number of events.

comment:4 Changed 10 years ago by Janik Zikovsky

(In [8834]) Refs #2244: Fix test build.

comment:5 Changed 10 years ago by Janik Zikovsky

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

comment:6 Changed 10 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester set to Michael Whitty

comment:7 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

tested in [9515]. noticeable change in workspace size, etc

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3091

Note: See TracTickets for help on using tickets.