-
Notifications
You must be signed in to change notification settings - Fork 50
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
Modify 05_slice example to use DefaultExecutionSpace #677
base: master
Are you sure you want to change the base?
Modify 05_slice example to use DefaultExecutionSpace #677
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code will need to be formatted - see https://github.com/ECP-copa/Cabana/blob/master/CONTRIBUTING.md for clang-format and pre-commit details
using MemorySpace = Kokkos::HostSpace; | ||
using ExecutionSpace = Kokkos::DefaultHostExecutionSpace; | ||
using ExecutionSpace = Kokkos::DefaultExecutionSpace; | ||
using MemorySpace = typename ExecutionSpace::memory_space; | ||
using DeviceType = Kokkos::Device<ExecutionSpace, MemorySpace>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note that #585 will change most of the examples to use memory_space
directly - it may be best to wait until that is merged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I will update the rest of the examples after #585 is merged. Will keep the PR open until then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dineshadepu should be okay to continue this PR now that we've merged the blocking PR - let me know if those changes are confusing in any way. But I do not plan to include this in the imminent release, so feel free to wait until that's finished
@@ -78,6 +77,9 @@ void sliceExample() | |||
Cabana::AoSoA<DataTypes, DeviceType, VectorLength> aosoa( "my_aosoa", | |||
num_tuple ); | |||
|
|||
// Create a mirror view of the aosoa on the host for accessing it legally |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Create a mirror view of the aosoa on the host for accessing it legally | |
// Create a mirror of the AoSoA on the host (this will not make a new copy if the memory is already allocated on the host, but will if it is in device memory) in order to print out the data. |
We declare the memory space in which the AoSoA will be allocated | ||
on the default execution space we are dealing with. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We declare the memory space in which the AoSoA will be allocated | |
on the default execution space we are dealing with. | |
We create the AoSoA particle data in the memory space associated | |
with the default execution space (see https://kokkos.github.io/kokkos-core-wiki/ProgrammingGuide/Initialization.html). |
Closes #624 .