Skip to content

Conversation

gdey
Copy link
Member

@gdey gdey commented May 2, 2018

Added the Doublaspeucker simplification algo.

gdey added 3 commits May 2, 2018 10:32
Added some handy planar functions, and defined an Interface for
Simplifers. This way we can add and change out simplifiers easily.
@gdey gdey requested review from ARolek and jasonsurratt May 2, 2018 18:53
@coveralls
Copy link

coveralls commented May 2, 2018

Pull Request Test Coverage Report for Build 61

  • 33 of 64 (51.56%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.7%) to 77.05%

Changes Missing Coverage Covered Lines Changed/Added Lines %
planar/simplify/debug.go 2 4 50.0%
planar/simplify/douglaspeucker.go 31 60 51.67%
Totals Coverage Status
Change from base Build 50: -2.7%
Covered Lines: 2387
Relevant Lines: 3098

💛 - Coveralls

gdey added a commit that referenced this pull request May 29, 2018
Removed the spherical version as we don't need it and there may be an
issue with the distance formula that was being used, and got pulled.

Improved test scaffolding, with the aim to add more tests.
Removed the spherical version as we don't need it and there may be an
issue with the distance formula that was being used, and got pulled.

Improved test scaffolding, with the aim to add more tests.
@gdey gdey force-pushed the douglaspeucker branch from 1f3db48 to abb6d65 Compare May 29, 2018 20:11
@gdey gdey merged commit 0b58be8 into master May 29, 2018
@gdey gdey deleted the douglaspeucker branch May 29, 2018 22:09
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.

2 participants