Ticket #5177 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

VSI: crash with double rebinning

Reported by: Janik Zikovsky Owned by: Michael Reuter
Priority: critical Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

This is on RHEL6.

  • Create a 3D MDEventWorkspace using OneStepMDEW
  • Open VSI from MantidPlot
  • Click Rebin and apply to get a first binned MDHistoWorkspace
  • Click back on the original MDEventWorkspace in the Paraview filter tree.
  • Click Rebin button and Apply again.
  • Segmentation fault!

Change History

comment:1 Changed 8 years ago by Michael Reuter

Do you want a second entry to appear or are you looking to modify the original workspace?

comment:2 Changed 8 years ago by Janik Zikovsky

Having a second entry might be useful, if that's not too hard to do. You could switch between the two rebinned views to compare...

comment:3 Changed 8 years ago by Michael Reuter

That's actually much more work than fixing a simple crash.

comment:4 Changed 8 years ago by Michael Reuter

  • Status changed from new to accepted

I'm going to fix this by not allowing multiple rebins for now.

comment:5 Changed 8 years ago by Michael Reuter

Refs #5177. Stopping multiple rebinning.

Changeset: 2e67b2cc2c589ea286cba1e338647d49a4124252

comment:6 Changed 8 years ago by Michael Reuter

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

To test, follow the above prescription. A warning dialog should now appear to stop you from actually completing the process.

comment:7 Changed 8 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:8 Changed 8 years ago by Owen Arnold

  • Status changed from verifying to closed

Great. I'm prevented from doing this. Get overplotting warning to let me know what happened.

comment:9 Changed 8 years ago by Michael Reuter

Refs #5177. Stopping multiple rebinning.

Changeset: 2e67b2cc2c589ea286cba1e338647d49a4124252

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6023

Note: See TracTickets for help on using tickets.