Ticket #2212 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

SANS: Wavelength loops analysis

Reported by: Nick Draper Owned by: Steve Williams
Priority: critical Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

a wlist=[2.2,3.0,4.0,6.0,8.0,10.0] string input into a box on the gui and a button to launch it, the better. It needs to do min to max wavelength, followed by the pairs in the list, eg 2.2 to 10.0, then 2.2 to 3, 3 to 4 etc up to 8.0 to 10.0

Change History

comment:1 Changed 10 years ago by Steve Williams

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Martyn Gigg

Time estimate from M. Gigg: I would this should be do able within a day. The underlying script is simple to call in the manner required so it should just be a matter of gathering the requested info from the GUI.

comment:3 Changed 10 years ago by Nick Draper

  • Blocking 2287 added

comment:4 Changed 10 years ago by Steve Williams

(In [8952]) Case consistency with "Small Angle Scattering" in Facities.xml and the ISIS SANS GUI has a new wav loop widget! refs #1399 and #2212

comment:5 Changed 10 years ago by Steve Williams

(In [8956]) Convert Microsoft VS Intellisense completions into code that, like is valid re #2212

comment:6 Changed 10 years ago by Steve Williams

(In [8975]) ISIS SANS plot ranges neaten up the plotting re #2212

comment:7 Changed 10 years ago by Steve Williams

Look in the "Limits" section under the "Analysis Details" tab by Wavelength. Move the ComboBox to a "RANGE" entry and fill in the box. This should work with "Plot result" on the first tab and there is error checking for non-numeric entries.

comment:8 Changed 10 years ago by Steve Williams

(In [8983]) ISIS SANS plot ranges labels and docs re #2212

comment:9 Changed 10 years ago by Steve Williams

(In [8984]) ISIS SANS plot ranges correction re #2212

comment:10 Changed 10 years ago by Steve Williams

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

The feature freeze is about to start, submit for pre-release testing.

comment:11 Changed 10 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:12 Changed 10 years ago by Martyn Gigg

  • Status changed from verifying to closed

Working as advertised in 1.1.9493

comment:13 Changed 10 years ago by Steve Williams

  • Status changed from closed to reopened
  • Resolution fixed deleted

comment:14 Changed 10 years ago by Steve Williams

  • Status changed from reopened to accepted

Some of the reduction steps are repeated on the workspaces after the first one in the loop

comment:15 Changed 10 years ago by Steve Williams

(In [9876]) Facilitate running reduction multiple times re #2212

comment:16 Changed 10 years ago by Steve Williams

(In [9879]) Facilitate running reduction multiple times in different situations re #2212

comment:17 Changed 10 years ago by Steve Williams

(In [9881]) Facilitate running reduction multiple times in different situations re #2212

comment:18 Changed 10 years ago by Steve Williams

(In [9882]) Get another build going re #2212

comment:19 Changed 10 years ago by Steve Williams

(In [9883]) Facilitate running reduction multiple times in different situations re #2212

comment:20 Changed 10 years ago by Steve Williams

(In [9894]) The first range in the loop wasn't being included re #2212

comment:21 Changed 10 years ago by Steve Williams

  • Status changed from accepted to verify
  • Resolution set to fixed
  • Blocking 2287 removed
  • Milestone changed from Iteration 27 to Iteration 28

Richard has been using this and seems happy

comment:22 Changed 10 years ago by Steve Williams

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:23 Changed 10 years ago by Steve Williams

  • Status changed from reopened to accepted

Richard's found problems that the workspaces produced by the loop aren't identical to if they had been generated by individual reductions.

comment:24 Changed 10 years ago by Steve Williams

(In [10438]) LoadNexusProcess must load the Run (sample) object first so that it can load the correct IDF and stopped repeated steps in SANS loops re #2212

comment:25 Changed 10 years ago by Steve Williams

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

From Richard

Hi Steve,

I have not yet managed to break you build of this morning testing wavelength loops – which is good!

comment:26 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester changed from Martyn Gigg to Michael Reuter

comment:27 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

I performed both the loop reduction and some individual reductions and the results come out identical which is the aim of the ticket.

comment:28 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3059

Note: See TracTickets for help on using tickets.