Ticket #1783 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Add default reductionstep wrapper for algorithms

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: minor Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

To aid in making reducers a more frequently used thing, add a wrapper class around algorithm with a default implementation.

Also upgrade Reducer's append_step to automatically convert algorithms being added to a reducer into a reduction step.

Change History

comment:1 Changed 10 years ago by Peter Peterson

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Peter Peterson

(In [7022]) Added the AlgoReductionStep. Refs #1783.

comment:3 Changed 10 years ago by Peter Peterson

(In [7023]) Added a more meaningful representation properly handle adding None to a reducer. Refs #1783.

comment:4 Changed 10 years ago by Peter Peterson

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

comment:5 Changed 10 years ago by Michael Reuter

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

comment:6 Changed 10 years ago by Michael Reuter

  • Status changed from verifying to closed

After reading through the code, it seems to do what is being requested of it. This isn't in wide use yet, so I can't make a definitive test.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2630

Note: See TracTickets for help on using tickets.