Ticket #6767 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

SliceViewer: integrate peaks not displayed if defaults used in IntegratePeaksMD

Reported by: Nick Draper Owned by: Owen Arnold
Priority: critical Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Gesner Passos

Description

do this:

  1. integrate a peak list with IntegratePeaksMD, but leave the Background inner and outer radius empty
  2. Look at it with sliceviewer
  3. The peaks are not visible

Change History

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

Tester: This is how you reproduce the issue.

1) Run the following.

# Basic parameters  for  Triphylite Crystal
#Name of the workspaces to create
ws_name = "TOPAZ_3132" # This file lives in the system test data directory
filename = ws_name +"_event.nxs"
ws = LoadEventNexus(Filename=filename,FilterByTofMin=3000, FilterByTofMax=16000)

# -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
# Part 1. Basic Reduction

# Spherical Absorption and Lorentz Corrections
ws = AnvredCorrection(InputWorkspace=ws, LinearScatteringCoef=0.451, LinearAbsorptionCoef=0.993, Radius=0.14)

# Convert to Q space
LabQ = ConvertToDiffractionMDWorkspace(InputWorkspace=ws, LorentzCorrection='0',
        OutputDimensions='Q (lab frame)', SplitInto=2, SplitThreshold=150)
        
# Find peaks
PeaksLattice = FindPeaksMD(InputWorkspace=LabQ,MaxPeaks=100)

# 3d integration to centroid peaks
PeaksLattice = CentroidPeaksMD(InputWorkspace=LabQ, CoordinatesToUse='Q (lab frame)',
        PeakRadius=0.12, PeaksWorkspace=PeaksLattice)
        
# Find the UB matrix using the peaks and known lattice parameters
FindUBUsingLatticeParameters(PeaksWorkspace=PeaksLattice, a=10.3522, b=6.0768, c=4.7276,
                alpha=90, beta=90, gamma=90, NumInitial=20, Tolerance=0.12)
                
# And index to HKL            
IndexPeaks(PeaksWorkspace=PeaksLattice, Tolerance=0.12)

2) Now run this.

PeaksLattice_Integrated = IntegratePeaksMD(InputWorkspace=LabQ,PeakRadius=0.12,
        BackgroundOuterRadius=0.18,BackgroundInnerRadius=0.15,
        PeaksWorkspace=PeaksLattice)

Plot the peaks in the SliceViewer and you should get spherical peaks. You should be able to switch on/off the background radius.

3) Now run this.

PeaksLattice_Integrated = IntegratePeaksMD(InputWorkspace=LabQ,PeakRadius=0.12,
        PeaksWorkspace=PeaksLattice)

Plot the peaks in the SliceViewer and you get a grey screen!

comment:3 Changed 8 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:4 Changed 8 years ago by Owen Arnold

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

Tester: Use the python scripts above to test. Rather than giving a grey screen at step 3. You should get a properly rendered spherical peak region. Showing the peak background (using the checkbox) should have no effect.

branch is: bugfix/6767_integrated_peaks

comment:5 Changed 8 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:6 Changed 8 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:7 Changed 8 years ago by Gesner Passos

  • Status changed from verifying to closed

as described, now, no grey screen appears at step 3, but the proper answer.

comment:8 Changed 7 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:9 Changed 7 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:10 Changed 7 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:11 Changed 7 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:12 Changed 7 years ago by Owen Arnold

refs #6767. Fix and unit test.

Changeset: 1179322a61e784beb5db3e29d57b0826ec74414d

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7613

Note: See TracTickets for help on using tickets.