Ticket #4178 (closed: fixed)
Add getxmin and getxmax to MatrixWorkspace
| Reported by: | Peter Peterson | Owned by: | Peter Peterson |
|---|---|---|---|
| Priority: | major | Milestone: | Release 2.0 |
| Component: | Mantid | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Jose Borreguero |
Description
Often enough it is useful to get the minimum and maximum x-value of a workspace. Rather than coding it everywhere add it to the MatrixWorkspace directly. Also, have a second method that gives the min/max of the events, which can be different from the histogram representation.The new functions should be called:
- getXMin
- getXMax
- getXMinMax(double &, double &)
- getEventXMin
- getEventXMax
- getEventXMinMax(double &, double &)
Change History
comment:1 Changed 9 years ago by Peter Peterson
- Status changed from new to accepted
- Owner set to Peter Peterson
comment:2 Changed 9 years ago by Peter Peterson
Force the compiler to use the const version. Refs #4178.
Changeset: 16b344e0366e9938a37785d94fbd7a2cf4f9dd1f
comment:3 Changed 9 years ago by Peter Peterson
Adding code in MatrixWorkspace to get min/max x-value. Refs #4178.
Changeset: bf026de84f350bd15775d0327199b46e9dfb6d8e
comment:4 Changed 9 years ago by Peter Peterson
Moving over to new MatrixWorkspace functions. Refs #4178.
Changeset: e367e0aba5b97f3a9194fe45b11c0fed6f0a2514
comment:5 Changed 9 years ago by Peter Peterson
Using the correct const version: readX. Refs #4178.
Changeset: 64e56cd9ce54633ca6acb1f0c88bf419ecbc3de2
comment:6 Changed 9 years ago by Peter Peterson
Adding additional requirements to ISpectrum. Refs #4178 and #2320.
Also with default implementations.
Changeset: a5cb768144dc3a8abe8cc6887c04779a5d7c50f4
comment:7 Changed 9 years ago by Peter Peterson
Renaming variables and using central code. Refs #4178 and #3866.
Member variables now start with 'm_' and changed to central code for finding min and max x-values.
Changeset: f95d505545a861e9489b917aa3708e38edb9d8f8
comment:8 Changed 9 years ago by Peter Peterson
Copied code from ConvertUnits. Refs #4178.
Changeset: 2a5c41b213934cee11feaa8ac570f190e1facea2
comment:9 Changed 9 years ago by Peter Peterson
Adding new methods and redirecting old to new. Refs #4178.
Changeset: 584f76d8328bf043bf1114519187bbacf0c50601
comment:10 Changed 9 years ago by Peter Peterson
Added doxygen and small optimizations to the code. Refs #4178.
Changeset: 6da6bef490918be44cfdbf47abd6a5869b0f22bd
comment:11 Changed 9 years ago by Peter Peterson
Using centralized code for finding tof min. Refs #4178.
Changeset: f29dc7fe5bc35c4319905d64810a7c0a1029dcec
comment:12 Changed 9 years ago by Peter Peterson
Using centralized code for finding tof min. Refs #4178.
Changeset: 97f33a1eb95fa8bb6cc3c9edcebe0b25a90a4f89
comment:13 Changed 9 years ago by Peter Peterson
Using central code for x-range. Refs #4178.
Changeset: 9266eb660bd19a7c9bd65ba3d4b8fedabde005be
comment:14 Changed 9 years ago by Peter Peterson
- Status changed from accepted to verify
- Resolution set to fixed
comment:15 Changed 9 years ago by Janik Zikovsky
Refs #4178: ISpectrum fix for ManagedWorkspaces
Changeset: 66583a5e6b8ba3f93e59e239311ddba6865f24dd
comment:16 Changed 9 years ago by Jose Borreguero
- Status changed from verify to verifying
- Tester set to Jose Borreguero
comment:18 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 5025
