Ticket #9399 (closed: fixed)
Add optional tableworkspace containing peak range to GetDetOffsetsMultiPeaks
Reported by: | Wenduo Zhou | Owned by: | Wenduo Zhou |
---|---|---|---|
Priority: | major | Milestone: | Release 3.2 |
Component: | Framework | Keywords: | |
Cc: | petersonpf@… | Blocked By: | |
Blocking: | Tester: | Peter Peterson |
Description
Add an optional input TableWorkspace to GetDetOffsetsMultiPeaks's properties. The TableWorkspace should contain the range of all peaks of all spectra for calibration.
GetDetOffsetsMultiPeaks then will parse the table workspace and pass the peak fitting range to FindPeaks.
Attachments
Change History
comment:2 Changed 6 years ago by Wenduo Zhou
- Status changed from assigned to inprogress
Refs #9399. Added a TableWorkspace to input fit windows.
also including a unit test for the new feature.
Changeset: 82dc63a84d566d141ebb79f18a2876ff70303e56
comment:3 Changed 6 years ago by Wenduo Zhou
- Status changed from inprogress to verify
- Resolution set to fixed
For tester
- Download the attached python script, which will calibrate NOMAD with (new way) and without (old way) using Fit-Window-TableWorkspace;
- Build mantid and run script;
- Compare the 2d plots of the calibrated workspaces of 2 approaches;
- The new approach will mask more spectra, for example, 75949, 75956, 75863 and 95961. You will find that these spectra are too noisy to fit peaks. The old approach just does some false fitting.
comment:4 Changed 6 years ago by Peter Peterson
- Status changed from verify to verifying
- Tester set to Peter Peterson
comment:5 Changed 6 years ago by Pete Peterson
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/9399_fitwindow_tablews'
Full changeset: 3d366091d087051f83490227debb753fa52efd50
Note: See
TracTickets for help on using
tickets.