-
Notifications
You must be signed in to change notification settings - Fork 141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix AA fixed Coulomb energy observable #1841
Conversation
Can one of the admins verify this patch? |
Would you mind posting your output from the deterministic tests ( |
100% tests passed, 0 tests failed out of 229 Label Time Summary: Total Test time (real) = 12.44 sec |
ok to test |
Overloading CoulombPotential::addObservables to use with forces had some features missing, leading to fail all tests needing IonIon and ElecElec energy component, #1840.