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

OpenROAD.IOPlacement options #253

Open
kareefardi opened this issue Nov 5, 2023 · 1 comment
Open

OpenROAD.IOPlacement options #253

kareefardi opened this issue Nov 5, 2023 · 1 comment
Labels
enhancement New feature or request waiting on op This issue requires more information from its original creator

Comments

@kareefardi
Copy link
Collaborator

kareefardi commented Nov 5, 2023

Description

io_layer_variables

io_layer_variables = [
used in OpenROAD.IOPlacement have no validation. Layout would be generated with geometry related issues such as overlapping pins and pins that violate spacing rules. Back in the day, these variables were added to ensure maximum routability for macro integration. In a recent (will find the link) discussion with OpenROAD it was questioned whether these options are really needed and should be removed. We didn't get back to OpenROAD if we value these options.

Proposal

Study the feasibility and value of these options and decide on either removing them or keeping them and have some sort of validation for these variables

@kareefardi kareefardi added the enhancement New feature or request label Nov 5, 2023
@donn
Copy link
Member

donn commented Apr 3, 2024

Do you have a link to the issue where they asked for us to re-investigate our need for those options?

@donn donn added the waiting on op This issue requires more information from its original creator label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request waiting on op This issue requires more information from its original creator
Projects
None yet
Development

No branches or pull requests

2 participants