JuliaParallel / Dagger.jl

A framework for out-of-core and parallel execution
Other
638 stars 67 forks source link

Fix error in documentation #427

Closed meggart closed 1 year ago

meggart commented 1 year ago

On Dagger 0.18.2 for me copy/pasting the sharding example resulted in the error

ERROR: MethodError: no method matching getindex(::Dagger.Chunk{Base.Threads.Atomic{Int64}, MemPool.DRef, OSProc, ProcessScope})
Stacktrace:
 [1] (::var"#9#10")(ctr::Dagger.Chunk{Base.Threads.Atomic{Int64}, MemPool.DRef, OSProc, ProcessScope})
   @ Main ./REPL[6]:2
 [2] iterate
   @ ./generator.jl:47 [inlined]
 [3] collect(itr::Base.Generator{Dagger.Shard, var"#9#10"})
   @ Base ./array.jl:782
 [4] map(f::Function, A::Dagger.Shard)
   @ Base ./abstractarray.jl:3289
 [5] top-level scope
   @ REPL[6]:2

I don't know if my fix is the idiomatic way to do it, but this worked for me at least.

jpsamaroo commented 1 year ago

Thanks @meggart , and sorry this took so long to get merged!