Ticket #9868 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Workspaces toolbox filtering option

Reported by: Owen Arnold Owned by: Nick Draper
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Dan Nixon

Description

MantidPlot suggestion for SANS/Reflectometry (but also wider)

The typical processing used in SANS/Reflectometry generates very large numbers of workspaces/runs. This makes navigating the workspace list very hard. A user suggestion by Max is that we have a filter option on the workspace list (could have a Regex option as Max suggests). That way users could find all those workspaces where workspace name matches say _IvsQ only.

Nick: Could you consider this suggestion and assign the ticket if you think this is a valid thing to do in MantidPlot.

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Nick Draper

  • Summary changed from MantidPlot suggestion to Workspaces toolbox filtering option

To Test

This script is useful as it creates a lot of workspaces

C="a"
B=""
for i in range (52):
	CreateSampleWorkspace(OutputWorkspace = B+C)
	if C == "z":
	 C = "a"
	 B="A"
	else:
	 C = chr(ord(C)+1)

You need to test:

  1. how this works with groups
    • intended behavior is that
      1. included groups would list all children
      2. groups with a child that matches would include just matching children and the parent group.
  2. Test that the filtering is fine with workspaces being created updated and deleted.
  3. Test the reaction to clear all memory.

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from assigned to inprogress

re #9868 simple search filtering enabled

Changeset: 203d0e82932c339869a465fad378a0da00cdd264

comment:4 Changed 6 years ago by Nick Draper

re #9868 regex filtering

Changeset: 3f36947e579d994fe6955f11bb4bf27897678b36

comment:5 Changed 6 years ago by Nick Draper

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:6 Changed 6 years ago by Dan Nixon

  • Status changed from verify to verifying
  • Tester set to Dan Nixon

comment:7 Changed 6 years ago by Dan Nixon

  • Status changed from verifying to reopened
  • Resolution fixed deleted

This script demonstrates an issue with matching groups that have matching children when filtering for "g":

names = ['a', 'b', 'c', 'g1', 'g2', 'g3']
for name in names:
	CreateSampleWorkspace(OutputWorkspace=name)

GroupWorkspaces(['a', 'g2'], OutputWorkspace='grp')
GroupWorkspaces(['b', 'g3'], OutputWorkspace='data')

comment:8 Changed 6 years ago by Nick Draper

re #9868 Adjust to handle lazy loading & perform stress test

Changeset: a6e5a96a2904282766b697810f4b128bf6e001f4

Last edited 6 years ago by Nick Draper (previous) (diff)

comment:9 Changed 6 years ago by Nick Draper

  • Status changed from reopened to verify
  • Resolution set to fixed

comment:10 Changed 6 years ago by Nick Draper

Merge branch 'feature/9868_Workspace_FilterBox' into develop

re #9868

Conflicts:

Code/Mantid/MantidPlot/src/Mantid/MantidDock.cpp Code/Mantid/scripts/Interface/ui/reflectometer/refl_gui.py

Changeset: 339e378abc196a5e20bec28dbdf28b239d2a0bc2

comment:11 Changed 6 years ago by Nick Draper

remove a bit of debugging code that troubled linux, re #9868

Changeset: b086eaab14731b08c6a2a37a6d9b6f7f59ba5c8b

comment:12 Changed 6 years ago by Nick Draper

  • Tester Dan Nixon deleted

Remove Dan as a tester as he is on holiday for a week

comment:13 Changed 6 years ago by Dan Nixon

  • Status changed from verify to verifying
  • Tester set to Dan Nixon

comment:14 Changed 6 years ago by Nick Draper

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/master' into feature/9868_Workspace_FilterBox

Full changeset: f0943d6b5b266919b29ffba58cec4c5edfbbb4fd

comment:15 Changed 6 years ago by Dan Nixon

Merge remote-tracking branch 'origin/feature/9868_Workspace_FilterBox'

Full changeset: 9c21f83465d638eafd1e0b68b6a78d32ef3c8b8f

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10710

Note: See TracTickets for help on using tickets.