Ticket #8353 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

LoadNexus doesn't handle group names properly

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: critical Milestone: Release 3.1
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Jay Rainey

Description


Change History

comment:1 Changed 7 years ago by Samuel Jackson

  • Status changed from new to inprogress

Refs #8353 Changed LoadNexus to simply pass up output properties.

Changeset: 40393a7a98b31fe00c52133149c258e71dc003a4

comment:2 Changed 7 years ago by Samuel Jackson

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

To Tester

Check that when using LoadNexus to open a workspace group, the names for each of the child workspaces are set properly.

Currently LoadNexus simply renames whatever child workspaces are called to be ParentWorkspace_1, ParentWorkspace_2 etc. With this fix it will simply use whatever the child algorithm called the workspace. A good example file to use would be irs26173_graphite002_ResNorm_Paras.nxs in the system tests as this is a saved workspace group where each of the child workspaces have unique names.

comment:3 Changed 7 years ago by Samuel Jackson

  • Milestone changed from Backlog to Release 3.1

comment:4 Changed 7 years ago by Jay Rainey

  • Status changed from verify to verifying
  • Tester set to Jay Rainey

comment:5 Changed 7 years ago by Jay Rainey

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/8353_LoadNexus_group_names'

Full changeset: f1e29728de0093aaa73e090733aa5933a9a65c1f

comment:6 Changed 7 years ago by Jay Rainey

When loading a group workspace with LoadNexus the names of each child workspace are now set correctly as described in comment:2.

I tested this with the file suggested, and several others from the systemtests. All functions as expected.

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9198

Note: See TracTickets for help on using tickets.