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.

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

Problem: no benchmarks #8

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Problem: no benchmarks #8

merged 1 commit into from
Feb 27, 2024

Conversation

vyskocilm
Copy link
Contributor

@vyskocilm vyskocilm commented Feb 27, 2024

Solution: add a single one

goos: linux
goarch: amd64
pkg: github.com/gomoni/it
cpu: AMD Ryzen 7 PRO 5850U with Radeon Graphics
BenchmarkRangeFilter
BenchmarkRangeFilter-16              448           2677041 ns/op
BenchmarkRangeFilter-16              435           2665242 ns/op
BenchmarkRangeFilter-16              442           2674867 ns/op
BenchmarkRangeFilter-16              432           2718882 ns/op
BenchmarkRangeFilter-16              408           2732315 ns/op
BenchmarkItFilterSlice
BenchmarkItFilterSlice-16             12          89357678 ns/op
BenchmarkItFilterSlice-16             12          89829099 ns/op
BenchmarkItFilterSlice-16             12          90112402 ns/op
BenchmarkItFilterSlice-16             12          89383558 ns/op
BenchmarkItFilterSlice-16             12          90150032 ns/op
BenchmarkItFilterFor
BenchmarkItFilterFor-16               19          58248848 ns/op
BenchmarkItFilterFor-16               19          58173282 ns/op
BenchmarkItFilterFor-16               18          58722532 ns/op
BenchmarkItFilterFor-16               19          58333300 ns/op
BenchmarkItFilterFor-16               19          58699434 ns/op
PASS
ok      github.com/gomoni/it    19.939s
michal@michal:~/projects/gomoni/it>

Solution: add a single one

```sh
goos: linux
goarch: amd64
pkg: github.com/gomoni/it
cpu: AMD Ryzen 7 PRO 5850U with Radeon Graphics
BenchmarkRangeFilter
BenchmarkRangeFilter-16              448           2677041 ns/op
BenchmarkRangeFilter-16              435           2665242 ns/op
BenchmarkRangeFilter-16              442           2674867 ns/op
BenchmarkRangeFilter-16              432           2718882 ns/op
BenchmarkRangeFilter-16              408           2732315 ns/op
BenchmarkItFilterSlice
BenchmarkItFilterSlice-16             12          89357678 ns/op
BenchmarkItFilterSlice-16             12          89829099 ns/op
BenchmarkItFilterSlice-16             12          90112402 ns/op
BenchmarkItFilterSlice-16             12          89383558 ns/op
BenchmarkItFilterSlice-16             12          90150032 ns/op
BenchmarkItFilterFor
BenchmarkItFilterFor-16               19          58248848 ns/op
BenchmarkItFilterFor-16               19          58173282 ns/op
BenchmarkItFilterFor-16               18          58722532 ns/op
BenchmarkItFilterFor-16               19          58333300 ns/op
BenchmarkItFilterFor-16               19          58699434 ns/op
PASS
ok      github.com/gomoni/it    19.939s
michal@90michal:~/projects/gomoni/it>
```
@vyskocilm vyskocilm merged commit d61e4e2 into main Feb 27, 2024
1 check passed
@DmitriyMV
Copy link
Contributor

DmitriyMV commented Feb 28, 2024

Latest fix with new benchmarks results is #10 and penalty for using iterators now is so much smaller.

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