Skip to content

Conversation

@lee30sonia
Copy link
Member

@lee30sonia lee30sonia commented Apr 26, 2021

  • Sort fanouts of PIs in aqfp_view when PIs need to be branched.
  • Add test cases for fanout_limit_view

@codecov-commenter
Copy link

Codecov Report

Merging #452 (b80008a) into master (4279f2f) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #452      +/-   ##
==========================================
+ Coverage   80.97%   81.00%   +0.02%     
==========================================
  Files         120      121       +1     
  Lines       12850    12937      +87     
==========================================
+ Hits        10405    10479      +74     
- Misses       2445     2458      +13     
Impacted Files Coverage Δ
include/mockturtle/views/aqfp_view.hpp 79.48% <100.00%> (+0.72%) ⬆️
include/mockturtle/views/fanout_limit_view.hpp 80.72% <100.00%> (ø)
...nclude/mockturtle/algorithms/dsd_decomposition.hpp 91.42% <0.00%> (+4.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4279f2f...b80008a. Read the comment docs.

@coveralls
Copy link

coveralls commented Apr 26, 2021

Pull Request Test Coverage Report for Build 789584955

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.9%) to 82.22%

Files with Coverage Reduction New Missed Lines %
include/mockturtle/utils/node_map.hpp 1 98.31%
Totals Coverage Status
Change from base Build 786141623: 0.9%
Covered Lines: 12273
Relevant Lines: 14927

💛 - Coveralls

@lee30sonia lee30sonia merged commit e9da8ee into master Apr 27, 2021
@lee30sonia lee30sonia deleted the aqfp_fix branch April 27, 2021 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants