Ticket #3065 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

DiffractionFocussing support for Q

Reported by: Peter Peterson Owned by: Nick Draper
Priority: major Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

DiffractionFocussing should support input workspaces in Q (read: momentum transfer) as well as d-spacing since they have some requirements on the data being commensurate.

This will help with processing data for PDF reduction.

Change History

comment:1 Changed 9 years ago by Wenduo Zhou

  • Status changed from new to verify
  • Resolution set to fixed

(In [12559]) Changed DiffractionFocussing2 such that it can support Q-space data. Fixes #3065.

comment:2 Changed 9 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:3 Changed 9 years ago by Janik Zikovsky

  • Status changed from verifying to reopened
  • Resolution fixed deleted

The following scripts shows a fairly significant difference between the Workspace2D and event versions of the algorithm:

# 2D version
LoadEventNexus(Filename="/home/8oz/data/PG3_1370_event.nxs",OutputWorkspace="pg3",CompressTolerance="0.050000000000000003")
ConvertUnits(InputWorkspace="pg3",OutputWorkspace="pg3",Target="MomentumTransfer")
Rebin(InputWorkspace="pg3",OutputWorkspace="pg3",Params="0,1,40")
CreateGroupingWorkspace(InstrumentName="PG3",GroupNames="bank124,bank144,bank164,bank184",OutputWorkspace="pg3_group")
ConvertToMatrixWorkspace(InputWorkspace="pg3",OutputWorkspace="pg32d")
DiffractionFocussing(InputWorkspace="pg32d",OutputWorkspace="pg3_foc_2d",GroupingWorkspace="pg3_group")

# Event version
CreateGroupingWorkspace(InstrumentName="PG3",GroupNames="bank124,bank144,bank164,bank184",OutputWorkspace="pg3_group")
DiffractionFocussing(InputWorkspace="pg3",OutputWorkspace="pg3_foc",GroupingWorkspace="pg3_group")

Minus("pg3_foc", "pg3_foc_2d", "pg3_diff")

If you compare pg3_foc and pg3_foc_2d on a plot, they look pretty different. I'm not sure if this is specific to Q; or if it has something to do with Martyn's "fixes" to spectrum numbers. Maybe this should reopen a new ticket?

comment:4 Changed 9 years ago by Nick Draper

  • Status changed from reopened to accepted
  • Owner changed from Wenduo Zhou to Nick Draper

New ticket #3285 created to deal with this.

comment:5 Changed 9 years ago by Nick Draper

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

comment:6 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester changed from Janik Zikovsky to Owen Arnold

comment:7 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

Issues raised by Janik Zikovsky in dedicated ticket #3285

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3912

Note: See TracTickets for help on using tickets.