qmcpack
qmcpack copied to clipboard
OperatorEstBase DataLocality modes
Is your feature request related to a problem? Please describe.
- Why OperatorEstBase::data_locality_ is not const?
- Should OperatorEstBase::myName const and set via constructor?
- SpinDensityNew implements copy constructor based on data_locality_ state but MomentumDistribution implements similar things in clone()
Please consolidate the design and add documentation.
I think 1 and 2 are probably correct. I'm working through my too clever copy constructor in 3 but I'm not sure that will put the logic in clone.