freud icon indicating copy to clipboard operation
freud copied to clipboard

Convert Locality Module to Use Nanobind

Open tommy-waltmann opened this issue 7 months ago • 0 comments

Description

This PR converts the locality module to use nanobind. It also enables the data module because it's pure python, but uses PeriodicBuffer internally.

Motivation and Context

We are removing Cython from freud, this is one of a series of PRs that accomplishes that goal.

How Has This Been Tested?

Testing has been added in the CI for newly enabled modules.

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds or improves functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Documentation improvement (updates to user guides, docstrings, or developer docs)

Checklist:

  • [ ] I have read the CONTRIBUTING document.
  • [ ] My code follows the code style of this project.
  • [ ] I have updated the documentation (if relevant).
  • [ ] I have added tests that cover my changes (if relevant).
  • [ ] All new and existing tests passed.
  • [ ] I have updated the credits.
  • [ ] I have updated the Changelog.

tommy-waltmann avatar Jun 25 '24 21:06 tommy-waltmann