Change Box API
Issue #67
closed
One suggestion: make specific classes which inherit from the base Box class e.g.:
box = freud.trajectory.SquareBox(L=L) box = freud.trajectory.CubicBox(L=L)
Dense constructor (Box) is always available to the user (Lx, Ly, Lz, xy, xz, yz, is2D)
Also: box might belong elsewhere (utils)?
Comments (5)
-
-
reporter Yeah, go for it. I really need to make freud a priority, but this paper is priority 1, 2, and 3 right now.
-
reporter - changed status to open
-
reporter -
assigned issue to
-
assigned issue to
-
- changed status to closed
Resolved in master.
- Log in to comment
I'm going to take this over if that's ok.