mantid icon indicating copy to clipboard operation
mantid copied to clipboard

Instrument view crashes when overlaying peaks in Side by Side view

Open RichardWaiteSTFC opened this issue 2 years ago • 1 comments

Original reporter: Michael Ritchie (SXD student)

Describe the bug Instrument view hard-crashes when overlaying peaks in side-by-side view (at least for SXD data).

To Reproduce (1) Run this script to load a run (from system test data) and make peaks

Load(Filename=r'SXD23767.raw', OutputWorkspace='SXD23767')
CreatePeaksWorkspace(InstrumentWorkspace='SXD23767', NumberOfPeaks=0, OutputWorkspace='SingleCrystalPeakTable')
AddPeak(PeaksWorkspace='SingleCrystalPeakTable', RunWorkspace='SXD23767', TOF=3271, DetectorID=32615)

(2) Open SXD23767 in instrument view (right-click on ws in ADS > Show INstrument) (3) From the drop-down box at the top of the Render tab select Side by Side (4) Drag the SingleCrystalPeakTable ws from ADS into the instrument view window

Expected behavior It would be great to support peak overlay in Side by Side mode, from memory I don't think we have ever done this though. In which case I think we should disable the feature properly, or at least catch it and throw a warning.

Platform/Version (please complete the following information):

  • OS: Windows and IDAaaS
  • Mantid Version: Nightly 26-Aug, Manual testing release candidate

Additional context

RichardWaiteSTFC avatar Sep 13 '22 12:09 RichardWaiteSTFC

This crash also occurs in v6.4 and v6.3 so in the first instance we'll look to disable the feature for peak overlays.

sf1919 avatar Sep 14 '22 12:09 sf1919

I'd got as far as finding that it is throwing the error on this line but it doesn't seem to be handling that well https://github.com/mantidproject/mantid/blob/main/qt/widgets/instrumentview/src/PanelsSurface.cpp#L270

sf1919 avatar Sep 28 '22 09:09 sf1919

Fixed by #34460

mantid-builder avatar Oct 04 '22 10:10 mantid-builder