OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

Dg replace merge

Open openroad-robot opened this issue 1 year ago • 5 comments

Hi Matt,

I added the init codes of DG-RePlAce. Could you please take a look ?

Thanks, Zhiang

openroad-robot avatar Apr 16 '24 15:04 openroad-robot

@ZhiangWang033 needs clang-format and clang-tidy cleaned up. Also it needs to gracefully handle not having cuda installed.

maliberty avatar Apr 16 '24 15:04 maliberty

Sorry, my bad. I modified the wrong files when I tested the DR.

I will fix this.

On Tue, Apr 16, 2024 at 9:44 AM Matt Liberty @.***> wrote:

@.**** commented on this pull request.

On src/drt/src/gr/FlexGR.cpp https://urldefense.com/v3/__https://github.com/The-OpenROAD-Project/OpenROAD/pull/4959*discussion_r1567672245__;Iw!!Mih3wA!C2AopV8Q_ntooHCylbsGVM6R6yRX3G2b1jye7hbRY80Ne02D1g3Yu3QrXM7METd1IpxJMGXulWTaj68fMOsgMBP8HVo$ :

Why are there changes in drt? It seems unrelated. Using std::cout is not ok and would require use of the logger if these are intended.

— Reply to this email directly, view it on GitHub https://urldefense.com/v3/__https://github.com/The-OpenROAD-Project/OpenROAD/pull/4959*pullrequestreview-2004156145__;Iw!!Mih3wA!C2AopV8Q_ntooHCylbsGVM6R6yRX3G2b1jye7hbRY80Ne02D1g3Yu3QrXM7METd1IpxJMGXulWTaj68fMOsg1R-My14$, or unsubscribe https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AQR3KJX5STLA3VGXGNOBBZLY5VIQDAVCNFSM6AAAAABGJTAKHWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMBUGE2TMMJUGU__;!!Mih3wA!C2AopV8Q_ntooHCylbsGVM6R6yRX3G2b1jye7hbRY80Ne02D1g3Yu3QrXM7METd1IpxJMGXulWTaj68fMOsgWfV-99U$ . You are receiving this because you were mentioned.Message ID: @.***>

ZhiangWang033 avatar Apr 17 '24 08:04 ZhiangWang033

Please fix up the copyright organization and year.

rovinski avatar Apr 18 '24 04:04 rovinski

@ZhiangWang033 any eta for your updates?

maliberty avatar May 07 '24 21:05 maliberty