Ticket #1672 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Memory is swallowed up when reloading the instrument many times

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

The following script simply reloads the instrument 50 times. After the first load the memory usage should stay constant but instead it spirals up to about 500MB. 

for i in range(50):
    LoadInstrument('../../../Test/Instrument/SANS2D_Definition.xml','sans2d')

Change History

comment:1 Changed 10 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Martyn Gigg

(In [6347]) Fix a memory leak that occured when reloading the instrument due to the XML document not being released. Also fixed a small issue that Visual Leak Detector picked up in the library loader. Re #1672

comment:3 Changed 10 years ago by Martyn Gigg

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

comment:4 Changed 10 years ago by Michael Whitty

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

comment:5 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

There are still some small memory leaks, but in general this seems to work well when tested in a manner it could possibly be used (ie, not just reloading the instrument thousands of times over and over). The increase detected is orders of magnitude below that which was present in [5988] and is unlikely to inconvenience users.

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2519

Note: See TracTickets for help on using tickets.