Ticket #6546 (closed: worksforme)

Opened 8 years ago

Last modified 5 years ago

MergeMD does not keep instrument

Reported by: Andrei Savici Owned by: Andrei Savici
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

An MD workspace created with MergeMD does not have an instrument. This causes problem for FindPeaksMD. To test: 

Load(Filename='CNCS_7860_event.nxs',OutputWorkspace='CNCS_7860_event')
DgsReduction(SampleInputWorkspace='CNCS_7860_event',IncidentBeamNormalisation='ByCurrent',OutputWorkspace='w')
ConvertToMD(InputWorkspace='w',OutputWorkspace='md1',QDimensions='Q3D',QConversionScales='HKL',MinValues='-3,-3,-3,-3',MaxValues='3,3,3,3')
ConvertToMD(InputWorkspace='w',OutputWorkspace='md2',QDimensions='Q3D',QConversionScales='HKL',MinValues='-3,-3,-3,-3',MaxValues='3,3,3,3')
MergeMD(InputWorkspaces='md1,md2',OutputWorkspace='merged')

FindPeaksMD should not give error when input workspace is merged. For this particular data set the peaks workspace is empty

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:2 Changed 7 years ago by Nick Draper

  • Owner set to Alex Buts
  • Status changed from new to assigned

comment:3 Changed 7 years ago by Andrei Savici

  • Status changed from assigned to accepted
  • Owner changed from Alex Buts to Andrei Savici

comment:4 Changed 7 years ago by Andrei Savici

  • Status changed from accepted to verify
  • Resolution set to worksforme

This seems to be working now. Probably the changes for vates just before the release have solved this problem as well

comment:5 Changed 7 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:6 Changed 7 years ago by Martyn Gigg

  • Status changed from verifying to closed

Confirmed that FindPeaksMD, which requires an instrument, now works with the output of MergeMD.

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7392

Note: See TracTickets for help on using tickets.