Ticket #9470 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Remove Mandatory Stitching Defaults

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: critical Milestone: Release 3.2
Component: Reflectometry Keywords:
Cc: Blocked By: #9171, #9500, #9501, #9785
Blocking: #9371 Tester: Peter Parker

Description

Since #9171. Stitch1D has been adapted so that most of the parameters are not mandatory. This relaxation has not been propagated to the following Algorithms:

  • CreateTransmissionWorkspaceAuto
  • ReflectometryReductionOneAuto

This is a necessary step for creating the new reflectometry gui.

Change History

comment:1 Changed 6 years ago by Owen Arnold

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Owen Arnold

  • Status changed from assigned to inprogress

comment:3 Changed 6 years ago by Owen Arnold

refs #9470. Invoke error.

Changeset: 0b90f656935ef9c6402223ac2e6548391ca8e8e1

comment:4 Changed 6 years ago by Keith Brown

  • Blocked By 9500, 9501 added

added #9500 and #9501 as blockers here as the new C++ algs will need this update

Last edited 6 years ago by Keith Brown (previous) (diff)

comment:5 Changed 6 years ago by Owen Arnold

  • Blocked By 9785 added

comment:6 Changed 6 years ago by Owen Arnold

Tester. The following python scripts should now work since Params, StartOverlap and EndOverlap are not mandatory.

LoadISISNexus(Filename='INTER000013463', OutputWorkspace='13463')
LoadISISNexus(Filename='INTER000013464', OutputWorkspace='13464')
 
transmission_new_style = CreateTransmissionWorkspaceAuto(
                                           FirstTransmissionRun='13463', 
                                           SecondTransmissionRun='13464')

# transmission_new_style should be a workspace in wavelength

Same for ReflectometryReductionOneAuto

LoadISISNexus(Filename='INTER000013463', OutputWorkspace='13463')
LoadISISNexus(Filename='INTER000013464', OutputWorkspace='13464')
run = LoadISISNexus(Filename='INTER000013460', OutputWorkspace='13460')
 
incidentAngle = 0.7
 
                                           
a, b, c = ReflectometryReductionOneAuto(InputWorkspace='13460',
                                           FirstTransmissionRun='13463', 
                                           SecondTransmissionRun='13464', ThetaIn=incidentAngle)
                                           
# a in q
# b in wavelength
# c is a double scale factor

Check that the workspaces are outputted in the correct X-units. Unit tests have already been added for these features.

No code changes to be made here, so no branch to build or merge . These relate to work that has already been done under #9785.

Last edited 6 years ago by Owen Arnold (previous) (diff)

comment:7 Changed 6 years ago by Owen Arnold

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

comment:8 Changed 6 years ago by Peter Parker

  • Status changed from verify to verifying
  • Tester set to Peter Parker

comment:9 Changed 6 years ago by Peter Parker

Both scripts now work. The resulting workspaces had the expected units.

comment:10 Changed 6 years ago by Peter Parker

  • Status changed from verifying to closed

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10313

Note: See TracTickets for help on using tickets.