Ticket #3943 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

CTE Interface does not work for HYSPEC

Reported by: Stuart Campbell Owned by: Andrei Savici
Priority: major Milestone: Release 2.0
Component: MantidPlot Keywords:
Cc: hagenme@… Blocked By:
Blocking: Tester: Michael Reuter

Description

The ConvertToEnergy GUI doesn't work correctly for HYSPEC.

I used run 398 to test (/SNS/HYS/IPTS-6746/1/398/NeXus/HYS_398_event.nxs with an Ei=19.

The error given is 

ConvertUnits-[Notice] ConvertUnits started
PropertyManager-[Error] Property "INPUTWORKSPACE" is not set to a valid value: "Enter a name for the Input/InOut workspace".
PropertyManager-[Error] Property "INPUTWORKSPACE" is not set to a valid value: "Enter a name for the Input/InOut workspace".

Having a quick look the error looks like the error is on line 268 of DirectEnergyConversion.py. It is the variable result_ws not set correctly as it is trying to use a non-existent workspace name (_398_event.nxs) - so this might be down to the common GUI data loader (in CommonFunctions.py) stripping of the "HYS". Or something down to HYSPEC having a different long and short names - but this should also be the case for SEQUOIA.

Andrei, Can you have a look further in it ?

Change History

comment:1 Changed 9 years ago by Andrei Savici

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Andrei Savici

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

In [15220]:

One of the algorithms is not event aware. Added the PreserveEvents=False flag to the rebin command. Fixes #3943

comment:3 Changed 9 years ago by Stuart Campbell

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:4 Changed 9 years ago by Stuart Campbell

In [15224]:

Added HYSPEC as an SNS instrument. refs #3943

comment:5 Changed 9 years ago by Andrei Savici

  • Status changed from reopened to accepted

comment:6 Changed 9 years ago by Andrei Savici

In [15319]:

Updated DEC script. Refs #3943

comment:7 Changed 9 years ago by Andrei Savici

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

comment:8 Changed 9 years ago by Michael Reuter

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

comment:9 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

OK, HYSPEC data does reduce via the CTE interface.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4790

Note: See TracTickets for help on using tickets.