Ticket #5594 (closed: fixed)
Fix error in PeakIntegration
| Reported by: | Ruth Mikkelson | Owned by: | Ruth Mikkelson |
|---|---|---|---|
| Priority: | major | Milestone: | Release 2.3 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Dennis Mikkelson |
Description
PeakIntegration currently does not work in the case of fit slices when more than one run number is in the peaksworkspace
Change History
comment:2 Changed 8 years ago by Ruth Mikkelson
- Status changed from accepted to verify
- Resolution set to fixed
commit 2ccf2f7878e3df2597ab9db638f5e706e94bc03f Author: Ruth Mikkelson <mikkelsonr@…> Date: Mon Jul 9 08:24:35 2012 -0500
Refs #5594 Fixed error in PeakIntegration
comment:3 Changed 8 years ago by Ruth Mikkelson
Refs #5594 Fixed error in PeakIntegration
Changeset: 821cf8964f5817af9ce290396ff5235f9e87569c
comment:4 Changed 8 years ago by Ruth Mikkelson
Refs #5594 Fixed error in PeakIntegration
Changeset: 821cf8964f5817af9ce290396ff5235f9e87569c
comment:5 Changed 8 years ago by Vickie Lynch
- Status changed from verify to verifying
- Tester set to Vickie Lynch
comment:6 Changed 8 years ago by Vickie Lynch
- Status changed from verifying to verify
- Tester Vickie Lynch deleted
comment:7 Changed 8 years ago by Dennis Mikkelson
- Status changed from verify to verifying
- Tester set to Dennis Mikkelson
comment:8 Changed 8 years ago by Dennis Mikkelson
- Status changed from verifying to closed
Two TOPAZ runs, 5637 and 5638 were loaded, rebinned, converted to diffraction MD workspaces. I then found peaks in both workspaces and merged the workspaces by saving the peaks to a file and appending. After loading the merged peaks list, it was used with the PeakIntegration algorithm to integrate peaks from each run, placing the results from each run in separate new peaks workspaces. The new peaks workspaces now have the correct peaks and the correct integrated intensities.
NOTE: It would be convenient if the output of PeakIntegration could be written to the original peaks workspace rather than having to create new output peaks workspaces to hold the integrated intensities.
