Improve performance for selection

Yet Another Terminal :: Serial Communication :: Engineer/Test/Debug

Brought to you by: maettu_this, y-a-terminal

#461 Improve performance for selection

Status: closed-fixed

Owner: Maettu

Labels: View (59)

Priority: 7

Updated: 2022-10-19

Created: 2022-03-11

Creator: jonarque

Private: No

First of all, YAT is great ! Congrats for the hard work !

When you reach the 1000 line limit in the display and you select them all. It takes a long time. YAT can even report itself as begin "unresponsive" on a quite powerful computer.

Thanks for taking consideration of this.

Discussion

Maettu - 2022-03-11

labels: --> View

status: unread --> open-accepted

assigned_to: Maettu

Priority: 5 --> 7

Last edit: Maettu 2022-03-11
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Maettu - 2022-03-11

I'll check on this. Keep in mind that YAT is a simple .NET WinFoms app with user drawn listboxes. I did quite some performance analysis in the past but it just has limitations: The drawing/coloring takes most of the CPU's time.

But note you may always reduce the number of lines in the buffer, or disable formatting completely. Both will speed up YAT's responsiveness.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Maettu - 2022-10-13

Confirming this issue, it is caused by the find/search feature, which needs to perform a feasibility check for each selected line. But of course this should not be of any relevance...

...when deliberately selecting all lines.

...when there is no intention to perform a find/search.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Maettu - 2022-10-16

Update:

The find/search feature did add its part, but it is not the main culprit, that is the stupid implementation of the .NET ListBox that just isn't performance optimized. Good news though, I think I have an approach to further optimize the performance.

Last edit: Maettu 2022-10-16

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Maettu - 2022-10-18

status: open-accepted --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Maettu - 2022-10-18

Terminal > Select All now happens instantly even with very many lines.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- jonarque - 2022-10-19
  
  Excellent !
  
  If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.