JuliaPolyhedra / Polyhedra.jl

Polyhedral Computation Interface
Other
172 stars 27 forks source link

Don't remove redundancy twice for planar V-rep #261

Closed blegat closed 3 years ago

blegat commented 3 years ago

We already remove the redundancy with planar_hull, no need for the second part of course...

codecov[bot] commented 3 years ago

Codecov Report

Merging #261 (acf18a4) into master (3342c44) will decrease coverage by 0.09%. The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #261      +/-   ##
==========================================
- Coverage   78.18%   78.08%   -0.10%     
==========================================
  Files          39       39              
  Lines        3213     3213              
==========================================
- Hits         2512     2509       -3     
- Misses        701      704       +3     
Impacted Files Coverage Δ
src/redundancy.jl 90.95% <75.00%> (-1.43%) :arrow_down:

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 3342c44...acf18a4. Read the comment docs.