Ticket #1695 (closed: wontfix)

Opened 10 years ago

Last modified 5 years ago

Long lines in the MantidPlot Python interpreter cause slowdowns/crashes

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Janik Zikovsky

Description (last modified by Martyn Gigg) (diff)

This is most noticeable when viewing a script from the workspace history that contains many thousands of masked spectra. The line length to MaskDetectors is very large and MantidPlot becomes unresponsive

The file attached has this issue. Be aware it will probably kill MantdPlot

Attachments

MERLIN_error.py​ (23.8 KB) - added by Martyn Gigg 10 years ago.

Change History

Changed 10 years ago by Martyn Gigg

comment:1 Changed 10 years ago by Martyn Gigg

  • Description modified (diff)

comment:2 Changed 10 years ago by Nick Draper

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

comment:3 Changed 10 years ago by Nick Draper

To be reevaluated after the long MaskDetectors lines have been removed as a problem. #1901

Is the length of reasonable lines ok, eg. up to 2 screen widths.

comment:4 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

comment:5 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:6 Changed 10 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:7 Changed 10 years ago by Martyn Gigg

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

Now that we have the ability to Mask by a workspace then this problem is very unlikely to occur. A script with a line length of 5 screen widths does not demonstrate the problem.

Marking as wontfix.

comment:8 Changed 9 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:9 Changed 9 years ago by Janik Zikovsky

  • Status changed from verifying to closed

The script window works even with a really long line of several thousand entries.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2542

Note: See TracTickets for help on using tickets.