Convert Locality Module to Use Nanobind

tommy-waltmann commented 5 months ago

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 commented 4 months ago

There are some classes for histogramming in locality.py (_SpatialHistogram and _SpatialHistogram1D) that are unused by any compute in the locality module. They exist solely for the purpose of exposing common properties of classes that use binning to do their calculation. I choose to leave the conversion of those classes for a future PR because we have no way of testing them right now.

tommy-waltmann commented 4 months ago

Please ignore the clang-tidy errors on this PR, they are related to uncoverted modules.

glotzerlab / freud