I think we should be more thorough with unit tests, especially when it comes to the numerics. For example, in the case of the OverlapModule, it would be nice to verify that the numerical gradient of the overlap with respect to the momentum of the ket Gaussian equals the analytical integral within some acceptable threshold. I say this not because I somehow doubt the implementation, but because we should really enforce this kind of behavior in the future. Whenever someone implements a new integral or other small formula, I think they should be required to add a unit test to verify the implementation is correct. @danielhollas do you have any opinions on this?
I think we should be more thorough with unit tests, especially when it comes to the numerics. For example, in the case of the OverlapModule, it would be nice to verify that the numerical gradient of the overlap with respect to the momentum of the ket Gaussian equals the analytical integral within some acceptable threshold. I say this not because I somehow doubt the implementation, but because we should really enforce this kind of behavior in the future. Whenever someone implements a new integral or other small formula, I think they should be required to add a unit test to verify the implementation is correct. @danielhollas do you have any opinions on this?