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
Union Potentials in HPMC #1725
Union Potentials in HPMC #1725
Conversation
r_cut update can be expensive now, but the number of types is usually very small.
Some potentials (e.g. Union) will need to know the r_cut of their child potential.
Commit bb71f90 passes hoomd-validation tests: glotzerlab/hoomd-validation#86 |
This pull request is complete and ready for final review. |
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.
Code looks good. I added some suggestions for more modern C++.
Co-authored-by: Brandon Butler <butlerbr@umich.edu>
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.
Nice work.
Co-authored-by: Tim Moore <mtimc@umich.edu>
Description
This PR adds union potentials in HPMC.
Motivation and context
Union potentials are a replacement for CPPPotentialUnion, which was based on llvm's JIT engine that is being removed from hoomd-blue in the version 5.0 release.
Partially Resolves #1652
How has this been tested?
I have written a set of unit tests for the new class. Should validation tests be added to hoomd-validation as well?
Change log
Checklist:
sphinx-doc/credits.rst
) in the pull request source branch.