Ticket #6558 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

SANS: Load Event file does not show the rectangular banks

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: critical Milestone: Release 2.5
Component: Mantid Keywords:
Cc: anders.markvardsen@… Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Loading Nexus Event files for SANS2D does not show the bank detectors.

This code will fail for Load, but will not fail for LoadNexus.

OBS: it may fail for LoadNexus if you tried once Load as well. 

#SET INSTRUMENT AS ISIS->SANS2D
#wd = LoadNexus('15163')
wd = Load('15163')
assert wd.getInstrument().nelements() == 5

Tester:

Make sure that this scripts pass.

Attachments

6558_event_files.patch​ (759 bytes) - added by Gesner Passos 8 years ago.
Patch that applies the deleteBank only for TOPAZ, in agreement with Vickie

Change History

comment:1 Changed 8 years ago by Gesner Passos

  • Owner set to Gesner Passos
  • Status changed from new to accepted

comment:2 Changed 8 years ago by Gesner Passos

  • Owner changed from Gesner Passos to Martyn Gigg
  • Status changed from accepted to assigned

This error was seen after the implementation of #6350.

The ISIS nexus file for SANS2D has only one detector_1 defined, for the two rectangular banks: front-detector, rear-detector.

Removing the deleteBanks (LoadEventNexus:1519) solves this ticket, but breaks the #6350.

Requires a decision on how to maintain both:

  • Apply deleteBanks only for TOPAZ instruments?
  • Avoid deleteBanks for SANS2D instruments?

Changed 8 years ago by Gesner Passos

Patch that applies the deleteBank only for TOPAZ, in agreement with Vickie

comment:3 Changed 8 years ago by Gesner Passos

  • Owner changed from Martyn Gigg to Gesner Passos
  • Status changed from assigned to accepted

comment:4 Changed 8 years ago by Gesner Passos

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

The solution of #6568 also corrects this issue.

comment:5 Changed 8 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:6 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to closed

wd = Load('15163') assert wd.getInstrument().nelements() == 5

worked and also now you can see the instrument in instrument view

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7404

Note: See TracTickets for help on using tickets.