Ticket #5717 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Stitch1D should handle group workspaces

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Should just be able to point the algorithm at a multiperiod group workspace and let it pair and process each workspace.

Just and idea, but since the multiperiod group processing now in MergeRuns is quite generic, could we have an itermediate base class to do this for us? Maybe another ticket.

Change History

comment:1 Changed 8 years ago by Owen Arnold

First have to convert Stitch1D into a c++ algorithm, so that the base class in #5744 can be used. See #5754

comment:2 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

Under ticket #5754, I both converted to algorithm to a c++ version and upgraded it to work correctly with multiperiod group data. Should have done the latter part under this ticket, but forgot to reference it. This functionality is unit tested as part of those changes.

comment:3 Changed 8 years ago by Owen Arnold

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

comment:4 Changed 8 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:5 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to reopened
  • Resolution fixed deleted

If RHSWorkspace = LHSWorkspace then crash

comment:6 Changed 8 years ago by Anders Markvardsen

  • Status changed from reopened to assigned

comment:7 Changed 8 years ago by Owen Arnold

refs #5717. Fix zero overlap issue

Changeset: 3ff154f9679118a02418a15054eb7ddae52bdc06

comment:8 Changed 8 years ago by Owen Arnold

  • Status changed from assigned to accepted

comment:9 Changed 8 years ago by Owen Arnold

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

Shouldn't crash

comment:10 Changed 8 years ago by Anders Markvardsen

  • Status changed from verify to verifying

comment:11 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Tested this by reading workspaces created with createmdhistoworkspace into alg

comment:12 Changed 8 years ago by Owen Arnold

refs #5717. Fix zero overlap issue

Changeset: 3ff154f9679118a02418a15054eb7ddae52bdc06

comment:13 Changed 8 years ago by Owen Arnold

refs #5717. Fix zero overlap issue

Changeset: 3ff154f9679118a02418a15054eb7ddae52bdc06

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6563

Note: See TracTickets for help on using tickets.