Ticket #5860 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

SaveNexus cannot be run as a subalgorithm

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

It passes the InputWorkspace around internally as a string of its name, which is the wrong thing to do. It should use the pointer so that it can be run when given just a pointer to the input workspace rather than a name, as is usually the case when algorithms are run as sub-algorithms.

Change History

comment:1 Changed 8 years ago by Russell Taylor

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Russell Taylor

Re #5860. Pass workspace internally by pointer not name in SaveNexus.

Also add a unit test for this case.

Changeset: 55f6ec4cb4a00b1f7c6f354285765ab427f5d664

comment:3 Changed 8 years ago by Russell Taylor

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

comment:4 Changed 8 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:5 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

This looks like it was done correctly.

comment:6 Changed 8 years ago by Russell Taylor

Re #5860. Pass workspace internally by pointer not name in SaveNexus.

Also add a unit test for this case.

Changeset: 55f6ec4cb4a00b1f7c6f354285765ab427f5d664

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6706

Note: See TracTickets for help on using tickets.