Context Navigation

Ticket #8444 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Improve the way we do reduction of multi period data

Reported by:	Gesner Passos	Owned by:	Gesner Passos
Priority:	major	Milestone:	Release 3.1
Component:	SANS	Keywords:
Cc:	peter.parker@…	Blocked By:	#8443
Blocking:	#8505	Tester:	Peter Parker

Description

This will be an important step for LARMOR which intends to have multi-period event files for their spin echo experiements.

Change History

comment:1 Changed 7 years ago by Gesner Passos

Status changed from new to inprogress

comment:2 Changed 7 years ago by Gesner Passos

We should be able to execute this script and get as output a group workspace with the reduced data.

from ISISCommandInterface import * 
SANS2D()
Set1D()
Detector("rear-detector")
MaskFile('MASKSANS2Doptions.091A')
Gravity(True)
AssignSample('5512')
reduced = WavRangeReduction()

Last edited 7 years ago by Gesner Passos (previous) (diff)

comment:3 Changed 7 years ago by Gesner Passos

re #8444: Allow reduction of multiperiod data

Changeset: bdd2be07c321298fede76d8e2bee6b4bfdc374c7

comment:4 Changed 7 years ago by Gesner Passos

re #8444: Rename variable wksp_name

Changeset: f9280bf50b5133df3e52ed11272ab2b5ff8bd6d5

comment:5 Changed 7 years ago by Gesner Passos

re #8444: Revise LoadRun and LoadSample to support group workspace

Changeset: 8f7af6853b7f5f300ebff66fd5fe780bfeb7f867

comment:6 Changed 7 years ago by Gesner Passos

re #8444: Do not use protected attribute _period outside LoadRun

Improve the way we get the period name

Changeset: 3052ea90890902ba33b827f538709b8c3c0df59d

comment:7 Changed 7 years ago by Gesner Passos

re #8444: Move check outside LoadSample

LoadSample itself should not care about this. This will allow the creation of the tests on the load as well.

Changeset: 1906137d17819b3d65db9b6af90df16450585877

comment:8 Changed 7 years ago by Gesner Passos

re #8444: Support multiperiod reduction

Changeset: d2255c869153e9fbf571fdb676746bf4c43a3ed9

comment:9 Changed 7 years ago by Gesner Passos

re #8444: Correct the usage of period in the output name

The output name should show period only if the workspace has many periods or if the user defined which period to run.

Changeset: ffab3157867d1ca8b170a7da2bc1a167692a6a39

comment:10 Changed 7 years ago by Gesner Passos

re #8444: Ensure that suffix is correctly applied

Changeset: 65592a63450b3326be2c7c76645ecad34222b355

comment:11 Changed 7 years ago by Gesner Passos

re #8444: LoadRun corrections

1 - It corrects the access to multiperiod data. It uses the _index_of_group and not _period which is not changed internally by LoadRun (keeps memory of the way it was initialised) 2 - It corrects the wrong assumpition that the datafile will have the runnumber correctly placed. It asks the workspace its runnumber.

Changeset: 320ca8de73586f57a0b2f7b8cc886a42b854d728