Ticket #2262 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

EventWorkspace: reduce memory use, various changes

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: critical Milestone: Iteration 27
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

  • Ensure vectors are the right size.
  • CompressEvents
  • Clearing out unused vector (STL hanging on to memory)

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Janik Zikovsky

(In [8851]) Refs #2262: Re-enabled the precount option in LoadSNSEventNexus. This can significantly reduce memory use when loading.

comment:3 Changed 10 years ago by Janik Zikovsky

(In [8856]) Refs #2262: Making sure memory is released as much as possible in EventList.

comment:4 Changed 10 years ago by Janik Zikovsky

(In [8938]) Refs #2262: DiffractionFocussing pre allocates memory and clears it as it goes.

comment:5 Changed 10 years ago by Janik Zikovsky

(In [8985]) Refs #2262: Do a call to free up any memory before starting any algorithm.

comment:6 Changed 10 years ago by Janik Zikovsky

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

comment:7 Changed 10 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:8 Changed 10 years ago by Owen Arnold

  • Status changed from verifying to closed

All tests pass. Changes made look sensible including use of STL swap trick to release memory associated with containers.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3109

Note: See TracTickets for help on using tickets.