Ticket #3790 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Fitted workspaces should be put into the correct grouped workspace.

Reported by: Robert Whitley Owned by: Robert Whitley
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

All workspaces that are created when making a fit against some data (currently 3) should be grouped under the same workspace.

Change History

comment:1 Changed 9 years ago by Robert Whitley

  • Priority changed from major to critical

comment:2 Changed 9 years ago by Robert Whitley

Refs #3790. Adding code to group the fitted workspaces.

Not working for table workspaces because it doesn't allow grouping of different types of workspaces.

Changeset: 9c22cc2542280beb1a52a6b2bec8739f175016fd

comment:3 Changed 9 years ago by Robert Whitley

Refs #3790. Adding code to group the fitted workspaces.

Not working for table workspaces because it doesn't allow grouping of different types of workspaces.

Changeset: 9c22cc2542280beb1a52a6b2bec8739f175016fd

comment:4 Changed 9 years ago by Robert Whitley

Refs #3790. Fitted workspaces within the muon interface are grouped.

Includes code for #4055. Expand the group workspaces allowing for mixed workspaces with table workspaces. The algorithms work with the newly formed groups but will ignore the table workspaces if they come across one.

​https://github.com/mantidproject/mantid/commit/a42ba792541f578e9099a4e0a8724cc0b1d84caf

comment:5 Changed 9 years ago by Robert Whitley

  • Status changed from new to accepted

comment:6 Changed 9 years ago by Robert Whitley

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

comment:7 Changed 9 years ago by Robert Whitley

Refs #3790. Fitted workspaces within the muon interface are grouped.

Includes code for #4055. Expand the group workspaces allowing for mixed workspaces with table workspaces. The algorithms work with the newly formed groups but will ignore the table workspaces if they come across one.

Changeset: a42ba792541f578e9099a4e0a8724cc0b1d84caf

comment:8 Changed 9 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:9 Changed 9 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Checked this works for:

Load current EMU

Load current Argus

data loaded from AutoTestData

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4637

Note: See TracTickets for help on using tickets.