Context Navigation

← Previous Ticket
Next Ticket →

Ticket #6971 (inprogress)

Opened 7 years ago

Last modified 5 years ago

TOSCA sample position correction algorithm

Reported by:	Dereck Kachere	Owned by:	Dan Nixon
Priority:	major	Milestone:	Release 3.5
Component:	Indirect Inelastic	Keywords:
Cc:	silvia.imberti@…	Blocked By:
Blocking:		Tester:

Description (last modified by Dan Nixon) (diff)

Add an algorithm that can be used to correct for change in the sample position of reduced TOSCA data.

Silvia has requested involvement in testing the ticket.

Some runs that demonstrate the issue are: 14007, 14009, 14011, 14013, 14015

Change History

comment:1 Changed 7 years ago by Nick Draper

Status changed from new to assigned
Owner set to Dereck Kachere

comment:2 Changed 7 years ago by Nick Draper

Status changed from assigned to new

comment:3 Changed 7 years ago by Nick Draper

Component changed from Mantid to Framework

comment:4 Changed 7 years ago by Nick Draper

Milestone changed from Release 2.6 to Backlog

Moved to backlog at the code freeze for R2.6

comment:5 Changed 7 years ago by Samuel Jackson

Owner changed from Dereck Kachere to Samuel Jackson
Description modified (diff)

comment:6 Changed 7 years ago by Samuel Jackson

Component changed from Framework to Indirect Inelastic

comment:7 Changed 7 years ago by Nick Draper

Status changed from new to assigned

Bulk move to assigned at the introduction of the triage step

comment:8 Changed 6 years ago by Samuel Jackson

Description modified (diff)

comment:9 Changed 6 years ago by Samuel Jackson

Description modified (diff)

comment:10 Changed 6 years ago by Nick Draper

Owner changed from Samuel Jackson to Dan Nixon

comment:11 Changed 6 years ago by Dan Nixon

Milestone changed from Backlog to Release 3.4

This can probably be implemented as a general option where the position reported by Instrument.getSample().getPos() can be offset by a given value in each axis.

comment:12 Changed 6 years ago by Dan Nixon

Blocked By 10854 added

comment:13 Changed 6 years ago by Dan Nixon

Status changed from assigned to inprogress

Added offset options to UI

Refs #6971

Changeset: e958f6f140f4d3fa791a3d7ba35aa0385af91c7e

comment:14 Changed 6 years ago by Dan Nixon

Use a combo box to define type, display default position

Refs #6971

Changeset: 6c9f93f198a9c8bbde6934de6ad540c1ff532758

comment:15 Changed 6 years ago by Dan Nixon

UI part done, needs algorithm support.

Will be easiest to do through the reduction algorithm using the MoveInstrumentComponent algorithm.

comment:16 Changed 6 years ago by Dan Nixon

Summary changed from [C2E] - Energy Transfer Tab Additional Feature - (TOSCA Sample Position) to IDR ET: TOSCA Sample Position Option

comment:17 Changed 6 years ago by Dan Nixon

After discussion with instrument scientists it has been decided that an algorithm to find peaks on the forward and back detector bank spectra and apply a symmetric shift in x such that the peak centres are at the same point is the way to go about this.