Skip to content

Different halo#2812

Open
wsmoses wants to merge 3 commits intomainfrom
dhalo
Open

Different halo#2812
wsmoses wants to merge 3 commits intomainfrom
dhalo

Conversation

@wsmoses
Copy link
Copy Markdown
Member

@wsmoses wsmoses commented Apr 11, 2026

No description provided.

@wsmoses wsmoses requested a review from avik-pal April 11, 2026 17:37
Comment on lines +611 to +627
lows = ntuple(d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
clamp(a ÷ b, 1, size(arr, d))
end, ndims(arr))

highs = ntuple(d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
clamp(a ÷ b + 1, 1, size(arr, d))
end, ndims(arr))

rems = ntuple(d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
a % b
end, ndims(arr))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
lows = ntuple(d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
clamp(a ÷ b, 1, size(arr, d))
end, ndims(arr))
highs = ntuple(d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
clamp(a ÷ b + 1, 1, size(arr, d))
end, ndims(arr))
rems = ntuple(d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
a % b
end, ndims(arr))
lows = ntuple(
d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
clamp(a ÷ b, 1, size(arr, d))
end, ndims(arr)
)
highs = ntuple(
d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
clamp(a ÷ b + 1, 1, size(arr, d))
end, ndims(arr)
)
rems = ntuple(
d -> begin
a = (2 * I.I[d] - 1) * size(arr, d) + fsize[d]
b = 2 * fsize[d]
a % b
end, ndims(arr)
)

Comment on lines +659 to +660
src, final_size_halo, sharding, Reactant.InterpolationType.Nearest, halo_val;
src_halo=(0, 0, 0)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change
src, final_size_halo, sharding, Reactant.InterpolationType.Nearest, halo_val;
src_halo=(0, 0, 0)
src,
final_size_halo,
sharding,
Reactant.InterpolationType.Nearest,
halo_val;
src_halo=(0, 0, 0),

src, final_size_halo, sharding, Reactant.InterpolationType.Nearest, halo_val;
src_halo=(0, 0, 0)
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

idx = ntuple(d -> idx_map[I.I[d]], 3)
expected_halo[I] = src[CartesianIndex(idx)]
end

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[JuliaFormatter] reported by reviewdog 🐶

Suggested change

wsmoses added 2 commits April 11, 2026 18:17
This reverts commit 62b919c.
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.

1 participant