Ticket #6227 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

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:4 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:5 Changed 7 years ago by Nick Draper

  • Status changed from assigned to new

comment:6 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

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:9 Changed 7 years ago by Gesner Passos

  • Component changed from Framework to SANS

comment:10 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

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

Note: See TracTickets for help on using tickets.