mphowardlab / azplugins

A HOOMD-blue component for soft matter simulations.
BSD 3-Clause "New" or "Revised" License
20 stars 12 forks source link

Add shift parameter to FENE bond potential #61

Closed judevishnu closed 2 years ago

judevishnu commented 2 years ago

I have made changes to the BondEvaluatorFene.h and associated files like module.cc, test_bond_fene.py and bond.py , like you have asked, in order to change the FENE bond potential to V = -0.5kR0R0ln(1-((r-delta)/R0)^2)

mphoward commented 2 years ago

There is a failing test (see output above) related to the energy in your new potential energy code. Could you please investigate? Note that you are able to run these tests locally as well using ctest if you want to check on your workstation before comitting.

judevishnu commented 2 years ago

Dear Mike,

Would you tell me the command to run these unit-tests. I think I found the mistake, but will run this before I commit.

Regards,

Jude


From: Michael Howard @.***> Sent: Tuesday, March 1, 2022 3:45:45 PM To: mphowardlab/azplugins Cc: Vishnu, Jude Ann; Author Subject: Re: [mphowardlab/azplugins] Feature fenebondshift (PR #61)

There is a failing test (see output above) related to the energy in your new potential energy code. Could you please investigate? Note that you are able to run these tests locally as well using ctest if you want to check on your workstation before comitting.

— Reply to this email directly, view it on GitHubhttps://github.com/mphowardlab/azplugins/pull/61#issuecomment-1055520750, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AX5W2YTPWGGT7RHD6BKXPILU5YURTANCNFSM5PQWPGUQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.***>

mphoward commented 2 years ago

How to run tests is described in the README, but in the build directory:

make test

will run all tests, or you can invoke ctest with options to run only certain tests like:

ctest -R azplugins-test_bond_fene-*

will get you all the tests whose name matches this pattern