lsils / mockturtle

C++ logic network library
MIT License
202 stars 136 forks source link

Fix `substitute_node`; substitute without re-strashing #586

Closed lee30sonia closed 1 year ago

lee30sonia commented 1 year ago

IDEA: Besides the strategy of substitution without re-strashing, we could also implement another simpler substitute_node which does re-strashing normally but assumes that the network is in topological order (because in that case we don't need the fix in #560) --> future work

lee30sonia commented 1 year ago

Tagging some people with whom I have discussed this issue: @Nozidoali @aletempiac @hriener

codecov-commenter commented 1 year ago

Codecov Report

Merging #586 (9638125) into master (510c9dd) will increase coverage by 0.03%. The diff coverage is 92.59%.

@@            Coverage Diff             @@
##           master     #586      +/-   ##
==========================================
+ Coverage   83.23%   83.27%   +0.03%     
==========================================
  Files         165      165              
  Lines       21508    21558      +50     
==========================================
+ Hits        17903    17952      +49     
- Misses       3605     3606       +1     
Impacted Files Coverage Δ
include/mockturtle/networks/aig.hpp 93.24% <92.59%> (-0.15%) :arrow_down:
...nclude/mockturtle/algorithms/dsd_decomposition.hpp 91.54% <0.00%> (+4.22%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.