Ticket #3126 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

PeaksWorkspace: clone() method and add to CloneWorkspace

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: critical Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

This will be needed to have output Peaks workspaces that take in InputWorkspaces in some algorithms, e.g. PeakIntegration.

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

(In [12278]) Refs #3126: clone() method and copy constructor for PeaksWorkspace.

comment:3 Changed 9 years ago by Janik Zikovsky

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

(In [12282]) Fixes #3126: PeaksWorkspace algorithms use the Clone() method in order to give a different output workspace, if desired. Also added progress reporting to PeakIntegration.

comment:4 Changed 9 years ago by Owen Arnold

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

comment:5 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

Textbook 'Virtual Constructor' pattern implemented. Good test coverage, introduced tests for both copy constructor and Clone method. Tests pass.

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3973

Note: See TracTickets for help on using tickets.