Skip to content

update metrics for gpl PR extending max iterations#3355

Closed
openroad-ci wants to merge 9 commits intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-gpl-extend-iter-by-routability
Closed

update metrics for gpl PR extending max iterations#3355
openroad-ci wants to merge 9 commits intoThe-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:secure-gpl-extend-iter-by-routability

Conversation

@openroad-ci
Copy link
Collaborator

@openroad-ci openroad-ci commented Jul 23, 2025

update metrics for https://github.com/The-OpenROAD-Project-private/OpenROAD/tree/gpl-extend-max-iter-by-routability

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: clk: 333.0000.
designs/ihp-sg13g2/spi/rules-base.json updates:

Metric Old New Type
finish__timing__drv__setup_violation_count 10 14 Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.

Messages from CI

[INFO] asap7/cva6 not included in CI.
[INFO] asap7/minimal not included in CI.
[INFO] gf12 not included in the update.
[INFO] gf55 not included in the update.
[INFO] nangate45/bp_quad not included in CI.
[INFO] nangate45/mempool_group not included in CI.
[INFO] rapidus2hp not included in the update.
[INFO] src not included in the update.

Updated Rules

[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: mrx_clk_pad_i: 300.0000.
[WARNING] Multiple clocks not supported. Will use first clock: clk: 333.0000.
designs/ihp-sg13g2/spi/rules-base.json updates:

Metric Old New Type
finish__timing__drv__setup_violation_count 10 14 Failing

[WARNING] Multiple clocks not supported. Will use first clock: ext_clk: 15.0000.

Messages from CI

[INFO] asap7/cva6 not included in CI.
[INFO] asap7/minimal not included in CI.
[INFO] gf12 not included in the update.
[INFO] gf55 not included in the update.
[INFO] nangate45/bp_quad not included in CI.
[INFO] nangate45/mempool_group not included in CI.
[INFO] rapidus2hp not included in the update.
[INFO] src not included in the update.

gudeh added 2 commits July 23, 2025 10:04
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh gudeh added the UpdateRules Starts GHA to update rules label Jul 23, 2025
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Jul 23, 2025
@gudeh gudeh requested a review from eder-matheus July 24, 2025 08:39
gudeh added 2 commits July 24, 2025 15:53
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@eder-matheus
Copy link
Member

Why does PR The-OpenROAD-Project/OpenROAD#7877 even need metric updates? I thought it was just changing variables and function names.

@gudeh
Copy link
Contributor

gudeh commented Jul 24, 2025

Why does PR The-OpenROAD-Project/OpenROAD#7877 even need metric updates? I thought it was just changing variables and function names.

Whops! it's not 7877 it is 7772: The-OpenROAD-Project/OpenROAD#7772

@gudeh gudeh added the UpdateRules Starts GHA to update rules label Jul 28, 2025
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Jul 28, 2025
@vvbandeira vvbandeira added the UpdateRules Starts GHA to update rules label Jul 28, 2025
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Jul 28, 2025
@vvbandeira
Copy link
Member

@gudeh
FYI, UpdateRules has been fixed, but there's a merge conflict

gudeh added 3 commits July 29, 2025 14:11
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@openroad-ci openroad-ci force-pushed the secure-gpl-extend-iter-by-routability branch from c232f88 to 1c0622a Compare July 29, 2025 14:19
Signed-off-by: Augusto Berndt <augusto.berndt@precisioninno.com>
@gudeh gudeh added the UpdateRules Starts GHA to update rules label Jul 30, 2025
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@openroad-ci openroad-ci removed the UpdateRules Starts GHA to update rules label Jul 30, 2025
@gudeh gudeh closed this Aug 1, 2025
@openroad-ci openroad-ci deleted the secure-gpl-extend-iter-by-routability branch August 1, 2025 09:30
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.

4 participants