Ticket #7139 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

SliceViewer crashes with elipsoidal integration

Reported by: Vickie Lynch Owned by: Vickie Lynch
Priority: major Milestone: Release 2.5.3
Component: Mantid Keywords: Patch,Candidate,Released
Cc: petersonpf@… Blocked By:
Blocking: Tester: Russell Taylor

Description

Run MantidEv and use Elipsoidal Integration. Start SliceViewer and load PeaksWorkspace. MantidPlot will crash with error "Unknown property search object PeakRadius"

Change History

comment:1 Changed 7 years ago by Vickie Lynch

  • Owner set to Vickie Lynch
  • Status changed from new to accepted

comment:2 Changed 7 years ago by Vickie Lynch

Refs #7139 Add radii of the major axis of the ellipsoidal peak and bkg

Changeset: 3b0d578df9ed54a9b912db7d08e792fe2b95528d

comment:3 Changed 7 years ago by Vickie Lynch

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

No longer crashes, but shows circle. Ticket #7141 is to draw ellipsoid

comment:4 Changed 7 years ago by Vickie Lynch

  • Keywords Patch Candidate added
  • type changed from enhancement to defect
  • Milestone changed from Release 2.6 to Release 2.5.3

comment:5 Changed 7 years ago by Vickie Lynch

To Test, run script below, right click on TOPAZ_6674_md and select Show_Slice_Viewer. Click on yellow star in slice viewer and select TOPAZ+6674_peaks. If no crash, bug is fixed and you can select peaks from peaks workspace and see circles around peaks. 

######################################################################
#Python Script Generated by GeneratePythonScript Algorithm
######################################################################
Load(Filename='TOPAZ_6674_event.nxs',OutputWorkspace='TOPAZ_6674_event',Precount='1',LoadMonitors='1')
ConvertToMD(InputWorkspace='TOPAZ_6674_event',OutputWorkspace='TOPAZ_6674_md',QDimensions='Q3D',dEAnalysisMode='Elastic',MinValues='-25,-25,-25',MaxValues='25,25,25',SplitInto='2',SplitThreshold='50',MaxRecursionDepth='13',MinRecursionDepth='7')
FindPeaksMD(InputWorkspace='TOPAZ_6674_md',PeakDistanceThreshold='0.37680000000000002',MaxPeaks='50',OutputWorkspace='TOPAZ_6674_peaks')
FindUBUsingFFT(PeaksWorkspace='TOPAZ_6674_peaks',MinD='3',MaxD='15',Tolerance='0.12')
IndexPeaks(PeaksWorkspace='TOPAZ_6674_peaks',Tolerance='0.12',RoundHKLs='0')
ShowPossibleCells(PeaksWorkspace='TOPAZ_6674_peaks',BestOnly='0')
SelectCellWithForm(PeaksWorkspace='TOPAZ_6674_peaks',FormNumber='9',Apply='1')
IntegrateEllipsoids(InputWorkspace='TOPAZ_6674_event',PeaksWorkspace='TOPAZ_6674_peaks',RegionRadius='0.25',SpecifySize='1',PeakSize='0.16',BackgroundInnerSize='0.161',BackgroundOuterSize='0.20999999999999999',OutputWorkspace='TOPAZ_6674_peaks')

comment:6 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:7 Changed 7 years ago by Russell Taylor

Merge remote-tracking branch 'origin/feature/7139_SliceViewer_Crash' into master

comment:8 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

The exception is gone.

comment:9 Changed 7 years ago by Vickie Lynch

Refs #7139 Add radii of the major axis of the ellipsoidal peak and bkg

Changeset: d542b88e42954e4840feace9a5a72118ee225996

comment:10 Changed 7 years ago by Vickie Lynch

Created 2nd branch for this ticket from release2.5: release2.5/7139_SliceViewer_Crash

comment:11 Changed 7 years ago by Vickie Lynch

Merge branch 'feature/7139_SliceViewer_Crash' into develop into 6856_ConvertToDiffractionMDWS_v2

comment:12 Changed 7 years ago by Vickie Lynch

Merge branch 'feature/7139_SliceViewer_Crash' into develop into 6856_ConvertToDiffractionMDWS_v2

comment:13 Changed 7 years ago by Vickie Lynch

Refs #7139 Add radii of the major axis of the ellipsoidal peak and bkg

Changeset: f2916016f54edd72eb6ce2904aa687b93ea280f2

comment:14 Changed 7 years ago by Nick Draper

  • Keywords Patch,Candidate added; Patch Candidate removed

comment:15 Changed 7 years ago by Nick Draper

  • Keywords Patch,Candidate,Released added; Patch,Candidate removed

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7985

Note: See TracTickets for help on using tickets.