Ticket #7699 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

EQSANS: reducing existing workspace fails

Reported by: Mathieu Doucet Owned by: Mathieu Doucet
Priority: major Milestone: Release 3.0
Component: SANS Keywords: PatchCandidate
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

It used to be that we could reduce a data set that was already loaded using LoadEventNexus. The API v2 version of the EQSANS reduction no longer allows that and should be repaired.

Change History

comment:1 Changed 7 years ago by Mathieu Doucet

  • Status changed from new to inprogress

Re #7699 Fix issue with loaded data

Changeset: acfe2643fd93be2c94bc15432f5ca715c0353709

comment:2 Changed 7 years ago by Mathieu Doucet

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

To test: Try the following script. The Nexus files are in the systemtests repo. The script should complete without any error. 

from mantid.simpleapi import *
from reduction_workflow.instruments.sans.sns_command_interface import *
config = ConfigService.Instance()
config["instrumentName"]='EQSANS'

ws = Load("EQSANS_3293_event.nxs")

EQSANS()
NoSolidAngle()
NoNormalization()
AzimuthalAverage(n_bins=20)
NoIQxQy()
OutputPath("/tmp")
SetBeamCenter(89.6749, 129.693)
NoSensitivityCorrection()
SetTransmission(1, 0)
AppendDataFile("ws")
NoSaveIq()
Reduce()

comment:3 Changed 7 years ago by Mathieu Doucet

  • Keywords PatchCandidate added

comment:4 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:5 Changed 7 years ago by Michael Reuter

Script runs to completion without errors.

comment:6 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/7699_fix_eqsans_preloaded'

comment:7 Changed 7 years ago by Mathieu Doucet

Re #7699 Fix issue with loaded data (cherry picked from commit acfe2643fd93be2c94bc15432f5ca715c0353709)

Changeset: b8b1d21709e7dd419897e4ef9aa20db5b5790729

comment:8 Changed 7 years ago by Nick Draper

  • Component changed from Framework to SANS

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8544

Note: See TracTickets for help on using tickets.