Ticket #2885 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

SANS transmission wave ranges isn't doing sensible things in many cases

Reported by: Steve Williams Owned by: Steve Williams
Priority: major Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Mathieu Doucet

Description

In a SANS reduction the arguments passed to FitTrans() or Trans Fit in the Analysis Tab of the SANS GUI should equivalently set the wave range of the workspace and the two wave limit arguments passed to CalculateTransmission.

LimitsWav(), WavRangeReduction() and the entries in the Wavelength boxes of the Analysis Details tab affect only the rebinning of the sample and can, NOT the transmission workspaces.

The cases where the wavelength range of the transmission fit is narrower than the sample analysis and vice-versa both needed to be handled.

This will be all documented and work as explained in documents linked from here http://www.mantidproject.org/Using_the_SANS_GUI_Analysis_Details

Change History

comment:1 Changed 9 years ago by Steve Williams

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Steve Williams

(In [11181]) account for transmission changes re #2885

comment:3 Changed 9 years ago by Steve Williams

(In [11298]) Save the SANS transmission workspace before the re-binning needed for a divide re #2885

comment:4 Changed 9 years ago by Steve Williams

(In [11431]) CalculateTransmission's output workspace can now have arbitrary binning and some names are changed in the SANS GUI re #2885

comment:5 Changed 9 years ago by Steve Williams

(In [11545]) Test for CalculateTransmission is now a bit more efficient re #2885

comment:6 Changed 9 years ago by Steve Williams

(In [11546]) Test for CalculateTransmission was failing re #2885

comment:7 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 29 to Iteration 30

Accepted and assigned tickets moved at iteration 29 code freeze

comment:8 Changed 9 years ago by Steve Williams

  • Status changed from accepted to verify
  • Resolution set to fixed
  • Milestone changed from Iteration 30 to Iteration 29

comment:9 Changed 9 years ago by Mathieu Doucet

  • Status changed from verify to verifying
  • Tester set to Mathieu Doucet

Works

comment:10 Changed 9 years ago by Mathieu Doucet

  • Status changed from verifying to closed

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3732

Note: See TracTickets for help on using tickets.