Ticket #7828 (closed: fixed)
EQSANS: issue with running reduction twice
| Reported by: | Mathieu Doucet | Owned by: | Mathieu Doucet | 
|---|---|---|---|
| Priority: | major | Milestone: | Release 3.0 | 
| Component: | SANS | Keywords: | PatchCandidate | 
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Russell Taylor | 
Description
A recent feature added to the EQSANS reduction is the ability to use an event workspace as the input of the reduction instead of a file. This causes a problem when a run number is used as the data input. The reduction then loads the file and calls the workspace by the run number. If you hit reduce a second time it will try to reload that workspace, which will make the reduction fail.
The logic that looks for a workspace should be updated to check that the given workspace not only exists but is an event workspace that was not already reduced.
Change History
comment:2 Changed 7 years ago by Mathieu Doucet
- Status changed from inprogress to verify
 - Resolution set to fixed
 - Milestone changed from Backlog to Release 3.0
 
To test:
- In the MantidPlot options, set SNS/EQSANS as your default instrument.
 - Make sure the systemtests Data directory is in the list of your Mantid data directories.
 - From MantidPlot's Interfaces menu, start the ORNL SANS UI.
 - If the EQSANS reduction UI doesn't start, go under Tools of the UI that just popped up and pick Change instrument: then pick EQSANS.
 - Under Tools of the EQSANS UI, make sure to turn on debug mode.
 - Fill in reduction options:
- Enter only a run number (use 3293) in the scattering data file field of the Data tab.
 - Under the Transmission section of the same tab, click the Fix transmission box and enter 1 for the transmission value.
 
 - Hit reduce, the reduction should complete and a workspace with the run number as a name should appear.
 - Hit reduce again. The reduction should run again to completion, without error. The results should be the same.
 
comment:3 Changed 7 years ago by Jay Rainey
- Status changed from verify to verifying
 - Tester set to Jay Rainey
 
comment:4 Changed 7 years ago by Jay Rainey
- Status changed from verifying to reopened
 - Resolution fixed deleted
 
I wasn't exactly sure which files to use in order to test. If possible could one of the SNS guys write a short guide on how to use the EQSANS reduction interface?
comment:5 Changed 7 years ago by Mathieu Doucet
- Status changed from reopened to verify
 - Resolution set to fixed
 
I've updated the testing instructions above (comment:2).
comment:7 Changed 7 years ago by Russell Taylor
- Status changed from verify to verifying
 - Tester set to Russell Taylor
 
comment:8 Changed 7 years ago by Russell Taylor
- Status changed from verifying to closed
 
Merge remote branch 'origin/feature/7828_fix_repeated_eqsans_reduction'
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 8673

Re #7828 Add event workspace check
Changeset: 0ea89b1efefee0db73033566ed7f2d8f71c38d1f