Fix to_hoomd for convex polygons #245

janbridley · 2024-10-30T15:45:52Z

Description

HOOMD expects an (N,2) array instead of the (N,3) representation coxeter uses for polygons.

¹The change breaks (or has the potential to break) existing functionality.

I am familiar with the Contributing Guidelines.
I agree with the terms of the Contributor Agreement.
The changes introduced by this pull request are covered by existing or newly introduced tests.
I have updated the changelog and the credits.

for more information, see https://pre-commit.ci

janbridley and others added 4 commits October 30, 2024 10:59

Fix to_hoomd for convex polygons

2912f08

Fix test for polygon::to_hoomd

1ef24f1

[pre-commit.ci] auto fixes from pre-commit.com hooks

811b1fa

for more information, see https://pre-commit.ci

Fix support for python 3.9

f975d00

janbridley marked this pull request as ready for review October 30, 2024 17:54

janbridley merged commit 723ebd6 into master Oct 30, 2024
10 checks passed

janbridley deleted the fix/to_hoomd_2d branch October 30, 2024 17:54