Skip to content

Commit d90039f

Browse files
committed
Fix reapply for Sequential
1 parent 0afb4a2 commit d90039f

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

src/transforms/sequential.jl

+1-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ function reapply(s::Sequential, table, cache)
4040

4141
current = table
4242
for (ctransform, ccache) in zip(s.transforms, cache)
43-
current, _ = reapply(ctransform, current, ccache)
43+
current = reapply(ctransform, current, ccache)
4444
end
4545
current
4646
end

test/transforms.jl

+7
Original file line numberDiff line numberDiff line change
@@ -407,6 +407,13 @@
407407
@test isapprox(mean(r.x), mean(t.x), atol=1e-8)
408408
@test isapprox(std(r.x), std(t.x), atol=1e-8)
409409

410+
# reapply with Sequential transform
411+
t = Table(x=rand(1000))
412+
T = ZScore() Quantile()
413+
n1, c1 = apply(T, t)
414+
n2 = reapply(T, t, c1)
415+
@test n1 == n2
416+
410417
# reapply with Parallel transform
411418
t = Table(x=rand(1000))
412419
T = ZScore() Quantile()

0 commit comments

Comments
 (0)