Ticket #11607 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

Mask POLDI detectors automatically using MedianDetectorTest

Reported by: Michael Wedel Owned by: Michael Wedel
Priority: major Milestone: Release 3.4
Component: Diffraction Keywords: POLDI
Cc: Blocked By:
Blocking: Tester: Federico Montesino Pouzols

Description (last modified by Michael Wedel) (diff)

Some substances produce gamma rays when irradiated by neutrons. The detectors at POLDI should be able to distinguish between gamma and neutron events, but when gamma intensities become too high, all bins of some spectra are systematically too high.

I would like to extend PoldiLoadRuns with an option to mask spectra that are not masked through the instrument definition yet but show derivations from detector median. The algorithm MedianDetectorTest performs exactly this type of analysis, so it just needs to be added as an extra step.

Change History

comment:1 Changed 5 years ago by Michael Wedel

  • Description modified (diff)

comment:2 Changed 5 years ago by Michael Wedel

  • Status changed from new to inprogress

Refs #11607. Modified PoldiLoadRuns, extended system test.

Changeset: e85b3a44d9bbd6f041cc77616c9213be1e472754

comment:3 Changed 5 years ago by Michael Wedel

Refs #11607. Additional parameter for setting threshold.

Also added paragraph to documentation.

Changeset: 4cb086ae7f179035ee20a2fa1c4d156cb58822e8

comment:4 Changed 5 years ago by Michael Wedel

Refs #11607. Fixing doc test.

New default behavior of PoldiLoadRuns changes result in the last digit.

Changeset: f88ad76e6df1b1efcf207668888f6bda50b1c0f8

comment:5 Changed 5 years ago by Michael Wedel

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

This is being verified as ​pull request #629.

comment:6 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from verify to verifying
  • Tester set to Federico Montesino Pouzols

comment:7 Changed 5 years ago by Federico Montesino Pouzols

All looks good and explained in the docs. Automated tests pass and from manual tests it seems to behave well, as I can see the increasing number of (masked) horizontal lines in the color plots.

comment:8 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from verifying to closed

Merge pull request #629 from mantidproject/11607_mask_poldi_spectra_automatically

Mask detectors in POLDI data automatically

Full changeset: ​79c33a07c3264ca46ab4b77222bfc3f426d97fa2

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​12445

Note: See TracTickets for help on using tickets.