Skip to content

Zoom (in) function with selected layers significantly slower in newer releases #1453

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
matthias-vorwerk opened this issue Aug 7, 2023 · 1 comment · Fixed by #1454
Closed
Assignees
Labels
Milestone

Comments

@matthias-vorwerk
Copy link

matthias-vorwerk commented Aug 7, 2023

When compiling and using latest klayout release 0.28.10 and opening a larger layout the right mouse zoom in function, but also keyboard based zoom in functionality has longer delays.

CentOS7, self compiled QT5 + klayout 0.28.10, removed $HOME .klayoutrc

The effect is as follows:

  • open a large layout
  • full hierarchy depth (*)
  • show only one selected layer, then add a second layer to show
  • try to zoom in (and out + in again, several times)

now: when zooming in often the zoom in shows significant delays.
BUT When I Show All layers. there is no such a delay. Showing only 2..3 individual layers again shows the zoom in issue.
This is very reproducible with different layouts, uncompressed GDS sizes used for tests: 3..5 GB

Then I compiled previous releases and it looks like this is happening since klayout 0.28.8.
It still works fine in 0.28.7.

@klayoutmatthias
Copy link
Collaborator

Thanks for reporting this problem.

I could reproduce and fix it - it was actually related to the mouse hover event and it was related to the change in the text box treatment. The search for the mouse hover was going too deep.

Thanks and best regards,

Matthias

@klayoutmatthias klayoutmatthias self-assigned this Aug 7, 2023
@klayoutmatthias klayoutmatthias added this to the 0.28.11 milestone Aug 7, 2023
klayoutmatthias pushed a commit that referenced this issue Aug 7, 2023
… search for texts in transient selection
klayoutmatthias added a commit that referenced this issue Aug 7, 2023
Fixes issue #1453: fixed performance issue which was caused by a deep…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants