libAtoms / workflow

python workflow toolkit
GNU General Public License v2.0
32 stars 18 forks source link

DFT calculators do all sorts of different things with nonperiodic systems #259

Closed bernstei closed 10 months ago

bernstei commented 1 year ago

In the process of implementing a universal kspacing definition, I discovered that the various DFT calculators do all sorts of different things when they encounter non-periodic systems. It would be nice if all these behaviors were rationalized and made consistent, although it's possible that the input params are so different that this won't actually achieve much. But I think it's worth a review.

bernstei commented 1 year ago

This may not be as bad as I had initially thought. I'm working on a PR that will address this as well as #258 and the universal kspacing heuristic that's mentioned in #255