Skip to content

Commit c384aad

Browse files
committed
Fix benchmarks
1 parent f1d6458 commit c384aad

File tree

3 files changed

+10
-6
lines changed

3 files changed

+10
-6
lines changed

benchmarks/n_body.jl

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,9 @@ function benchmark_n_body(neighborhood_search, coordinates_;
1818
# allows us to change the type of the array to run the benchmark on the GPU.
1919
# Passing `parallel = true` or `parallel = false` will not change anything here.
2020
coordinates = PointNeighbors.Adapt.adapt(parallelization_backend, coordinates_)
21-
nhs = PointNeighbors.Adapt.adapt(parallelization_backend, neighborhood_search)
21+
# Remove unnecessary data structures that are only used for initialization
22+
neighborhood_search_ = PointNeighbors.freeze_neighborhood_search(neighborhood_search)
23+
nhs = PointNeighbors.Adapt.adapt(parallelization_backend, neighborhood_search_)
2224

2325
# This preserves the data type of `coordinates`, which makes it work for GPU types
2426
mass = 1e10 * (rand!(similar(coordinates, size(coordinates, 2))) .+ 1)

benchmarks/plot.jl

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,10 +68,8 @@ function plot_benchmarks(benchmark, n_points_per_dimension, iterations;
6868
for i in eachindex(neighborhood_searches)
6969
neighborhood_search = neighborhood_searches[i]
7070
initialize!(neighborhood_search, coordinates, coordinates)
71-
# Remove unnecessary data structures that are only used for initialization
72-
neighborhood_search_ = PointNeighbors.freeze_neighborhood_search(neighborhood_search)
7371

74-
time = benchmark(neighborhood_search_, coordinates; parallelization_backend)
72+
time = benchmark(neighborhood_search, coordinates; parallelization_backend)
7573
times[iter, i] = time
7674
time_string = BenchmarkTools.prettytime(time * 1e9)
7775
println("$(neighborhood_searches_names[i])")

benchmarks/smoothed_particle_hydrodynamics.jl

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,9 @@ function benchmark_wcsph(neighborhood_search, coordinates;
5454
density_diffusion = density_diffusion)
5555

5656
system = PointNeighbors.Adapt.adapt(parallelization_backend, fluid_system)
57-
nhs = PointNeighbors.Adapt.adapt(parallelization_backend, neighborhood_search)
57+
# Remove unnecessary data structures that are only used for initialization
58+
neighborhood_search_ = PointNeighbors.freeze_neighborhood_search(neighborhood_search)
59+
nhs = PointNeighbors.Adapt.adapt(parallelization_backend, neighborhood_search_)
5860
semi = DummySemidiscretization(nhs, parallelization_backend)
5961

6062
v = PointNeighbors.Adapt.adapt(parallelization_backend,
@@ -105,7 +107,9 @@ function benchmark_wcsph_fp32(neighborhood_search, coordinates_;
105107
density_diffusion = density_diffusion)
106108

107109
system = PointNeighbors.Adapt.adapt(parallelization_backend, fluid_system)
108-
nhs = PointNeighbors.Adapt.adapt(parallelization_backend, neighborhood_search)
110+
# Remove unnecessary data structures that are only used for initialization
111+
neighborhood_search_ = PointNeighbors.freeze_neighborhood_search(neighborhood_search)
112+
nhs = PointNeighbors.Adapt.adapt(parallelization_backend, neighborhood_search_)
109113
semi = DummySemidiscretization(nhs, parallelization_backend)
110114

111115
v = PointNeighbors.Adapt.adapt(parallelization_backend,

0 commit comments

Comments
 (0)