-
Notifications
You must be signed in to change notification settings - Fork 68
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 ASE calculator returning stale forces (#261) #262
Fix ASE calculator returning stale forces (#261) #262
Conversation
…ing atoms via self.x instead of x = self.x, rm force call
…ver_update Feature/simon/nosehoover update
Codecov Report
@@ Coverage Diff @@
## master #262 +/- ##
==========================================
+ Coverage 54.50% 54.84% +0.34%
==========================================
Files 51 51
Lines 11919 11920 +1
==========================================
+ Hits 6496 6538 +42
+ Misses 5423 5382 -41
Continue to review full report at Codecov.
|
Hi, thanks for reporting the issue and make this fix! I'll review this pull request |
Fixes #261 by calling
ase.Calculator.calculate()
fromFLARE_Calculator.calculate()
as required by ASE.