-
Notifications
You must be signed in to change notification settings - Fork 13
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
Would like to pass in more than just numbers and arrays #56
Comments
@michel2323 please let us know your thoughts on this issue. Thanks! |
Oh ok. This restrictive. Okay. I'm gonna take look. |
The problem with |
How do you want to decide a |
I looked at OpenMP and they have the For the CPU we could define our own or use it straight from KA. All this would also remove the need for |
This is when launching a kernel. As long as the struct meets |
@michel2323 @PhilipFackler I have a general rough draft of this idea in this PR. This is similar to how we launch CPU vs GPU kernels in the ITensors.jl package |
@kmp5VT thanks, we'd like to explore these ideas keeping the API really simple for end-users who would like to stay closer to their science and not necessarily computational aspects. Otherwise, there is very little added value for using JACC. A good exercise is to examine the final API and integration effort. Currently, we are prioritizing issues that allow for easy integration with apps and less maintenance. Most of the design decisions are from looking at apps and what they need and make incremental progress. @PhilipFackler posted a minimal example in #51 |
This commit causes me problems:
6d12657
parallel_for
now accepts onlyNumber
s andArray
s (orCuArray
s in the cuda case). I'd like to pass in instances of structs and tuples also. Perhaps the function could useVararg{Any}
?The text was updated successfully, but these errors were encountered: