Ticket #5908 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

SumSpectra setting incorrect output spectrum number

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Wenduo Zhou

Description (last modified by Peter Peterson) (diff)

The spectrum id of the output spectrum should be the miminum spectrum id being summed rather than the minimum workspace index being summed.

Change History

comment:1 Changed 8 years ago by Peter Peterson

  • Owner set to Peter Peterson
  • Status changed from new to accepted

comment:2 Changed 8 years ago by Peter Peterson

  • Description modified (diff)

The "oddity" in specifying both a range and a list of workspace indices now makes sense. Both are used.

comment:3 Changed 8 years ago by Peter Peterson

Refs #5908. New function to determine the minimum spectrum id summed.

Note that the old way for this to work is to use the mimimum workspace index.

Changeset: a4d2f22186122164e7be3f264a6e55069623b78f

comment:4 Changed 8 years ago by Peter Peterson

Refs #5908. Changed test to reflect new behavior.

Changeset: e213dacbdc55df30010d25ec31e07114ba0c3ca5

comment:5 Changed 8 years ago by Peter Peterson

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

comment:6 Changed 8 years ago by Martyn Gigg

Update Offspec result from change to SumSpectra. Refs #5908

The data itself has not changed, spectrum axis is now correct.

Changeset: 8491bdc103c087a3de054b32270c2f3ad4511479

comment:7 Changed 8 years ago by Wenduo Zhou

  • Status changed from verify to verifying
  • Tester set to Wenduo Zhou

comment:8 Changed 8 years ago by Wenduo Zhou

  • Status changed from verifying to closed

comment:9 Changed 8 years ago by Peter Peterson

Refs #5908. New function to determine the minimum spectrum id summed.

Note that the old way for this to work is to use the mimimum workspace index.

Changeset: a4d2f22186122164e7be3f264a6e55069623b78f

comment:10 Changed 8 years ago by Peter Peterson

Refs #5908. Changed test to reflect new behavior.

Changeset: e213dacbdc55df30010d25ec31e07114ba0c3ca5

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6754

Note: See TracTickets for help on using tickets.