Ticket #3481 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SANS: Make FindCenterOfMassPosition compatible with events

Reported by: Mathieu Doucet Owned by: Mathieu Doucet
Priority: major Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

Make FindCenterOfMassPosition compatible with EventWorkspace input. Pull the integration into the algorithm instead of assuming that Integration was run on the input workspace.

Change History

comment:1 Changed 9 years ago by Mathieu Doucet

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Mathieu Doucet

In [13713]:

Added protection against pre-existing ws in event mode. Re #3481

comment:3 Changed 9 years ago by Mathieu Doucet

In [13715]:

Center of mass now accepts event ws. Re #3481

comment:4 Changed 9 years ago by Mathieu Doucet

In [13716]:

Remove Integrate call from RedStep. Re #3481

comment:5 Changed 9 years ago by Mathieu Doucet

In [13717]:

Fix problem with input workspace. Re #3481

comment:6 Changed 9 years ago by Mathieu Doucet

In [13718]:

Added beam finder system test. Re #3481

comment:7 Changed 9 years ago by Mathieu Doucet

In [13719]:

Test with event mode... Re #3481

comment:8 Changed 9 years ago by Mathieu Doucet

To test, run the EQSANSBeamCenter.py system test, which compares the event-mode calculation and the histo-mode calculation to the same reference results.

comment:9 Changed 9 years ago by Mathieu Doucet

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

comment:10 Changed 9 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:11 Changed 9 years ago by Martyn Gigg

  • Status changed from verifying to closed

Verified that the two system test pass.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4328

Note: See TracTickets for help on using tickets.