Skip to content

Commit

Permalink
Remove old periodicity keyword (#60)
Browse files Browse the repository at this point in the history
* Fix copy full grid cell list

* Remove `periodicity` keyword

* Fix tests
  • Loading branch information
efaulhaber authored Jul 15, 2024
1 parent 772c8cf commit 2b5d3ce
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
3 changes: 1 addition & 2 deletions src/cell_lists/full_grid.jl
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ end
supported_update_strategies(::FullGridCellList) = (SemiParallelUpdate, SerialUpdate)

function FullGridCellList(; min_corner, max_corner, search_radius = 0.0,
periodicity = false, backend = DynamicVectorOfVectors{Int32},
backend = DynamicVectorOfVectors{Int32},
max_points_per_cell = 100)
# Pad domain to avoid 0 in cell indices due to rounding errors.
# We can't just use `eps()`, as one might use lower precision types.
Expand Down Expand Up @@ -180,6 +180,5 @@ function copy_cell_list(cell_list::FullGridCellList, search_radius, periodic_box
(; min_corner, max_corner) = cell_list

return FullGridCellList(; min_corner, max_corner, search_radius,
periodicity = !isnothing(periodic_box),
backend = typeof(cell_list.cells))
end
6 changes: 2 additions & 4 deletions test/neighborhood_search.jl
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,13 @@
periodic_box = periodic_boxes[i],
cell_list = FullGridCellList(; min_corner,
max_corner,
search_radius,
periodicity = true)),
search_radius)),
GridNeighborhoodSearch{NDIMS}(; search_radius, n_points,
periodic_box = periodic_boxes[i],
cell_list = FullGridCellList(; min_corner,
max_corner,
search_radius,
backend = Vector{Vector{Int32}},
periodicity = true)),
backend = Vector{Vector{Int32}})),
PrecomputedNeighborhoodSearch{NDIMS}(; search_radius, n_points,
periodic_box = periodic_boxes[i]),
]
Expand Down

0 comments on commit 2b5d3ce

Please sign in to comment.