Ticket #4307 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Don't allow the ResultsTable tab to populate its tables with workspaces that have the same parameters

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

Description

Currently muon scientists have told us they shouldn't fit anything using different functions (and therefore different parameters).

However if they were to do this the current results table tab will either crash or seem to run normally but populate the final table with wrong information.

A check needs to be done before populating the lists to make sure no parameters tables with different parameters are populated into the tables on the ResultsTable tab.

To replicate crash/wrong information. 1) Load Muon Analysis.

2) Load a few data sets.

3) Using the fittings tab have some workspaces fitted with just Gaussian, then the others fitted using Gaussian and Exp Decay.

4) Click the results table tab.

5) Check to see if workspaces listed all have the same parameters.

6) Check to see if creating the table has the correct information (may crash here depending on order of parameters).

Change History

comment:1 Changed 9 years ago by Robert Whitley

Refs #4307. Add warning message when parameters don't match.

Populate the table with workspaces that contain the same parameters.

Changeset: 58d16ac4652c506c74300d8ea1e6c8fa62ed6770

comment:2 Changed 9 years ago by Robert Whitley

  • Status changed from new to accepted

comment:3 Changed 9 years ago by Robert Whitley

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

comment:4 Changed 9 years ago by Robert Whitley

Refs #4307. Add warning message when parameters don't match.

Populate the table with workspaces that contain the same parameters.

Changeset: 58d16ac4652c506c74300d8ea1e6c8fa62ed6770

comment:5 Changed 9 years ago by Anders Markvardsen

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

comment:6 Changed 9 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Test using linearbackground and emu data

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5154

Note: See TracTickets for help on using tickets.