From 88925f9d91615bb2d62e82048e31004c5f54e200 Mon Sep 17 00:00:00 2001 From: Erik Faulhaber <44124897+efaulhaber@users.noreply.github.com> Date: Thu, 16 Jan 2025 18:32:40 +0100 Subject: [PATCH] [Hotfix] Fix boundscheck for `foreach_point_neighbor` (#95) * [Hotfix] Fix boundscheck for `foreach_point_neighbor` * Increase version number --- Project.toml | 2 +- src/neighborhood_search.jl | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Project.toml b/Project.toml index 8478cd5d..7c5047fc 100644 --- a/Project.toml +++ b/Project.toml @@ -1,7 +1,7 @@ name = "PointNeighbors" uuid = "1c4d5385-0a27-49de-8e2c-43b175c8985c" authors = ["Erik Faulhaber "] -version = "0.4.7-dev" +version = "0.4.7" [deps] Adapt = "79e6a3ab-5dfb-504d-930d-738a2a938a0e" diff --git a/src/neighborhood_search.jl b/src/neighborhood_search.jl index b7d4f893..9c50dd19 100644 --- a/src/neighborhood_search.jl +++ b/src/neighborhood_search.jl @@ -165,7 +165,7 @@ end @inline function foreach_point_neighbor(f, system_coords, neighbor_coords, neighborhood_search, points, parallel::Val{true}) # Explicit bounds check before the hot loop (or GPU kernel) - @boundscheck checkbounds(system_coords, ndims(neighborhood_search)) + @boundscheck checkbounds(system_coords, ndims(neighborhood_search), points) @threaded system_coords for point in points # Now we can assume that `point` is inbounds @@ -181,7 +181,7 @@ end neighborhood_search, points, backend::ParallelizationBackend) # Explicit bounds check before the hot loop (or GPU kernel) - @boundscheck checkbounds(system_coords, ndims(neighborhood_search)) + @boundscheck checkbounds(system_coords, ndims(neighborhood_search), points) @threaded backend for point in points # Now we can assume that `point` is inbounds @@ -195,7 +195,7 @@ end @inline function foreach_point_neighbor(f, system_coords, neighbor_coords, neighborhood_search, points, parallel::Val{false}) # Explicit bounds check before the hot loop - @boundscheck checkbounds(system_coords, ndims(neighborhood_search)) + @boundscheck checkbounds(system_coords, ndims(neighborhood_search), points) for point in points # Now we can assume that `point` is inbounds