Ticket #4796 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

MedianDetectorTest should optionally allow for SolidAngle

Reported by: Nick Draper Owned by: Andrei Savici
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: 5us@… Blocked By:
Blocking: Tester: Stuart Campbell

Description

Add a boolean parameter for whether to take the solid angle into account. Default = off.

Contact Matthew Stone when this is available for download. 5us@…

Change History

comment:1 Changed 9 years ago by Peter Peterson

This appears to be a repeat of #3742.

comment:2 Changed 9 years ago by Andrei Savici

  • Owner changed from Martyn Gigg to Andrei Savici
  • Status changed from new to accepted

comment:3 Changed 9 years ago by Andrei Savici

Re-enabled solid angle correction. Refs #3742. Refs #4796

Changeset: 8deaadb969839697d697d940b94c2ecdab45839c

comment:4 Changed 9 years ago by Andrei Savici

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

wiki updated. To test first load an empty instrument (ARCS). This will have all spectra at 1. Apply the algorithm with a very small significance limit (0.01). MAKE SURE you ignore monitors!!! For ARCS, when empty instrument was loaded, that means starting from spectrum 2. Without the solid angle correction, no detectors should be masked. If solid angle correction is applied, the two short 8packs should be masked (2048 pixels)

comment:5 Changed 9 years ago by Andrei Savici

Re-enabled solid angle correction. Refs #3742. Refs #4796

Changeset: 8deaadb969839697d697d940b94c2ecdab45839c

comment:6 Changed 8 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:7 Changed 8 years ago by Stuart Campbell

  • Status changed from verifying to closed

Looks good.

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5643

Note: See TracTickets for help on using tickets.