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
cost view, cost aware resynthesis problem, cost generic resub #554
Conversation
Codecov Report
@@ Coverage Diff @@
## master #554 +/- ##
==========================================
+ Coverage 82.77% 82.81% +0.03%
==========================================
Files 148 155 +7
Lines 18535 19151 +616
==========================================
+ Hits 15343 15860 +517
- Misses 3192 3291 +99
Help us with your feedback. Take ten seconds to tell us how you rate us. |
A cost-generic resubstitution algorithm with a customizable cost function.
|
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.
I haven't actually looked into the code yet. Please check the documentation first (make html
in docs/
).
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.
Mostly good, but the documentation can still be clearer, especially for the part users need to modify (defining customized cost functions).
include/mockturtle/algorithms/experimental/cost_generic_resub.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.
Great! Last thing: Add these items in changelog
(algorithm: cost_generic_resub
, view: cost_view
, util: recursive_cost_functions
)
No description provided.