Osama Hammad

Results 72 comments of Osama Hammad

@maliberty Does GRT writes the guides to the db even if it produces an error? **Edit** After looking it doesn't. saveGuides is called after checkOverflow which produces the error. So...

@eder-matheus got it and removed that flag in the flow test. However, I find that the same issue is happening in ORFS. In design [nangate45/swerv_wrapper](https://jenkins.openroad.tools/blue/organizations/jenkins/OpenROAD-flow-scripts-Private/detail/secure-TR-via-access-layer/1/pipeline/237/). ![Image](https://github.com/user-attachments/assets/5e19fd7e-69c3-4ada-a2a0-16503c68f5e0) This happens at the...

https://jenkins.openroad.tools/blue/organizations/jenkins/OpenROAD-flow-scripts-Private/detail/secure-TR-via-access-layer/6/pipeline sky130hd/microwatt antenna repair flow: > [ERROR DRT-0155] Guide in net net56035 uses layer li1 (2) that is outside the allowed routing range [met1 (4), met5 (12)] with via access...

@maliberty Could you please review?

@vijayank88 This is a low priority issue at the moment. It's not done yet. It would be a good first issue for new comers.

Yeah sorry for misleading but we should use snake case not camel case: except_edge_....

Are you sure this is a dpl issue? I ran check_placement in the first scenario and it seems to identify the power switch cells overlap correctly. And dpl doesn't place...

I created a [PR](https://github.com/The-OpenROAD-Project-private/OpenROAD/pull/2557) that should fix tapcell ignoring the power switches but still waiting for the secure CI results.

@maliberty Should this issue be closed now?

@rafaelmoresco Also please address: - Clang-tidy review requests - CI failures - format the code