-
Notifications
You must be signed in to change notification settings - Fork 17
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
Material centric2 #213
Material centric2 #213
Conversation
Just a note to any reviewers. We changed the cmake to do a more monolithic build because we were having issues with compilation. In the end, our issues were probably not from creating libraries and then linking them. We may revert back at some point, but only after further consideration and changes to how we handle everything in the "Common" directory. |
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.
lgtm
Material centric implementation |
Description
The new material centric implementation is up and running on CPUs and GPUs. This version loops over the elements in the mesh that have a material (key to eliminating thread divergence). The material centric data structures also allow developers to append data to the end of the material storage, key for any future remap research (for ALE) or future Eulerian AMR research. The coding also solves an energy equation per material.
This version also renames variables (more of them) according to the google style guide and reorganizes the files for cleanliness.
Type of change
Please select all relevant options
How Has This Been Tested?
Ran tests on CPUs and GPUs
Test Configuration:
OS version: Mac OS
Hardware: x86 CPU and NVidia GPU
Compiler: clang with openMP backend
OS version: Linux,
Hardware: x86 CPU with a V100 GPU
Compiler: gcc plus CUDA
Checklist: