Ticket #4322 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Fix MemoryLeak in ConvertToMatrixWorkspace

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: martyn.gigg@… Blocked By:
Blocking: Tester: Wenduo Zhou

Description

Martyn reports a memory leak when running: 

for i in range(0,10):
	Load('CNCS_7860_event.nxs','WS')
	Rebin('WS','WS','45000,10,60000')
	ConvertToMatrixWorkspace('WS','WS')

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted
  • type changed from enhancement to defect

comment:2 Changed 9 years ago by Janik Zikovsky

Refs #4322 : Fix a memory leak in EvenList

Related to MRU's being set by both dataY and dataE. Took out the Y setting from E getting and this fixes the leak. Speed should still be good as long as you call dataY() first.

Changeset: bc0d66a8c7d7d47ace4eea6fa4c82b649becf387

comment:3 Changed 9 years ago by Janik Zikovsky

Refs #4322 : Fix a memory leak in EvenList

Related to MRU's being set by both dataY and dataE. Took out the Y setting from E getting and this fixes the leak. Speed should still be good as long as you call dataY() first.

Changeset: bc0d66a8c7d7d47ace4eea6fa4c82b649becf387

comment:4 Changed 9 years ago by Janik Zikovsky

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

comment:5 Changed 9 years ago by Wenduo Zhou

  • Status changed from verify to verifying
  • Tester set to Wenduo Zhou

comment:6 Changed 9 years ago by Wenduo Zhou

  • Status changed from verifying to closed

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5169

Note: See TracTickets for help on using tickets.