Ticket #3229 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

ReferenceSpectra for CalibrateRectangularDetectors

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Wenduo Zhou

Description

Python algorithm assumed that workspace index was the same as the spectra which is no longer true.

Change History

comment:1 Changed 9 years ago by Vickie Lynch

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Vickie Lynch

(In [12793]) Refs #3229 Change workspace index to detector ID which is now the same as spectra number

comment:3 Changed 9 years ago by Vickie Lynch

(In [12843]) Refs #3229 Change reference pixel to spectra number

comment:4 Changed 9 years ago by Vickie Lynch

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

Works for calibrating examples on wiki page

comment:5 Changed 9 years ago by Vickie Lynch

  • Status changed from verify to reopened
  • Resolution fixed deleted
  • Milestone changed from Iteration 29 to Iteration 30

comment:6 Changed 9 years ago by Vickie Lynch

  • Status changed from reopened to accepted
  • Milestone changed from Iteration 30 to Iteration 29

Need to back out these changes. Changes should be made in CrossCorrelate.

comment:7 Changed 9 years ago by Vickie Lynch

(In [12868]) Refs #3229 Reference pixel should be input as workspace index

comment:8 Changed 9 years ago by Vickie Lynch

(In [12871]) Refs #3229 Used offsets and grouping workspaces and only write cal file at end

comment:9 Changed 9 years ago by Vickie Lynch

(In [12872]) Refs #3229 Use workspace indices instead of spectra as input to CrossCorrelate as Martyn suggested

comment:10 Changed 9 years ago by Vickie Lynch

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

Calibrations match wiki results again

comment:11 Changed 9 years ago by Wenduo Zhou

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

comment:12 Changed 9 years ago by Wenduo Zhou

  • Status changed from verifying to closed

It works fine.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4076

Note: See TracTickets for help on using tickets.