From 6bac83d68809ef6e42ce27b702e9600b3b06977d Mon Sep 17 00:00:00 2001 From: Simone Silvestri <33547697+simone-silvestri@users.noreply.github.com> Date: Fri, 1 Mar 2024 15:49:48 -0500 Subject: [PATCH 1/2] bugfix --- test/test_distributed_hydrostatic_model.jl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/test_distributed_hydrostatic_model.jl b/test/test_distributed_hydrostatic_model.jl index 8511a4c8ad..dc2b7b5d6e 100644 --- a/test/test_distributed_hydrostatic_model.jl +++ b/test/test_distributed_hydrostatic_model.jl @@ -79,7 +79,7 @@ Ny = 32 for arch in archs @testset "Testing distributed solid body rotation" begin grid = LatitudeLongitudeGrid(arch, size = (Nx, Ny, 1), - halo = (3, 3, 3), + halo = (4, 4, 4), latitude = (-80, 80), longitude = (-160, 160), z = (-1, 0), From 8f43b2afb1927d9e94eda77311cda5bc65d1c172 Mon Sep 17 00:00:00 2001 From: Simone Silvestri <33547697+simone-silvestri@users.noreply.github.com> Date: Mon, 4 Mar 2024 11:41:46 -0500 Subject: [PATCH 2/2] remove comment --- src/Fields/field.jl | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/Fields/field.jl b/src/Fields/field.jl index 9337523cea..3d84363a08 100644 --- a/src/Fields/field.jl +++ b/src/Fields/field.jl @@ -741,15 +741,6 @@ end function fill_halo_regions!(field::Field, args...; kwargs...) reduced_dims = reduced_dimensions(field) - # To correctly fill the halo regions of fields with non-default indices, we'd have to - # offset indices in the fill halo regions kernels. - # For now we punt and don't support filling halo regions on windowed fields. - # Note that `FieldBoundaryConditions` _can_ filter boundary conditions in - # windowed directions: - # - # filtered_bcs = FieldBoundaryConditions(field.indices, field.boundary_conditions) - # - # which will be useful for implementing halo filling for windowed fields in the future. fill_halo_regions!(field.data, field.boundary_conditions, field.indices,