Open ehthiede opened 8 months ago
As a part of this, it would be good to re-vamp something like the AtomCloud
. If we eventually want to optimize over parameters in an atomic representation, we should be storing these parameters in the leaves of some pytree (here, an eqx.Module
).
The question of "what do we do with AtomCloud
" is a good one... I think you might be suggesting merging it with an equinox module containing the data of the atomic representation. I think this is appealing down the road, but I kind of want to make those API changes a separate issue for future reference.
That makes sense!
I believe we now have resolved item 1 on this list with closing of #167 and merging of #218. Have written up the math in #216. I will add to this list
The atoms to voxel model is largely unproven, and we likely need some work to make it more physical. A nonexhaustive list of things to consider: