Skip to content

Commit 8f572ee

Browse files
committed
ClimaDiagnostics downstream fix : API update
1 parent eaa60a7 commit 8f572ee

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/Remapping/distributed_remapping.jl

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -275,18 +275,20 @@ Remapper(
275275
horizontal_method::AbstractRemappingMethod = SpectralElementRemapping(),
276276
) = _Remapper(space; target_zcoords, target_hcoords, buffer_length, horizontal_method)
277277

278-
# Purely vertical case
278+
# Purely vertical case (horizontal_method accepted for uniform API, ignored)
279279
Remapper(
280280
space::Spaces.FiniteDifferenceSpace;
281281
target_zcoords::AbstractArray = default_target_zcoords(space),
282282
buffer_length::Int = 1,
283+
horizontal_method::AbstractRemappingMethod = SpectralElementRemapping(),
283284
) = _Remapper(space; target_zcoords, target_hcoords = nothing, buffer_length)
284285

285286
# Purely vertical, positional
286287
Remapper(
287288
space::Spaces.FiniteDifferenceSpace,
288289
target_zcoords::AbstractArray;
289290
buffer_length::Int = 1,
291+
horizontal_method::AbstractRemappingMethod = SpectralElementRemapping(),
290292
) = _Remapper(space; target_zcoords, target_hcoords = nothing, buffer_length)
291293

292294
# Purely horizontal case
@@ -533,12 +535,13 @@ function _Remapper(
533535
)
534536
end
535537

536-
# Constructor for the case with vertical spaces
538+
# Constructor for the case with vertical spaces (horizontal_method accepted, ignored)
537539
function _Remapper(
538540
space::Spaces.FiniteDifferenceSpace;
539541
target_zcoords::AbstractArray,
540542
target_hcoords::Nothing,
541543
buffer_length::Int = 1,
544+
horizontal_method::AbstractRemappingMethod = SpectralElementRemapping(),
542545
)
543546
comms_ctx = ClimaComms.context(space)
544547
FT = Spaces.undertype(space)

0 commit comments

Comments
 (0)