Ticket #9856 (closed: fixed)
Some minor improvements on GetDetOffsetsMultiPeaks
Reported by: | Wenduo Zhou | Owned by: | Wenduo Zhou |
---|---|---|---|
Priority: | major | Milestone: | Release 3.3 |
Component: | Documentation | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Harry Jeffery |
Description (last modified by Wenduo Zhou) (diff)
It is found that sometime most of the spectra can have exactly same fit windows. On the other hand, it takes long time to generate a TableWorkspace containing millions rows in such as for Corelli in python script. Therefore, it will be better for FitwindowTableWorkspace of GetDetOffsetsMultiPeaks accept universal fit windows for all detectors.
Improve the documentation of GetDetOffsetsMultiPeaks.
Change History
comment:1 Changed 6 years ago by Wenduo Zhou
- Status changed from new to assigned
- Description modified (diff)
- Summary changed from Improve the documentation of GetDetOffsetsMultiPeaks to Some minor improvements on GetDetOffsetsMultiPeaks
comment:3 Changed 6 years ago by Wenduo Zhou
Refs #9856. Made improvement on fit windows and doc.
Including
- Add support of default fit windows to FitwindowTableWorkspace, as
well as the unit test;
- Improve the doc.
Changeset: 7befc15ae13880404f024e44c4f36bd67f42dce8
comment:4 Changed 6 years ago by Wenduo Zhou
Made minor mod to doc. Refs #9856.
Changeset: 4c6524ad47b2f3f914de9cfea20fc811a9e51306
comment:5 Changed 6 years ago by Wenduo Zhou
Refs #9856. Made improvement on fit windows and doc.
Including
- Add support of default fit windows to FitwindowTableWorkspace, as
well as the unit test;
- Improve the doc.
Changeset: e9513593c7a1c423474a94012e95a0d613c0a834
comment:6 Changed 6 years ago by Wenduo Zhou
Made minor mod to doc. Refs #9856.
Changeset: d44087e8e60821b3a6f40d146deb5136b97db2ce
comment:7 Changed 6 years ago by Wenduo Zhou
- Status changed from inprogress to verify
- Resolution set to fixed
For testers
The change from this ticket is to enable the various sizes of peak fit window on different spectra. It is an expansion to original setup that all spectra have the same fit windows for peaks at same d-spacing. And the new feature is applied in the case that the resolution of different detectors varies significantly.
For testing, please look at the source codes and the new unit test for the new feature. You can make a workspace with multiple spectra, each of which contains 2 or 3 peaks but with different resolution. Then you can set up a table workspace for fit windows.
comment:8 Changed 6 years ago by Harry Jeffery
- Status changed from verify to verifying
- Tester set to Harry Jeffery
comment:9 Changed 6 years ago by Harry Jeffery
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/9856_getdetoffsets_minor_mod'
Full changeset: 29241d4e13bbd16c09b6084401dec4c693c95b44
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 10698