Ticket #8081 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Muon Analysis interface causes Mantid to fail when shutting down

Reported by: Peter Parker Owned by: Arturs Bekasovs
Priority: blocker Milestone: Release 3.0
Component: Muon Keywords:
Cc: martyn.gigg@… Blocked By:
Blocking: Tester: Michael Reuter

Description

To reproduce, simply open up the interface and then close down Mantid. You will see the interface be cleared of its widgets, but the window itself will still be open and Mantid will hang.

I tried a few other windows / interfaces and this does not appear to be a problem outside of this specific interface.

It also doesn't occur when you close down the interface before shutting down Mantid.

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Priority changed from critical to blocker

comment:2 Changed 7 years ago by Arturs Bekasovs

This was reported in another ticket as well, but I were not able to reproduce it nor on my Ubuntu 12.04 nor on my Windows 7 VM. Could you please specify the OS you are using when this happens?

On my machine I noticed that when you close the MantidPlot with Muon Analysis opened, you get a segfault, which I assumed could be the reason for these problems on you machine. I've fixed it on the develop, so if it is the master you are using, could you please try develop and check if that happens in there as well?

Last edited 7 years ago by Arturs Bekasovs (previous) (diff)

comment:3 Changed 7 years ago by Peter Parker

This happens on my Windows 7 64-bit machine.

The bug has definitely made it into master (and is not just a quirk of my particular build environment), since the latest nightly build of master has the same problem.

comment:4 Changed 7 years ago by Peter Parker

Arturs, I didn't see your edited comment for some reason.

I've checked and the develop branch appears to be free of the problem.

comment:5 Changed 7 years ago by Arturs Bekasovs

So this should have been fixed on #6691 and will get to the master as soon as it will be verified.

comment:6 Changed 7 years ago by Arturs Bekasovs

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

#6691 was merged to master, so the problem should be solved.

comment:7 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

Yes, this seems to be working.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8926

Note: See TracTickets for help on using tickets.