Ticket #5418 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Python Console Window Crash

Reported by: Peter Parker Owned by: Martyn Gigg
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

Seen on Windows 7. In Mantid since at least the 2.1.0 release.

To reproduce:

  • Enter the following script into the MantidPlot Python console, replacing the filename string to point to some file you have:
filename = "C:/test/OSIRIS/osi95111_graphite002_Correct_95157.nxs"
Load(Filename=filename,OutputWorkspace="test")

ws = mtd["test"]
  • Hit Ctrl+Enter to run the script.
  • Add a new line to the end of the script, by doing exactly the following:
    • Type ws.getD
    • Hit Enter to autocomplete to ws.getDetector
    • Hit "(" to open up some function brackets.

Mantid will crash.

Change History

comment:1 Changed 8 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Martyn Gigg

comment:2 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:3 Changed 8 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:4 Changed 8 years ago by Martyn Gigg

Avoid crash during autocompletion in ScriptEditor. Refs #5418

Implements a workaround for a bug in QScintilla. The modification to the python is not strictly necessary but it helps with comparisons under the hood.

Changeset: 86a3ca6579f2a8e9fbce9b6f857bd4757e9beaa7

comment:5 Changed 8 years ago by Martyn Gigg

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

comment:6 Changed 8 years ago by Martyn Gigg

Avoid crash during autocompletion in ScriptEditor. Refs #5418

Implements a workaround for a bug in QScintilla. The modification to the python is not strictly necessary but it helps with comparisons under the hood.

Changeset: 86a3ca6579f2a8e9fbce9b6f857bd4757e9beaa7

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

By following the test description, I couldn't get this to Crash in Windows any longer.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6264

Note: See TracTickets for help on using tickets.