Ticket #5447 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

GroupDetectors does not correctly handle the mask in the average mode

Reported by: Andrei Savici Owned by: Andrei Savici
Priority: minor Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

The following script should get an instrument view with only ones or masked pixels. In the current version you get 4s, 3s, ... 

GenerateGroupingSNSInelastic(AlongTubes='4',AcrossTubes='1',Instrument='CNCS',Filename='CNCS4x1.xml')
Load(Filename='CNCS_7860_event.nxs',OutputWorkspace='cal')
Rebin(InputWorkspace='cal',OutputWorkspace='cal',Params='44000,17000,61000',PreserveEvents='0')
MedianDetectorTest(InputWorkspace='cal',OutputWorkspace='mask',ExcludeZeroesFromMedian='1')
MaskDetectors(Workspace='cal',MaskedWorkspace='mask')
Load(Filename='CNCS_7860_event.nxs',OutputWorkspace='data')
Rebin(InputWorkspace='data',OutputWorkspace='data',Params='44000,1700,61000',PreserveEvents='0')
Divide(LHSWorkspace='data',RHSWorkspace='cal',OutputWorkspace='div')
GroupDetectors(InputWorkspace='div',OutputWorkspace='gr',MapFile='CNCS4x1.xml',Behaviour='Average')

Change History

comment:1 Changed 8 years ago by Andrei Savici

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Andrei Savici

Fix average grouping with masked spectra. Refs #5447

Changeset: 838e65425cf28ee9ac33efe3d03bd1d1a8a65904

comment:3 Changed 8 years ago by Andrei Savici

Fix average grouping with masked spectra. Refs #5447

Changeset: 838e65425cf28ee9ac33efe3d03bd1d1a8a65904

comment:4 Changed 8 years ago by Andrei Savici

Fix average grouping with masked spectra. Refs #5447

Changeset: 838e65425cf28ee9ac33efe3d03bd1d1a8a65904

comment:5 Changed 8 years ago by Andrei Savici

Fix average grouping with masked spectra. Refs #5447

Changeset: 838e65425cf28ee9ac33efe3d03bd1d1a8a65904

comment:6 Changed 8 years ago by Andrei Savici

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

comment:7 Changed 8 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 8 years ago by Michael Reuter

  • Status changed from verifying to closed

This seems to be working.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6293

Note: See TracTickets for help on using tickets.