Skip to content

Remove ecut from PairDensity

Ask Hjorth Larsen requested to merge no-more-pair.ecut into master

After all this time, it turns out PairDensity does not in fact use ecut at all.

  • Remove ecut parameter and corresponding attribute from PairDensity
  • In particular, this means self.ecut can't be None. (Looking at you, EXX)
  • Other classes are now responsible for normalizing the unit of ecut if they want to do that
  • Since I got trouble with None vs default value (being 50. Sometimes), we now default to None which we normalize immediately to an actual value.
Edited by Ask Hjorth Larsen

Merge request reports