Ticket #6227 (closed: fixed)
SANS: Display / 1D Analysis allow selecting deleted workspaces
Reported by: | Gesner Passos | Owned by: | Gesner Passos |
---|---|---|---|
Priority: | major | Milestone: | Release 3.0 |
Component: | SANS | Keywords: | |
Cc: | stephen.king@…, anders.markvardsen@… | Blocked By: | |
Blocking: | Tester: | Jay Rainey |
Description (last modified by Gesner Passos) (diff)
The Display tab on SANS interface (that will be renamed to 1D Analysis #6226 ) shows in the workspace combobox deleted workspaces.
In fact, its filtering is not working well. It should show only 1D workspaces (I believe we should assume a workspace with just one spectra), but, right now, it allows the user to select many spectra workspaces, deleted workspaces, but but some times does not show the ones it should
To reproduce the defect do the following:
- Open SANS Interface (Menu->Interfaces->SANS ISIS)
- Select LOQ instrument (combobox -> Run Numbers tab)
- Load User file ([systemtest]/Data/LOQ/MASK.CENTER)
- Change to Geometry Tab
- Set the BeamCentre radio to 'front' or 'hab'
- Change the max iterations to 3 (to speed up the process)
- Change to the Display/1DAnalysis tab
- Open the workspace combobox
You will see for example Down_tmp that is not available on the Workspaces MantidPlot, but will not see Down (the one you should see, because it is a reduced file)
Change History
comment:1 Changed 8 years ago by Nick Draper
- Owner set to Anders Markvardsen
- Status changed from new to assigned
comment:2 Changed 8 years ago by Anders Markvardsen
- Owner changed from Anders Markvardsen to Gesner Passos
comment:3 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:7 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to backlog at the code freeze for R2.6
comment:8 Changed 7 years ago by Gesner Passos
- Description modified (diff)
- Milestone changed from Backlog to Release 3.0
comment:11 Changed 7 years ago by Gesner Passos
Run this with the SANS interface opened, and you will see three lines with 'left' workspace written in the Workspace input at Sans Analysis interface.
for i in range(3): left_tmp = CreateWorkspace([1], [1]) left = RenameWorkspace(left_tmp)
comment:12 Changed 7 years ago by Gesner Passos
Correct WorkspaceSelector to avoid duplicate workspace names
re #6227
Changeset: 61d324ecdc7a78b24c1b444eb617415be3300727
comment:13 Changed 7 years ago by Gesner Passos
- Status changed from inprogress to verify
- Resolution set to fixed
Tester:
The easiest way to check that this ticket has been fixed is running the script in comment:11. But you may want to run the description as well, which has triggered the ticket. What is important is that the WorkspaceSelector should not display the same name twice.
comment:14 Changed 7 years ago by Jay Rainey
- Status changed from verify to verifying
- Tester set to Jay Rainey
comment:15 Changed 7 years ago by Jay Rainey
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/bugfix/6227_sans_1d_analysis'
comment:16 Changed 7 years ago by Jay Rainey
Testing
- The 1D Analysis tab no longer shows deleted workspaces & only shows 1D workspaces.
- Tested on Ubuntu 12.04
Code Review
- Clean, and easy to understand (well written) code.
comment:17 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7073