Ticket #7581 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Mantidplot crash when change plot association

Reported by: Anders Markvardsen Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.0
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

To reproduce

  1. Use the example in #7579 to create a table and plot the scatter plot
  1. In the scatter plot double click on a data point, which will open MantidPlot - Plots details
  1. Right click on Graph1->Layer1->Table... and select Plot Associations
  1. Change the x-axis plot assocation

and MantidPlot crashes

Attachments

createTestTable.py​ (248 bytes) - added by Arturs Bekasovs 7 years ago.
Script to create testing table

Change History

comment:1 Changed 7 years ago by Arturs Bekasovs

  • Status changed from new to inprogress

Convert indentation to spaces.

Refs #7581

Changeset: 08080061e709dfd4d056ddd8f20c305141f22d62

comment:2 Changed 7 years ago by Arturs Bekasovs

Use checkable table items instead of checkboxes.

This eliminates lots of possible problems, as we work with items only and avoid lots of dangerous casts. One of the casts was causing the segfault in this ticket.

Refs #7581

Changeset: 0819cba9e1dee4438cae61068e30afb12a555f8e

comment:3 Changed 7 years ago by Arturs Bekasovs

Tester:

Check that the scenario described doesn't lead to MantidPlot crash anymore.

In addition to that, use the script attached to create a nice little table. Plot any of the Y columns first, and then, using Plot Associations menu, change it so that another Y column is used. Try other combinations of column types. That is to make sure it works properly, as I've had to do some refactoring.

Changed 7 years ago by Arturs Bekasovs

Script to create testing table

comment:4 Changed 7 years ago by Arturs Bekasovs

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

comment:5 Changed 7 years ago by Anders Markvardsen

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

comment:6 Changed 7 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/7581_plot_assoc_crash'

comment:7 Changed 7 years ago by Anders Markvardsen

Tested this both with attached script (that is from a QTIPlot table) and a mantid TableWorkspace generated using muonanalysis - worked smoothly

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8426

Note: See TracTickets for help on using tickets.