Ticket #230 (closed: fixed)

Opened 12 years ago

Last modified 5 years ago

Merge Workspaces algorithm

Reported by: Nick Draper Owned by: Russell Taylor
Priority: major Milestone: Iteration 12
Component: Keywords:
Cc: Blocked By:
Blocking: Tester:

Description

This should allow two partial and non overlapping workspaces to be merged into one. For example you have processed a raw file in to halves, and then want to merge the results into a single workspace.

Pre-requisites

  • workspaces come from the same instrument
  • No detectors that contribute to spectra should overlap
  • x units and bins must match

Change History

comment:1 Changed 12 years ago by Russell Taylor

(In [1413]) Adding an algorithm to stick together two workspaces containing half of a dataset that's been processed in two parts (probably for memory reasons). Re #230.

comment:2 Changed 12 years ago by Russell Taylor

(In [1414]) Changes to WorkspaceProperty in check-in #1411 appear at first glance to have made it impossible to reassign a workspace property once set. No time to look into this now, so to try and get a passed build for the weekend I've commented out the lines of my new test that fall foul of this. Re #230.

comment:3 Changed 12 years ago by Russell Taylor

(In [1417]) Fixed a bug which had unintentionally restricted ConjoinWorkspaces to input workspaces of same size. Re #230.

comment:4 Changed 12 years ago by Russell Taylor

(In [1419]) Reworking of WorkspaceProperty to remove evil side-effect of calling its isValid() method. Re #230.

comment:5 Changed 12 years ago by Russell Taylor

  • Status changed from new to closed
  • Resolution set to fixed

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​1078

Note: See TracTickets for help on using tickets.