Ticket #10409 (closed: fixed)
Replace ElWin code with ElasticWindow algorithm
Reported by: | Dan Nixon | Owned by: | Dan Nixon |
---|---|---|---|
Priority: | major | Milestone: | Release 3.3 |
Component: | Indirect Inelastic | Keywords: | |
Cc: | spencer.howells@… | Blocked By: | #10368 |
Blocking: | #10492 | Tester: | Harry Jeffery |
Description (last modified by Dan Nixon) (diff)
There is an algorithm that does the same routine as the UI so the UI may as well use it.
Attachments
Change History
comment:3 Changed 6 years ago by Dan Nixon
- Blocked By 10368 added
Need an algorithm runner, added one in #10368.
comment:4 Changed 6 years ago by Dan Nixon
- Status changed from assigned to inprogress
Added new ElasticWindowMultiple algorithm
Refs #10409
Changeset: 5318ec9fe9bebad609959e5aaf25c067b9dc8240
comment:5 Changed 6 years ago by Dan Nixon
Add ELF workspace and correct vertical units
Refs #10409
Changeset: 2f74e975cb11fc9fa2184b7365d0cfe2a5509fb5
comment:7 Changed 6 years ago by Dan Nixon
Make changed to existing Elwin system test
Refs #10409
Changeset: dfed485a78c5301707669757e8fd58b057fc32d4
comment:9 Changed 6 years ago by Dan Nixon
Work in progress adding new algo to existing UI
Refs #10409
Changeset: 5b34f4cb10698ca7c105ecb8cf4ac306714bc440
comment:10 Changed 6 years ago by Dan Nixon
comment:11 Changed 6 years ago by Dan Nixon
Tidy UI, use naming required for MSD fit
Refs #10409
Changeset: 24fdf2c8a5bd90b6ce3a08a6f5b41ac902d2c467
comment:12 Changed 6 years ago by Dan Nixon
comment:13 Changed 6 years ago by Dan Nixon
comment:14 Changed 6 years ago by Dan Nixon
Use invisible workspaces in ElasticWindowMultiple
Refs #10409
Changeset: 1954a3b2f26fe990126f91c6936fab13950bc627
comment:15 Changed 6 years ago by Dan Nixon
comment:16 Changed 6 years ago by Dan Nixon
comment:17 Changed 6 years ago by Dan Nixon
comment:18 Changed 6 years ago by Dan Nixon
comment:19 Changed 6 years ago by Dan Nixon
Note there are systemtest changes here: https://github.com/mantidproject/systemtests/compare/feature/10409_elwin_ui_use_elasticwindow
To test:
- Ensure all system and doc tests are passing
- Download the files attached to this ticket and put them somewhere Mantid will find them
- Open Indirect > Data Analysis > Elwin
- Select the two IRIS S(Q, w) files (S(Q, w) never worked on this interface before)
- Enable plot result and Run
- Should give the q, q2 and elf workspaces, elf x axis should be in run number
- Select the two OSIRIS reduced files (temp is in log in files)
- Enable Normalise to Lowest Temp
- Enable plot result and Run
- Should give q, q2, elf and elt workspaces, elf and etl x axis should be in temperature
- Select the two IRIS reduced files (no temp. in files, should be loaded from log files)
- Enable Normalise to Lowest Temp
- Enable plot result and Run
- Should give q, q2, elf and elt workspaces, elf and etl x axis should be in temperature
comment:21 Changed 6 years ago by Dan Nixon
- Status changed from inprogress to verify
- Resolution set to fixed
comment:22 Changed 6 years ago by Harry Jeffery
- Status changed from verify to verifying
- Tester set to Harry Jeffery
comment:23 Changed 6 years ago by Harry Jeffery
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/10409_elwin_ui_use_elasticwindow'
Full changeset: 7479c6d85f6176b91624547f8949f22632be1e98
comment:24 Changed 6 years ago by Harry Jeffery
Merge remote-tracking branch 'origin/feature/10409_elwin_ui_use_elasticwindow'
Full changeset: c9bf6ee16adddb5f2b38e1d047fce9e1ff476861
comment:25 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 11251