Segfault when trying to display an instrument without sample set

[mantid-builder]

This issue was originally TRAC 9206

Original Reporter: Arturs Bekasovs

Save the attached file, which is a very minimal custom IDF, and run the following script, setting the right path:

idf = ".../IDF_for_locations_test.xml" # Set to the actual path
ws = CreateWorkspace([1,2,3], [1,2,3])
LoadInstrument(ws, idf)
getInstrumentView("ws")

Adding following lines to the IDF fixes the problem:

  `<component type="sample">`
    `<location/>`
  `</component>`
  `<type name="sample" is="samplePos"/>`

I don't think it's a major problem, since all the real instruments will have samples set. Seems to be a simple missing null-check somewhere in the InstrumentView code.

[stuartcampbell]

http://trac.mantidproject.org/mantid/raw-attachment/ticket/9206/IDF_for_locations_test.xml
(uploaded by Arturs Bekasovs on 2014-03-19T16:25:02)

[NickDraper]

A sample pos is vital to an instrument definition.
error handling could be better but very rare/unlikely - closed

wontfix