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

Instance Property form: Array Instance col/row would be more intuitive then the current row/col #1356

Closed
stefanottili opened this issue Apr 30, 2023 · 3 comments · Fixed by #1667
Milestone

Comments

@stefanottili
Copy link

In the current Instance Property form, the Array Instances entries are presented as rows/colums.
Please reverse this order, col/row (x, y) would be more intuitive..

It's also the order in which gds aref col/row number and coordinates are defined.
Oasis repetitions are defined as N-column (x) by M-row (y) dimensions.

@klayoutmatthias
Copy link
Collaborator

klayoutmatthias commented Mar 23, 2024

This is how the three places for arrays now look like:

image

image

image

@stefanottili
Copy link
Author

stefanottili commented Mar 24, 2024

great, 'now' as in 'upcoming', right ?
I find the col/row (x/y dimension) much more intuitive then today's master.
It should say 'This is instance [c,r] of array with'.
image

@klayoutmatthias
Copy link
Collaborator

Yes, "now" means "soon" :)
I want to finalize and push out 0.29. It has been in the oven for too long already :(
Matthias

@klayoutmatthias klayoutmatthias linked a pull request Mar 29, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants