Ticket #10809 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

BugFix: VSI freezes for MDEvent workspace in Multislice view

Reported by: Anton Piccardo-Selg Owned by: Anton Piccardo-Selg
Priority: critical Milestone: Release 3.3
Component: GUI Keywords: VATES
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

The MDEvent workspace is causing the VSI to freeze when it is loaded into the Mulitslice view. This seems to be a bug which has been present at least since the last release.

Attachments

MDHisto_Larmor.nxs (1.2 MB) - added by Anton Piccardo-Selg 6 years ago.
MDHisto_Osiris.nxs (164.6 KB) - added by Anton Piccardo-Selg 6 years ago.
Test Script for Bugfix 10809.docx (15.3 KB) - added by Anton Piccardo-Selg 6 years ago.

Change History

comment:1 Changed 6 years ago by Anton Piccardo-Selg

  • Priority changed from blocker to critical

The issue only appears when switching from either a STANDARD or SPLATTERPLOT view into a MULTISLICE view. Directly loading the MDEvent workspace into the MULTISLICE view does not cause the issue.

The bug only appears on Windows 7 machines. It has been tested on OSX and Ubuntu, where it does not cause the same issues.

We have tracked the issue down to

QObject::connect(activeObjects, SIGNAL(representationChanged(pqRepresentation*)),

this->ui.propertiesPanel, SLOT(setRepresentation(pqRepresentation*)));

in the MdViewerWidget's setParaViewComponentsForView().

When we uncomment this option, the issue stops, but we are left without the option to change the representation (such as the opacity etc) in the properties panel.

The actual freezing stems from Qt evaluating messages from the Windows message queue. They seem to suggest that something is constantly trying to resize the window. A QScrollBar is sending an event to a QScrollArea. Although it is not clear yet, how this is related to the part of the code presented above.

Utkarsh Ayachit of ParaView has been contacted regarding this issue. He will be able to look at it on Monday the 22nd of December.

comment:2 Changed 6 years ago by Anton Piccardo-Selg

Refs #10809 Fix Multislice freeze of MDEvent data in Win7

Changeset: 1752c1c4e6540e14b4c3694191addd4caac2df0f

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

comment:3 Changed 6 years ago by Anton Piccardo-Selg

For Testing: Please find the test instructions and the required data sets in the attachment.

comment:4 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from new to assigned

comment:5 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from assigned to inprogress

comment:6 Changed 6 years ago by Anton Piccardo-Selg

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

Please see comment 3 for testing

Last edited 6 years ago by Anton Piccardo-Selg (previous) (diff)

comment:7 Changed 6 years ago by Owen Arnold

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

comment:8 Changed 6 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/10809_fix_multislice_freeze_with_mdevent_data'

Full changeset: 1324972ebb1b35ea94447b73cf9106ef03791ad8

comment:9 Changed 6 years ago by Owen Arnold

Nicely done. Good test instructions.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11651

Note: See TracTickets for help on using tickets.