Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Optimize spatial filters #24

Merged
merged 4 commits into from
Sep 7, 2016
Merged

Optimize spatial filters #24

merged 4 commits into from
Sep 7, 2016

Conversation

anthonynsimon
Copy link
Owner

Significantly improved performance of spatial filters. Added Erode and Dilate tests to ensure no change in output.

benchmark              old ns/op      new ns/op      delta
BenchmarkMedian1-8     8960811        6144771        -31.43%
BenchmarkMedian4-8     371908494      222312970      -40.22%
BenchmarkMedian8-8     4289995847     2485732320     -42.06%

benchmark              old allocs     new allocs     delta
BenchmarkMedian1-8     65542          65544          +0.00%
BenchmarkMedian4-8     65542          65544          +0.00%
BenchmarkMedian8-8     65545          65546          +0.00%

benchmark              old bytes     new bytes     delta
BenchmarkMedian1-8     3670588       3678857       +0.23%
BenchmarkMedian4-8     23606308      23626594      +0.09%
BenchmarkMedian8-8     84542640      84539120      -0.00%

@anthonynsimon anthonynsimon merged commit c86db79 into master Sep 7, 2016
@anthonynsimon anthonynsimon deleted the optimize-spatial-filters branch September 14, 2016 19:23
defart added a commit to defart/bild that referenced this pull request Sep 17, 2016
@defart
FloodFill (anthonynsimon#24)
1070157
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@anthonynsimon