-
Notifications
You must be signed in to change notification settings - Fork 189
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
aovid error with widom insertion. exclusion radius should not have any effect here #2294
Conversation
Codecov Report
@@ Coverage Diff @@
## python #2294 +/- ##
=======================================
+ Coverage 71% 71% +<1%
=======================================
Files 380 380
Lines 18938 18936 -2
=======================================
+ Hits 13586 13588 +2
+ Misses 5352 5348 -4
Continue to review full report at Codecov.
|
src/core/reaction_ensemble.cpp
Outdated
E_pot_new = std::numeric_limits<double>::max(); | ||
else | ||
E_pot_new = calculate_current_potential_energy_of_system(); | ||
double E_pot_new= calculate_current_potential_energy_of_system(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be const
now
LB test fails because it does not science the desired accuracy. Untouched. |
This fix workes.
Am Sa., 3. Nov. 2018, 06:20 hat Kai Szuttor <notifications@github.com>
geschrieben:
… @jonaslandsgesell <https://github.com/jonaslandsgesell>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2294 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE8wPb5K4XwbjRBj1xn0uj-2aGLInNfOks5urSeVgaJpZM4XIrab>
.
|
style check failed |
@jonaslandsgesell what is the status on this? |
aovid error with widom insertion. exclusion radius should not have any effect here
No description provided.