-
Notifications
You must be signed in to change notification settings - Fork 163
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
[WIP] Added option to define subtile in tile_annotation section #828
base: master
Are you sure you want to change the base?
Conversation
Is this already considered and merged? or we are waiting for more information? |
@ganeshgore Can you resolve the merging conflicts? I will then review. You need a simple test case to validate this new feature. |
@ganeshgore A good start on the subtile support. I think the tile annotation is where we should add more syntax. I will create an issue to detail the support we want. I will follow up on this PR. |
@ganeshgore Changes are o.k. I will follow-up to finish the subtile support as planned in #916 |
Let me know if the changes on the python scripts are fine to merge. Then I will merge it. |
To support subtile feature of latest VPR.
Now you can define
tile_name
andsub_tile_name
as follows.sub_tile_name
declaration is optional, if not defined it is same astile_name