OpenROAD-flow-scripts icon indicating copy to clipboard operation
OpenROAD-flow-scripts copied to clipboard

Published 20 hours ago • verified publisher icon The-OpenROAD-Project

Consistent treatment of flag variables

Open povik opened this issue 10 months ago • 6 comments

Description

There are a number of variables to turn things on/off, as documented on the "flow variables" page: https://openroad-flow-scripts.readthedocs.io/en/latest/user/FlowVariables.html

As far as I can tell those variables don't have consistent interpretation, for some variables they must be set to 1 for the documented effect (e.g. SKIP_CTS_REPAIR_TIMING), for other variables the existence of the variable is all that matters, the value is ignored (e.g. EQUIVALENCE_CHECK).

Suggested Solution

The obvious suggestion is to have a helper to check flags which would be used throughout the Tcl code; I have no idea if there's a fitting place to define one.

Additional Context

No response

povik avatar Apr 11 '24 20:04 povik

I agree things have gotten somewhat inconsistent.

I believe all steps start with load.tcl so that could be a place to put the utility (I see append_env_var is there which seems like a utility). You could also create a utility.tcl and source it from load.tcl to make a cleaner separation.

Are you interested in resolving this?

maliberty avatar Apr 11 '24 21:04 maliberty

Are you interested in resolving this?

Yes! If I get to it I will make the change.

povik avatar Apr 12 '24 07:04 povik

@povik are you working on this? @socclosure will take it up if you are not.

maliberty avatar Sep 17 '24 17:09 maliberty

@maliberty Not really, no

povik avatar Sep 17 '24 17:09 povik

Thanks Folks.I will look into this issue then.Regards Jignesh On Sep 17, 2024, at 10:05 AM, Martin Povišer @.***> wrote: @maliberty Not really, no

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

socclosure avatar Sep 17 '24 17:09 socclosure

@maliberty @povik I am proposing to add a small makefile file target to print value and name of all user's configurable variables which can be used in all subsequent TCL code of any other targets.

For reference check "print_all_env_cfg" in Makefile in the "ISSUE_1935" branch of my local git Repo at https://github.com/socclosure/OpenROAD-flow-scripts/blob/ISSUE_1935/flow/Makefile

Let me know your feedback,

socclosure avatar Sep 22 '24 06:09 socclosure

@maliberty @povik Any feedback on my proposal on code implementation of this task?

socclosure avatar Sep 27 '24 05:09 socclosure

There is a lot of progress on this recently in image

I'm not sure how printing the variables ensures consistent treatment of them.

maliberty avatar Oct 07 '24 23:10 maliberty

As per my understanding the user wants to know the value & existence of each configuration variable which can be set in multiple files. So, I thought printing the name and value of each configuration variable can provide the information that the user is looking for.

But it seems I might be missing something. Can you please elaborate on the requirements of this issue?

-REgards Jignesh Can you elaborate

On Mon, Oct 7, 2024 at 4:12 PM Matt Liberty @.***> wrote:

There is a lot of progress on this recently in image.png (view on web) https://github.com/user-attachments/assets/c19d8ea8-a38b-4d1c-9e29-555ed1ef25e3

I'm not sure how printing the variables ensures consistent treatment of them.

— Reply to this email directly, view it on GitHub https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/issues/1935#issuecomment-2398108927, or unsubscribe https://github.com/notifications/unsubscribe-auth/BJW5OQE3ZMG6XHCUOBVNOFLZ2MIOZAVCNFSM6AAAAABGC63IWCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJYGEYDQOJSG4 . You are receiving this because you were assigned.Message ID: @.*** com>

socclosure avatar Oct 08 '24 01:10 socclosure