Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
benegee opened this issue Nov 13, 2023 · 1 comment
Closed

Comments

@benegee
Copy link

benegee commented Nov 13, 2023

Dear authors of ParallelStencil.jl!

I am just going through JuliaCon/proceedings-review#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
Copy link
Owner

omlins commented Nov 14, 2023

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

@omlins omlins closed this as completed Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants