Ticket #1527 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

3 BUGS: with group workspaces

Reported by: Janik Zikovsky Owned by: Sofia Antony
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

Bug 1:

  • Load two workspaces, cncs1, cncs2
  • Group them to NewGroup
  • Rebin NewGroup to NewGroup
  • workspace names become NewGroup_1 and NewGroup_2: it would be nice if they kept the original name

Bug 2, keep going:

  • Now try to load into workspace cncs1 again. Loading is successful, but cncs1 is nowhere to be seen.

Bug 3:

  • Load two workspaces, cncs1, cncs2
  • Group them to NewGroup
  • MaskDetectors on NewGroup
  • Group is now empty.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • type changed from enhancement to defect

comment:2 Changed 10 years ago by Russell Taylor

And another one....

Bug 4:

  • Running ConjoinWorkspaces (inherits from PairedGroupAlgorithm) seems to require that the members of the group are called ParentName_1, _2, _3 etc. At least the unit test failed until this was the case. This must be up in the generic processing somewhere.

comment:3 Changed 10 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner changed from Nick Draper to Sofia Antony

comment:4 Changed 10 years ago by Sofia Antony

  • Status changed from assigned to accepted
  • Component set to Mantid

comment:5 Changed 10 years ago by Sofia Antony

(In [6998]) re#1527 - fix for 3 workspace group bugs. 1 and 3 fixed.2 is not a bug - in this case the workspace is a member workspace which is part of already loaded group workspace.Expand the group workspace to see the updated member workspace. Now if an algorithm takes 1) i/p and o/p group workspaces are of different names and the input group members are of similar names,then the o/p group workspace members are named o/p group workspace name appended with "-" and period number. example.i/p group workspace is "InputGroup" with two members "InputGroup_1","InputGroup_2" and o/p group workspace is "OutputGroup" the members of "outputGroup" will be "OutputGroup_1","OutputGroup_2". 2) i/p and o/p group workspace of same names, the o/p group workspace member is named the same name of the corresponding i/p group member workspace. example i/p group workspace is "Group" with two members"group_1","group_2" and o/p workspace name is same as i/p,then the o/p group members are "group_1","group_2" 3)i/p and o/p group workspaces of different names and i/p members are of different names then the o/p group workspace members are named as below inputmemberwsname_outgroupworkspacename example i/p group workspace "Group" with two members "evs","csp" and o/p group workspace is "NewGroup",then the o/p group workspace members are "evs_NewGroup","csp_NewGroup"

comment:6 Changed 10 years ago by Sofia Antony

(In [6999]) re#1527 - fix for some build warnings

comment:7 Changed 10 years ago by Sofia Antony

(In [7072]) re#1527 - PairedGroupAlgorithm changed to rename the o/p group members if i/p and o/p groups are of same names or diff.names.

comment:8 Changed 10 years ago by Sofia Antony

(In [7106]) re#1527 - error messages added to PairedGroupAlgorithm when the member workspace not exists in ADS .

comment:9 Changed 10 years ago by Sofia Antony

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

Bug 4 is fixed now.But Conjoinworkspaces is not working when one of the inputs is a group workspace and the second input is a single workspace.Also it's not functioning properly when we join two group workspaces with member workspaces of overlapping spectrum numbers though this prints error message. a new ticket will be created against this in the next iteration.

comment:10 Changed 10 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:11 Changed 10 years ago by Nick Draper

Tested on 1.1.7148

Bug 1: Fixed Bug 2: No longer an issues once bug 1 is fixed, overwriting is expected behaviour Bug 3: fixed Bug 4: Spawned new ticket #1809

comment:12 Changed 10 years ago by Nick Draper

  • Status changed from verifying to closed

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2374

Note: See TracTickets for help on using tickets.