Ticket #1862 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Check Diagnose Detectors (diag) is working for SNS instruments

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: critical Milestone: Iteration 26
Component: Mantid Keywords:
Cc: Martyn, Gigg Blocked By:
Blocking: Tester: Janik Zikovsky

Description

Need to make sure that the "Diagnose Detectors" tab works correctly (and the mask is used) for the SNS direct geometry beamlines.

Attachments

CNCS_8045_data.png (123.0 KB) - added by Stuart Campbell 10 years ago.
CNCS_8045_mask.png (14.0 KB) - added by Stuart Campbell 10 years ago.

Change History

comment:1 Changed 10 years ago by Stuart Campbell

  • Status changed from new to accepted
  • Owner set to Stuart Campbell

comment:2 Changed 10 years ago by Stuart Campbell

  • Cc Martyn, Gigg added

Seems to crash if you select "Check Background"

comment:3 Changed 10 years ago by Stuart Campbell

comment:4 Changed 10 years ago by Stuart Campbell

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

It seems to be working at first inspection. The mask generated looks roughly what you would expect.

comment:5 Changed 10 years ago by Stuart Campbell

  • Status changed from verify to reopened
  • Resolution fixed deleted

Changed 10 years ago by Stuart Campbell

Changed 10 years ago by Stuart Campbell

comment:6 Changed 10 years ago by Stuart Campbell

  • Status changed from reopened to accepted

I have added a couple of attachments, one shows the data from a vanadium measurement and the other is the mask that is calculated.

There is reasonable agreement between them.

comment:7 Changed 10 years ago by Stuart Campbell

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

comment:8 Changed 10 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:9 Changed 10 years ago by Janik Zikovsky

  • Status changed from verifying to closed

Crashes if this is run against an Event file... Ticket #2073 created for it. When run on histogrammed data, it looks reasonable.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2709

Note: See TracTickets for help on using tickets.