Skip to content
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.

Sign up for GitHub

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

Jump to bottom

.29.10 crash by instantiate a Cell in a Library-Cell #1948

Open
olisnr opened this issue Dec 6, 2024 · 1 comment · May be fixed by #1951
Open

.29.10 crash by instantiate a Cell in a Library-Cell #1948

olisnr opened this issue Dec 6, 2024 · 1 comment · May be fixed by #1951
Assignees
@klayoutmatthias
Labels
bug
Milestone
0.29.11

Comments

@olisnr
Copy link

olisnr commented Dec 6, 2024

  1. make a new document
  2. make a cell from a library like Basic or SG13_dev (a PCell)
  3. make this cell to top
  4. try to drag an other cell in to it or use instantiate
@klayoutmatthias klayoutmatthias self-assigned this Dec 6, 2024
@klayoutmatthias
Copy link
Collaborator

Hi @olisnr,

Thanks for reporting this issue. I can easily reproduce it.

Fortunately this isn't an issue introduced recently, but has been there for long. Basically, editing a library cell should not be possible at all. The reasoning is that libraries are external resources (similar to Python modules you install with pip for example), and these should be immutable. In my case a message pops up, but then the application crashes.

Of course that should not happen.

Best regards,

Matthias

@klayoutmatthias klayoutmatthias removed their assignment Dec 6, 2024
@klayoutmatthias klayoutmatthias added this to the 0.29.11 milestone Dec 8, 2024
@klayoutmatthias klayoutmatthias self-assigned this Dec 8, 2024
klayoutmatthias pushed a commit that referenced this issue Dec 8, 2024
Fixing issue #1948 (crash on dragging a cell) - problem was the popup…
58376b3 
… which Qt does not like during drag-and-drop operations.
@klayoutmatthias klayoutmatthias linked a pull request Dec 8, 2024 that will close this issue
Fixing issue #1948 (crash on dragging a cell) - problem was the popup… #1951
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@klayoutmatthias klayoutmatthias

Labels
bug
Projects
None yet
Milestone
0.29.11
Development

Successfully merging a pull request may close this issue.

Fixing issue #1948 (crash on dragging a cell) - problem was the popup… KLayout/klayout
2 participants
@klayoutmatthias @olisnr