Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DRT PA Refactor #5721

Open
bnmfw opened this issue Sep 10, 2024 · 0 comments · May be fixed by #6359
Open

DRT PA Refactor #5721

bnmfw opened this issue Sep 10, 2024 · 0 comments · May be fixed by #6359
Assignees
@bnmfw
Labels
documentation Improvements or additions to documentation drt Detailed Routing

Comments

@bnmfw
Copy link
Contributor

bnmfw commented Sep 10, 2024 •
edited
Loading

Description

This Issue is derived from Issue 5610. Specifically for the Pin Access Module

Refactoring Requirements

  • Clear and Easy code to understand and maintain
  • Less number of lines per method
  • Cyclomatic complexity less than 20 (preferably less than 10)
  • Doxygen Documentation for all functions/classes
  • Following google code style c++
  • Eliminate code duplication
  • clang-tidy clean
  • Coverity clean

Related Pull Requests

This issue was solved over several pull requests, here is a list of them:
#5744 #5760 #5770 #5778 #5783 #5788 #5796 #5803 #5820 #5821 #5822 #5823 #5858 #5862 #5872 #5891 #5892 #5901 #5921 #5903 #6321 #5945 #6359
@bnmfw bnmfw added documentation Improvements or additions to documentation drt Detailed Routing labels Sep 10, 2024
@bnmfw bnmfw self-assigned this Sep 10, 2024
@bnmfw bnmfw added this to Docs Sep 10, 2024
@github-project-automation github-project-automation bot moved this to Todo in Docs Sep 10, 2024
@bnmfw bnmfw moved this from Todo to In Progress in Docs Sep 10, 2024
@bnmfw bnmfw mentioned this issue Sep 13, 2024
Merged
@bnmfw bnmfw linked a pull request Dec 16, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bnmfw bnmfw

Labels
documentation Improvements or additions to documentation drt Detailed Routing
Projects
Docs
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

drt: last name changes and doxygen bnmfw/OpenROAD_Bernardo_Sandoval
1 participant
@bnmfw