-
Notifications
You must be signed in to change notification settings - Fork 22
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.
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
✨ Post-Layout Optimization #260
✨ Post-Layout Optimization #260
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #260 +/- ##
==========================================
+ Coverage 95.12% 95.41% +0.28%
==========================================
Files 95 96 +1
Lines 8965 9422 +457
==========================================
+ Hits 8528 8990 +462
+ Misses 437 432 -5
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add unit tests in test/algorithms/physical_design/optimization.cpp
. We cannot know whether your code performs the intended functionality without unit tests. That also means, that when I make changes, I cannot see whether I broke something. Since I do not understand the code as well as you do, I would have to rely on the test output whenever I make adjustments.
…o post_placement_optimization
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
experiments/post_layout_optimization/post_layout_optimization.cpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
include/fiction/algorithms/physical_design/post_layout_optimization.hpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There were too many comments to post at once. Showing the first 25 out of 26. Check the log or trigger a new build to see more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
Description
This PR introduces an optimization algorithm that reduces the layout area of sub-optimal Cartesian 2DDWave-clocked layouts.
This algorithms includes:
This reduces the area of layouts for a set of benchmark functions generated with
ortho
andNanoPlaceR
by an average of50%
and20%
, respectively.Checklist: