Skip to content

Commit

Permalink
Merge pull request #278 from shishaochen/remove-useless-codes
Browse files Browse the repository at this point in the history
Remove duplicated calculation of coordinates.
  • Loading branch information
joanimato authored Jan 10, 2025
2 parents d8a0d53 + c4a183b commit ebf3a92
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/lib/monte_carlo.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -60,12 +60,9 @@ void monte_carlo::operator()(model& m, output_container& out, const precalculate
if(step == 0 || metropolis_accept(tmp.e, candidate.e, temperature, generator)) {
tmp = candidate;

m.set(tmp.c); // FIXME? useless?

// FIXME only for very promising ones
if(tmp.e < best_e || out.size() < num_saved_mins) {
quasi_newton_par(m, p, ig, tmp, g, authentic_v, evalcount);
m.set(tmp.c); // FIXME? useless?
tmp.coords = m.get_heavy_atom_movable_coords();
add_to_output_container(out, tmp, min_rmsd, num_saved_mins); // 20 - max size
if(tmp.e < best_e)
Expand Down

0 comments on commit ebf3a92

Please sign in to comment.