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.

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

Instantation path highlight mismatch with view window #1804

Closed
s910324 opened this issue Jul 27, 2024 · 1 comment · Fixed by #1805
Closed

Instantation path highlight mismatch with view window #1804

s910324 opened this issue Jul 27, 2024 · 1 comment · Fixed by #1805
Assignees
Labels
Milestone

Comments

@s910324
Copy link

s910324 commented Jul 27, 2024

System : Windows 11 x64 / Ubuntu 24.02
Software : Klayout 0.28.17 - 0.29.04

Hi Matthias,

I found that the Instantation path highlight seems to having some mismatch with the view window, this seems to only occured when I selected the lowest ranked element in the whole path.

Image below I select instance "C", which is the one on the most right side.

but the view window highlight pops up at (0, 0), which seems to be not correct.

image

Instantation path ABC.zip

@klayoutmatthias
Copy link
Collaborator

Hi @s910324,

thanks for this nicely prepared report!

I can reproduce the problem with the instantiation path display. That is, because the "instantation path" of an instance should actually stop above the instance cell - because the instance is the object, and the path to this object is TOP-ABC.

Currently it is included, which formally is not correct, but useful. If it is included, it should show the instance itself as the last entry.

Thanks and best regards,

Matthias

klayoutmatthias pushed a commit that referenced this issue Jul 27, 2024
…dow)

In addition, the highlighting was fixed in the
presence of a context path (aka "edit in place").
@klayoutmatthias klayoutmatthias self-assigned this Jul 27, 2024
@klayoutmatthias klayoutmatthias added this to the 0.29.5 milestone Jul 27, 2024
klayoutmatthias added a commit that referenced this issue Jul 30, 2024
Fixed issue #1804 (Instantation path highlight mismatch with view win…
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