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

Move neighbor list into force #150

Merged
merged 1 commit into from
Nov 19, 2024
Merged

Conversation

streeve
Copy link
Collaborator

@streeve streeve commented Nov 19, 2024

Template on memory since it now stores class data. Use default execution space for now. Will greatly simplify dual horizon approach #146 and contact/DEM lists #143 (to be rebased on this PR)

@streeve streeve requested a review from abisner November 19, 2024 13:48
@streeve streeve self-assigned this Nov 19, 2024
Copy link
Collaborator

@abisner abisner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Take it or leave it on the renaming suggestions.

src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
src/CabanaPD_Force.hpp Outdated Show resolved Hide resolved
@abisner
Copy link
Collaborator

abisner commented Nov 19, 2024

I assume that these changes were pulled from #143, which will subsequently be rebased as a dependent PR.

@streeve streeve force-pushed the neighbors_in_force branch 2 times, most recently from 6662b98 to 9cee462 Compare November 19, 2024 16:17
enables contact and dual horizon
@streeve streeve merged commit 5e66739 into ORNL:main Nov 19, 2024
14 checks passed
@streeve streeve deleted the neighbors_in_force branch November 19, 2024 17:32
@streeve streeve restored the neighbors_in_force branch November 20, 2024 22:23
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

Successfully merging this pull request may close these issues.

2 participants