Ticket #1686 (closed: fixed)
Get the SANS analysis to run under Mathieu's new frame work
Reported by: | Steve Williams | Owned by: | Steve Williams |
---|---|---|---|
Priority: | major | Milestone: | Iteration 25 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Michael Whitty |
Description
There should be system tests that import ISISCommandInterface
Change History
comment:10 Changed 10 years ago by Steve Williams
comment:11 Changed 10 years ago by Steve Williams
comment:12 Changed 10 years ago by Mathieu Doucet
comment:13 Changed 10 years ago by Steve Williams
comment:14 Changed 10 years ago by Mathieu Doucet
comment:15 Changed 10 years ago by Steve Williams
comment:16 Changed 10 years ago by Steve Williams
comment:17 Changed 10 years ago by Steve Williams
comment:18 Changed 10 years ago by Steve Williams
comment:19 Changed 10 years ago by Steve Williams
comment:20 Changed 10 years ago by Steve Williams
comment:21 Changed 10 years ago by Steve Williams
comment:22 Changed 10 years ago by Steve Williams
comment:23 Changed 10 years ago by Steve Williams
comment:24 Changed 10 years ago by Steve Williams
comment:25 Changed 10 years ago by Steve Williams
comment:26 Changed 10 years ago by Martyn Gigg
comment:27 Changed 10 years ago by Steve Williams
- Status changed from new to verify
- Resolution set to fixed
comment:28 Changed 10 years ago by Steve Williams
This stuff needds some thorough testing again the original code!
comment:29 Changed 10 years ago by Steve Williams
- Status changed from verify to reopened
- Resolution fixed deleted
comment:30 Changed 10 years ago by Steve Williams
comment:31 Changed 10 years ago by Steve Williams
comment:32 Changed 10 years ago by Steve Williams
comment:33 Changed 10 years ago by Steve Williams
comment:34 Changed 10 years ago by Steve Williams
comment:35 Changed 10 years ago by Steve Williams
- Status changed from reopened to verify
- Resolution set to fixed
comment:36 Changed 10 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:37 Changed 10 years ago by Steve Williams
- Status changed from verifying to reopened
- Resolution fixed deleted
comment:38 Changed 10 years ago by Steve Williams
- Status changed from reopened to verify
- Resolution set to fixed
comment:40 Changed 10 years ago by Nick Draper
- Status changed from verifying to verify
- Tester Martyn Gigg deleted
comment:41 Changed 10 years ago by Michael Whitty
- Status changed from verify to verifying
- Tester set to Michael Whitty
comment:42 Changed 10 years ago by Michael Whitty
- Status changed from verifying to reopened
- Resolution fixed deleted
Notes:
- 'save result' button brings up a dialog saying that the workspace failed to save, when this is not the case.
comment:43 Changed 10 years ago by Steve Williams
comment:44 Changed 10 years ago by Michael Whitty
More Notes (don't know if these fall under this ticket but putting them here for a notice to work on later, perhaps):
- Geometry tab, after loading raw data - Click on 'Run', get dialog message on fatal python error and SyntaxError: "EOL while scanning single-quoted string" in scripting console. Sorry, no line number.
- Running 2D Reduce with "Plot Result" checked, Python error - "NameError: "global name 'qti' is not defined" in file 'ISISCommands.py' at line 415"
- Running 2d reduce with LOQ 54431 sample - error
Component with name some-sample-holder was not found. Error in execution of algorithm MoveInstrumentComponent: Component with name some-sample-holder was not found.
Reproduce by: Run 2d with plot result checked, uncheck plot result and run again
comment:45 Changed 10 years ago by Michael Whitty
Additional to point 1 (geometry tab) above: it seems to clear the information from the Masking tab. Don't know if this is helpful.
comment:46 Changed 10 years ago by Steve Williams
comment:47 Changed 10 years ago by Michael Whitty
this line:
if reducer.to_Q.output_type == '1D': }} on line 412 of ISISCommands.py seems to return True for 2D as well as 1D. This shouldn't affect the analysis, but will stop the plotting of 2D ever working (you need to cut the "qti.app.MantidUI." off the start of line 415 for this to happen). Otherwise, the issues I've noticed seem to be resolved. The geometry tab works, saving nexus file works, and (since plot will always think it's 1d and won't go to the name error), that doesn't cause a problem running the reduction.
comment:49 Changed 10 years ago by Steve Williams
- Status changed from accepted to verify
- Resolution set to fixed
comment:50 Changed 10 years ago by Michael Whitty
- Status changed from verify to verifying
the problems I've noted have been resolved.
comment:51 Changed 10 years ago by Michael Whitty
- Status changed from verifying to closed
the problems I've noted have been resolved. hopefully there will be some documentation for this interface in the next release?
comment:52 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 2533
(In [6396]) Extended the Python Mask ReductionStep to handle different shapes re #1686