Merge pull request #107 from SciML/myb/mapreduce

Fix mapreduce

Author: ChrisRackauckas

Project.toml

@@ -1,7 +1,7 @@
 name = "RecursiveArrayTools"
 uuid = "731186ca-8d62-57ce-b412-fbd966d074cd"
 authors = ["Chris Rackauckas <[email protected]>"]
-version = "2.4.3"
+version = "2.4.4"
 
 [deps]
 ArrayInterface = "4fba245c-0d91-5ea0-9b3e-6abc04ee57a9"

src/vector_of_array.jl

@@ -151,6 +151,8 @@ end
   VA.t,VA.u
 end
 
+Base.mapreduce(f,op,A::AbstractVectorOfArray) = mapreduce(f,op,(mapreduce(f,op,x) for x in A.u))
+
 ## broadcasting
 
 struct VectorOfArrayStyle{N} <: Broadcast.AbstractArrayStyle{N} end # N is only used when voa sees other abstract arrays

test/basic_indexing.jl

@@ -4,6 +4,7 @@ using RecursiveArrayTools, Test
 recs = [[1, 2, 3], [4, 5, 6], [7, 8, 9]]
 testa = cat(recs..., dims=2)
 testva = VectorOfArray(recs)
+@test maximum(testva) == maximum(maximum.(recs))
 
 # broadcast with array
 X = rand(3, 3)