Ticket #3286 (closed: fixed)
PDFFourierTransform improvements
Reported by: | Nick Draper | Owned by: | Wenduo Zhou |
---|---|---|---|
Priority: | critical | Milestone: | Iteration 30 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Vickie Lynch |
Description
- The properties RMax, DeltaR, Qmin, Qmax, and DebugMode need to be documented (in the property declarations) including the fact that Qmin and Qmax default to the data range.
- The documentation for the InputWorkspace does not state that it can be in Q or d.
- The printing that is hiding in #if 0 should be converted to logging at debug level.
- Remove the DebugMode property.
- Rename OutputWorkspace0 and OutputWorkspace1 to be something meaningful. In other algorithms you will see one output workspace property and the algorithm generates two names.
- Trim the Q(S(Q)-1) Q range to be that selected to be Fourier transformed.
Change History
comment:5 Changed 9 years ago by Wenduo Zhou
PDFFourierTransform's wiki page (http://www.mantidproject.org/PDFFourierTransform) is improved for more detailed and accurate information.
comment:7 Changed 9 years ago by Wenduo Zhou
- Status changed from accepted to verify
- Resolution set to fixed
comment:10 Changed 9 years ago by Wenduo Zhou
In [14565]:
comment:11 Changed 9 years ago by Russell Taylor
In [14721]:
comment:12 Changed 9 years ago by Vickie Lynch
- Status changed from verify to verifying
- Tester set to Vickie Lynch
comment:13 Changed 9 years ago by Vickie Lynch
- Status changed from verifying to closed
Seems to work correctly.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4133
Note: See
TracTickets for help on using
tickets.
In [13327]: