Ticket #3886 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

CheckWorkspacesMatch does not do the correct thing with groups

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

To reproduce:

  • Start MantidPlot
  • Increase the log level to information
  • Run the following script (the data is in AutoTestData) 
    x = Load('CSP78173','x').workspace()
y = Load('CSP78173','y').workspace()
Scale(x,x,2)
CheckWorkspacesMatch(x,y)

The results will show that CheckWorkspacesMatch was run with both inputs coming from the first workspace group, i,e [x_1, x_1], [x_2,x_2] etc.

Change History

comment:1 Changed 9 years ago by Martyn Gigg

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

In [15001]:

Fix CheckWorkspacesMatch when used with WorkspaceGroup objects. It now has its own processGroups so that it can set an appropriate result string for different types of failure. Fixes #3886

comment:2 Changed 9 years ago by Nick Draper

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

comment:3 Changed 9 years ago by Nick Draper

this passes, although spotted a different issue #4491

comment:4 Changed 9 years ago by Nick Draper

  • Status changed from verifying to closed

comment:5 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4733

Note: See TracTickets for help on using tickets.