beignet.func.space

[henry-isaacson]

beignet.func.space defines a simulation space.

This function is fundamental in constructing simulation spaces derived from subsets of $\mathbb{R}^{D}$ (where $D = 1$, $2$, or $3$) and is instrumental in setting up simulation environments with specific characteristics (e.g., periodic boundary conditions). The function returns a a displacement function and a shift function to compute particle interactions and movements in space.

[0x00b1]

This should be moved to beignet.operators._transform.

[henry-isaacson]

Do you want me to add to your operators PR?

[0x00b1]

@henry-isaacson Create a new PR

[0x00b1]

Why does Ruff add a newline? Can you fix that?

[henry-isaacson]

It's because it wants to put a newline before all function and method definitions. I couldn't find an explicit rule for it, but if we really want to fix it I can turn off ruff for that code block.

[0x00b1]

Should we rename this to transformation?

[henry-isaacson]

If we expect parallelepiped to be true in most cases then yes we could rename. We could also introduce a second variable with the code using dimensions when parallelepiped is false and transformation when true.

[0x00b1]

@henry-isaacson Breakout the transform and inverse transform operators into a separate PR.