search

omlins / ParallelStencil.jl

Package for writing high-level code for parallel high-performance stencil computations that can be deployed on both GPUs and CPUs
BSD 3-Clause "New" or "Revised" License
301 stars 31 forks source link

[JuliaCon/proceedings-review] @parallel keyword argument `loopopt` deprecated? #124

Open benegee opened 8 months ago

benegee commented 8 months ago

Dear authors of ParallelStencil.jl!

I am just going through https://github.com/JuliaCon/proceedings-review/issues/138

The example provided in your repo, https://github.com/omlins/ParallelStencil.jl/tree/JuliaConProceeding2022#50-lines-example-deployable-on-gpu-and-cpu, works perfectly fine.

The snippet in your paper, however, does not work with the current version of PaperStencil.jl because the keyword loopopt is reported as invalid. I assume it is deprecated by now. Maybe you could update the snippet in the paper? Or provide the version of your package used to generate the results in the paper?

omlins commented 8 months ago

Thanks for highlighting this. We will update snippet in the paper: loopopt has been replaced with memopt.