Ticket #2324 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

ConvertToMatrixWorkspace and CompressEvents: call SortAll() before running

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

Description

... since it is necessary anyway, and can save time in some cases where the sorting algorithm picked is smarter (i.e. a single ginormous event list).

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted
  • Priority changed from major to minor

comment:2 Changed 10 years ago by Janik Zikovsky

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

(In [9021]) Fixes #2324: Call to sortAll before CompressEvents speeds things up from 80 to 30 seconds in one example (95 million event list). Same thing in ConvertToMatrixWorkspace. Nicer reporting.

comment:3 Changed 10 years ago by Michael Whitty

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

comment:4 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

works in [9515]

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3171

Note: See TracTickets for help on using tickets.