Ticket #7345 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

An exception is thrown while running detector calibration on PG3 run 13428

Reported by: Wenduo Zhou Owned by: Wenduo Zhou
Priority: critical Milestone: Release 2.6
Component: Framework Keywords:
Cc: petersonpf@… Blocked By:
Blocking: Tester: Peter Peterson

Description (last modified by Wenduo Zhou) (diff)

As running CalibrateRectangularDetectors on POWGEN run 13428, an exception was thrown. Here is the error message: GetDetOffsetsMultiPeaks: No idea how to deal with this! Spectrum 285: Find peak between 0.5(i = 0) and 0.583461(i = 386). Error reason: Estimate peak parameters error (FWHM cannot be zero): Input data size = 387, Xmin = 0.5(0), Xmax = 0.583461(386); Estimated peak centre @ 0.5004(2) with height = 21.3333; Lowest Y value = 0; Output error: . leftfwhm = -1, right fwhm = 0.00010008.

Attachments

error.png (2.6 MB) - added by Wenduo Zhou 7 years ago.
ticket7345.py (1.1 KB) - added by Wenduo Zhou 7 years ago.

Change History

comment:1 Changed 7 years ago by Wenduo Zhou

It is found that the error is caused by an ill-defined FitWindow (see the attached error.png). There are in fact more than 1 peak inside the window.

It is decided to make FitPeaks to deal with it as a non-fit situation.

Changed 7 years ago by Wenduo Zhou

comment:2 Changed 7 years ago by Wenduo Zhou

Fixed the bug. Refs #7345.

Changeset: cb1656b750fa72195d2c8bad6ca852d8d263d9b6

comment:3 Changed 7 years ago by Wenduo Zhou

Verification Instruction

  1. Locate data PG3_13428_event.nxs
  2. Download the attached ticket7345.py
  3. Run the script to see whether there is any exception thrown

Changed 7 years ago by Wenduo Zhou

comment:4 Changed 7 years ago by Wenduo Zhou

  • Status changed from new to accepted

comment:5 Changed 7 years ago by Wenduo Zhou

  • Status changed from accepted to verify
  • Resolution set to fixed
  • Description modified (diff)

comment:6 Changed 7 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:7 Changed 7 years ago by Peter Peterson

  • Status changed from verifying to closed

comment:8 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8191

Note: See TracTickets for help on using tickets.