Ticket #3927 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Refactor MDBoxIterator so that it holds a vector of boxes

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

This will allow it to know how large it is (getDataSize()) and will, in the future, allow one to paralellize the algorithm.

Use the workspace's getBoxes() to get the vector of boxes to fill.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Janik Zikovsky

Also make it inherit from IMDIterator.

comment:3 Changed 9 years ago by Janik Zikovsky

In [15177]:

Refs #3927: Made MDBoxIterator inherit from IMDIterator. Considerably simplified it by using a vector of boxes from getBoxes() instead of the old way. It is slightly slower in some cases.

comment:4 Changed 9 years ago by Janik Zikovsky

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

In [15178]:

Fixes #3927: createIterator() method on MDEventWorkspace means it fully inherits IMDWorkspace now.

comment:5 Changed 9 years ago by Owen Arnold

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

comment:6 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4774

Note: See TracTickets for help on using tickets.