Ticket #2954 (closed: fixed)
Integrate Peaks using Normal Fitting
Reported by: | Ruth Mikkelson | Owned by: | Ruth Mikkelson |
---|---|---|---|
Priority: | major | Milestone: | Iteration 30 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Vickie Lynch |
Description
The peak data for a time slice is fit to a background +Intensity*bivariate normal distribution. This integration is the FitPeaks method used by ISAW.
Change History
comment:10 Changed 9 years ago by Ruth Mikkelson
comment:11 Changed 9 years ago by Ruth Mikkelson
comment:12 Changed 9 years ago by Ruth Mikkelson
comment:13 Changed 9 years ago by Ruth Mikkelson
comment:14 Changed 9 years ago by Ruth Mikkelson
comment:15 Changed 9 years ago by Ruth Mikkelson
comment:16 Changed 9 years ago by Ruth Mikkelson
comment:17 Changed 9 years ago by Ruth Mikkelson
comment:18 Changed 9 years ago by Ruth Mikkelson
comment:19 Changed 9 years ago by Ruth Mikkelson
- Status changed from new to accepted
- Milestone changed from Iteration 29 to Iteration 30
comment:20 Changed 9 years ago by Ruth Mikkelson
(In [12955]) Introduced some debug logging in the BivariateNormal class. Added a test program and sample program for IntegratePeakTimeSlices Tweaked some heuristics to better match the implementation in ISAW. Added two output properties and added some input verification with error reporting to the Logger Added the time, channel, start/end rows/columns in the table output for IntegratePeakTimeSlices
Refs #2954
comment:21 Changed 9 years ago by Ruth Mikkelson
comment:22 Changed 9 years ago by Janik Zikovsky
In [13234]:
comment:23 Changed 9 years ago by Ruth Mikkelson
In [13365]:
comment:24 Changed 9 years ago by Ruth Mikkelson
In [13368]:
comment:25 Changed 9 years ago by Ruth Mikkelson
In [13489]:
comment:26 Changed 9 years ago by Ruth Mikkelson
In [13490]:
comment:27 Changed 9 years ago by Ruth Mikkelson
In [13491]:
comment:28 Changed 9 years ago by Mathieu Doucet
In [13496]:
comment:29 Changed 9 years ago by Russell Taylor
There are a number of changes that would make this class tidier and more C++-like:
- The static functions and variables should become private members of the IntegratePeakTimeSlices class.
- The #define should be replaced with enums.
- As AttrNames & ParamNames appear to be fixed size - consider using C arrays which can be initialized at construction. It's possible these could also become enums as well (depends how they're used).
- Avoid std::vector::push_back where possible. At a minimum try to pre-size the vector. Not so important for small arrays, but good practice.
comment:30 Changed 9 years ago by Ruth Mikkelson
In [13680]:
comment:31 Changed 9 years ago by Ruth Mikkelson
In [13681]:
comment:32 Changed 9 years ago by Ruth Mikkelson
In [13684]:
comment:33 Changed 9 years ago by Ruth Mikkelson
In [13690]:
comment:34 Changed 9 years ago by Ruth Mikkelson
In [13851]:
comment:35 Changed 9 years ago by Ruth Mikkelson
In [13854]:
comment:36 Changed 9 years ago by Ruth Mikkelson
In [13985]:
comment:37 Changed 9 years ago by Ruth Mikkelson
- Status changed from accepted to verify
- Resolution set to fixed
comment:38 Changed 9 years ago by Vickie Lynch
- Status changed from verify to verifying
- Tester set to Vickie Lynch
comment:39 Changed 9 years ago by Vickie Lynch
- Status changed from verifying to closed
Works for single peak and also when called by PeakIntegration for all peaks.
comment:40 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3801
(In [11323]) The IFitFunction for BivariateNormal fit to a peak in a time slice. Refs #2954