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

Placement channel_width_fac #2651

Open
amin1377 opened this issue Jul 17, 2024 · 0 comments
Open

Placement channel_width_fac #2651

amin1377 opened this issue Jul 17, 2024 · 0 comments

Comments

@amin1377
Copy link
Contributor

amin1377 commented Jul 17, 2024

Throughout the placement code, we need to frequently use min and max functions in time-sensitive parts of the code before accessing channel_width_fac. This data structure should be updated to cover the entire grid (from 0 to width-1, and so on). During the pre-computation of the inverse of the average channel width, we can assume that the channel width at width-1 is the same as the channel width at width-2. This change would simplify the code, remove an asymmetry that is no longer necessary, and might slightly speed up the code.

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

No branches or pull requests

1 participant