Ticket #11696 (closed: fixed)
BinMD/SliceMD info (Beta Test issue)
Reported by: | Owen Arnold | Owned by: | |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.4 |
Component: | Framework | Keywords: | vates |
Cc: | Blocked By: | ||
Blocking: | Tester: | Anton Piccardo-Selg |
Description
Following mantid-help beta test report by Jon Taylor 4th/May/2015 regarding the sliceviewer.
The problem is that for axis-aligned cuts, you can actually reduce the dimensionality of the output workspace compared to the input by ommiting the axis aligned binning information for some dimensions. If you try to slice again, then BinMD correctly identifies that the original workspace does not have the same dimensionality as the output. The following shows how this sort of slicing can be done.
CreateMDWorkspace(Dimensions=3, Extents='-10,10,-10,10,-10,10', Names='Q_lab_x,Q_lab_y,Q_lab_z', Units='A,A,A', OutputWorkspace='WS') FakeMDEventData(InputWorkspace='WS', PeakParams='10000,0,0,0,0.1', RandomizeSignal=True) BinMD(InputWorkspace='WS', AlignedDim0='Q_lab_x,-10,10,10', AlignedDim1='Q_lab_y,-10,10,10', OutputWorkspace='binned') print 'Dims before', mtd['WS'].getNumDims() print 'Dims after', mtd['binned'].getNumDims()
Change History
comment:2 Changed 5 years ago by Owen Arnold
- Status changed from new to inprogress
refs #11696. Emit warning
Changeset: e0a07b02ea758cf16bc1d8f9ae722545681fc9ca
comment:3 Changed 5 years ago by Owen Arnold
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #711.
comment:4 Changed 5 years ago by Anton Piccardo-Selg
- Status changed from verify to verifying
- Tester set to Anton Piccardo-Selg
comment:5 Changed 5 years ago by Anton Piccardo-Selg
The script produces a rebinned output and creates the warning regarding lower dimensional cuts.
comment:6 Changed 5 years ago by Anton Piccardo-Selg
- Status changed from verifying to closed
Merge pull request #711 from mantidproject/11696_slicing_algorithm_warning
BinMD/SliceMD axis-aligned warning
Full changeset: 8ad9300cb3eeb852c30c22e1fcee7a925b20eec6
After discussion with Jon Taylor, it was agreed that the best course of action would be to warn if this sort of cut was being made.