Ticket #4069 (closed: fixed)
Split out execTOFevent method of AlignDetectors into a separate algorithm
| Reported by: | Russell Taylor | Owned by: | Wenduo Zhou |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 2.0 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Vickie Lynch |
Description
As it's really not doing the same thing as the other ways of running this algorithm.
Also, it's only for EventWorkspaces AND RIGHT NOW CRASHES MANTID IF GIVEN A WORKSPACE2D!
Change History
comment:2 Changed 9 years ago by Wenduo Zhou
Split TOF-only code from AlignDetectors to AlignDetectorInTOF. Refs #4069.
Changeset: d527847e0bbd421726be7d0db0ec976dad5af341
comment:4 Changed 9 years ago by Wenduo Zhou
- Status changed from accepted to verify
- Resolution set to fixed
comment:5 Changed 9 years ago by Vickie Lynch
- Status changed from verify to verifying
- Tester set to Vickie Lynch
comment:6 Changed 9 years ago by Vickie Lynch
Intensities are higher after using AlignDetectorsInTOF. Comment for CalibrationFile input should be changed since it is not optional, not a cal extension and there is not an option for an offsets workspace.
comment:7 Changed 9 years ago by Wenduo Zhou
Input calibration file does use '.dat' as the extension, because it is not the same type of calibration file defined in Mantid. It is VULCAN's raw calibration file.
The option of offset workspace will be treated as a new request, and put to a ticket in next iteration.
comment:8 Changed 9 years ago by Vickie Lynch
- Status changed from verifying to closed
Works for vulcan data. More accurate hint for CalibrationFile. I put the new input for this algorithm into the wiki.
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4916
Split TOF-only code from AlignDetectors to AlignDetectorInTOF. Refs #4069.