Ticket #1359 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

EventWorkspace: Implement Most Recently Used histogram list

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 24
Component: Keywords: event
Cc: pf9@… Blocked By:
Blocking: Tester: Peter Peterson

Description

Hold a list of the N most recently accessed histograms in a cache. Clear the older histograms (free up the memory) as they fall off the end of the cache queue.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Keywords event added

comment:2 Changed 10 years ago by Russell Taylor

We have a most-recently-used list implemented in the file DataObjects/inc/MantidDataObjects/AbsManagedWorkspace2D.h You should re-use that code if possible (it's currently an inner class, but doesn't have to stay that way).

comment:3 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:4 Changed 10 years ago by Janik Zikovsky

(In [5224]) Refs #1359 and #1376. Implements a MRU for the EventWorkspace. Tracked down memory leak - memory usage seems constant in use in MantidPlot.

comment:5 Changed 10 years ago by Martyn Gigg

(In [5230]) Fix a test build. Re #1359

comment:6 Changed 10 years ago by Martyn Gigg

(In [5231]) Actually fix the test build this time... Re #1359

comment:7 Changed 10 years ago by Janik Zikovsky

(In [5239]) Refs #1359: adding a unit test for memory usage.

comment:8 Changed 10 years ago by Janik Zikovsky

(In [5249]) Refs #1359: Disabled test I can't get to pass.

comment:9 Changed 10 years ago by Janik Zikovsky

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

comment:10 Changed 10 years ago by Nick Draper

  • Status changed from testing to verify

Moved to verify state

comment:11 Changed 10 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:12 Changed 10 years ago by Peter Peterson

  • Status changed from verifying to closed

This works.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2206

Note: See TracTickets for help on using tickets.