Skip to content

add mode for RC correlation based on segments#3989

Merged
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
AcKoucher:util-segment-correlation
Mar 12, 2026
Merged

add mode for RC correlation based on segments#3989
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
AcKoucher:util-segment-correlation

Conversation

@AcKoucher
Copy link
Contributor

This should resolve the first item on #3969.

For now, I'm adding the segment-based regression as an additional mode rather than replacing net-based. The idea is to let the new mode stabilize.

Observations

  1. Net-based approach used parasitics data from STA (loaded with read_spef). On the other hand, segment-based needs the parasitics of each routing shape in ODB and, as 6_final.odb does not contain parasitics data, we need to extract_parasitics before trying to retrieve the required data from ODB.
  2. The parasitcs extraction needs to be made in such a way that the wire shapes correctly match the resistive segments (see the arguments used when calling extract_parasitics).
  3. Note that I also used segment-based for resistance. The reason is: even though it's not necessary, it would be more confusing to have a hybrid approach to generate set_layer_rc values i.e., net-based for resistance and segment-based for capacitance. In segment-based, it is also easier to process the resistance of the .csv file as we don't need to manually decrease the vias' resistance of each net.

@AcKoucher AcKoucher force-pushed the util-segment-correlation branch from 69526c3 to 4f08593 Compare March 12, 2026 18:33
Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
@AcKoucher AcKoucher force-pushed the util-segment-correlation branch from 4f08593 to 9750e41 Compare March 12, 2026 18:38
@AcKoucher AcKoucher requested a review from maliberty March 12, 2026 18:45
@maliberty maliberty merged commit c3226cd into The-OpenROAD-Project:master Mar 12, 2026
7 of 8 checks passed
@AcKoucher AcKoucher deleted the util-segment-correlation branch March 13, 2026 00:31
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 this pull request may close these issues.

2 participants