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

Mesh bending rigidity potential #1792

Merged
merged 43 commits into from
Aug 15, 2024
Merged

Mesh bending rigidity potential #1792

merged 43 commits into from
Aug 15, 2024

Conversation

SchoeniPhlippsn
Copy link
Contributor

Description

This adds a bending rigidity potential which can be applied to the mesh data structure.

Motivation and context

This generalized bending rigidity allows for applying different bending moduli to a membrane which is represented by a particle mesh.

How has this been tested?

I added pytests and used this potential in my current studies.

Change log

Addition of a generalized mesh bending rigidity potential 

Checklist:

@SchoeniPhlippsn SchoeniPhlippsn requested review from a team and joaander and removed request for a team June 1, 2024 18:26
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale There has been no activity on this for some time. label Jun 30, 2024
@joaander joaander removed the stale There has been no activity on this for some time. label Jul 1, 2024
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale There has been no activity on this for some time. label Jul 21, 2024
@joaander joaander removed the stale There has been no activity on this for some time. label Jul 22, 2024
Copy link
Member

@joaander joaander left a comment

Choose a reason for hiding this comment

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

Thanks! I've made some suggestions to clean up the code.

As @tommy-waltmann said, you also need to make some modifications to the documentation will appear in the sphinx generated docs.

hoomd/md/BendingRigidityMeshForceCompute.cc Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceCompute.cc Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceCompute.cc Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceCompute.cc Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceCompute.cc Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceComputeGPU.cu Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceComputeGPU.cu Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceComputeGPU.cu Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceComputeGPU.cu Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceComputeGPU.cu Outdated Show resolved Hide resolved
@SchoeniPhlippsn
Copy link
Contributor Author

pre-commit.ci autofix

Copy link
Member

@joaander joaander left a comment

Choose a reason for hiding this comment

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

Thanks! I have a few more suggestions to reduce code duplication and fix a formatting issue in the docs. I will apply these suggestions and then merge.

hoomd/md/BendingRigidityMeshForceComputeGPU.cc Outdated Show resolved Hide resolved
hoomd/md/BendingRigidityMeshForceComputeGPU.h Outdated Show resolved Hide resolved
sphinx-doc/module-md-mesh-bending.rst Outdated Show resolved Hide resolved
@joaander joaander merged commit adc180c into trunk-minor Aug 15, 2024
34 checks passed
@joaander joaander deleted the mesh_bending_rigidity branch August 15, 2024 15:17
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.

3 participants