| [5fa1e86] | 1 | /* | 
|---|
|  | 2 | * Project: MoleCuilder | 
|---|
|  | 3 | * Description: creates and alters molecular systems | 
|---|
|  | 4 | * Copyright (C)  2020 Frederik Heber. All rights reserved. | 
|---|
|  | 5 | * | 
|---|
|  | 6 | * | 
|---|
|  | 7 | *   This file is part of MoleCuilder. | 
|---|
|  | 8 | * | 
|---|
|  | 9 | *    MoleCuilder is free software: you can redistribute it and/or modify | 
|---|
|  | 10 | *    it under the terms of the GNU General Public License as published by | 
|---|
|  | 11 | *    the Free Software Foundation, either version 2 of the License, or | 
|---|
|  | 12 | *    (at your option) any later version. | 
|---|
|  | 13 | * | 
|---|
|  | 14 | *    MoleCuilder is distributed in the hope that it will be useful, | 
|---|
|  | 15 | *    but WITHOUT ANY WARRANTY; without even the implied warranty of | 
|---|
|  | 16 | *    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the | 
|---|
|  | 17 | *    GNU General Public License for more details. | 
|---|
|  | 18 | * | 
|---|
|  | 19 | *    You should have received a copy of the GNU General Public License | 
|---|
|  | 20 | *    along with MoleCuilder.  If not, see <http://www.gnu.org/licenses/>. | 
|---|
|  | 21 | */ | 
|---|
|  | 22 |  | 
|---|
|  | 23 | /* | 
|---|
|  | 24 | * BondifyAction.cpp | 
|---|
|  | 25 | * | 
|---|
|  | 26 | *  Created on: Oct 07, 2020 | 
|---|
|  | 27 | *      Author: heber | 
|---|
|  | 28 | */ | 
|---|
|  | 29 |  | 
|---|
|  | 30 | // include config.h | 
|---|
|  | 31 | #ifdef HAVE_CONFIG_H | 
|---|
|  | 32 | #include <config.h> | 
|---|
|  | 33 | #endif | 
|---|
|  | 34 |  | 
|---|
|  | 35 | //#include "CodePatterns/MemDebug.hpp" | 
|---|
|  | 36 |  | 
|---|
|  | 37 | #include "CodePatterns/Assert.hpp" | 
|---|
|  | 38 | #include "CodePatterns/Log.hpp" | 
|---|
|  | 39 |  | 
|---|
|  | 40 | #include <string> | 
|---|
|  | 41 |  | 
|---|
|  | 42 | #include "Actions/AtomAction/BondifyAction.hpp" | 
|---|
| [9782e20] | 43 | #include "Actions/UndoRedoHelpers.hpp" | 
|---|
| [5fa1e86] | 44 | #include "Atom/atom.hpp" | 
|---|
| [9782e20] | 45 | #include "Atom/AtomicInfo.hpp" | 
|---|
| [5fa1e86] | 46 | #include "Bond/bond.hpp" | 
|---|
| [9782e20] | 47 | #include "Bond/BondInfo.hpp" | 
|---|
| [5fa1e86] | 48 | #include "Descriptors/AtomsWithinDistanceOfDescriptor.hpp" | 
|---|
|  | 49 | #include "Graph/BondGraph.hpp" | 
|---|
|  | 50 | #include "RandomNumbers/RandomNumberDistributionFactory.hpp" | 
|---|
|  | 51 | #include "RandomNumbers/RandomNumberDistribution.hpp" | 
|---|
|  | 52 | #include "RandomNumbers/RandomNumberGenerator.hpp" | 
|---|
|  | 53 | #include "RandomNumbers/RandomNumberGeneratorFactory.hpp" | 
|---|
|  | 54 | #include "World.hpp" | 
|---|
|  | 55 |  | 
|---|
|  | 56 | using namespace MoleCuilder; | 
|---|
|  | 57 |  | 
|---|
|  | 58 | // and construct the stuff | 
|---|
|  | 59 | #include "BondifyAction.def" | 
|---|
|  | 60 | #include "Action_impl_pre.hpp" | 
|---|
|  | 61 |  | 
|---|
|  | 62 | typedef std::vector< std::pair<atom *, int> > candidates_t; | 
|---|
|  | 63 |  | 
|---|
| [9782e20] | 64 | const double MAX_DISTANCE = 5.0; | 
|---|
|  | 65 |  | 
|---|
| [5fa1e86] | 66 | static int getNumberOfHydrogenAtoms( | 
|---|
|  | 67 | const atom *Walker) | 
|---|
|  | 68 | { | 
|---|
|  | 69 | int num_hydrogens = 0; | 
|---|
|  | 70 | const BondList& ListOfBonds = Walker->getListOfBonds(); | 
|---|
|  | 71 | for (BondList::const_iterator bonditer = ListOfBonds.begin(); | 
|---|
|  | 72 | bonditer != ListOfBonds.end(); ++bonditer) { | 
|---|
|  | 73 | num_hydrogens += (int)((*bonditer)->GetOtherAtom(Walker)->getElementNo() == 1); | 
|---|
|  | 74 | } | 
|---|
|  | 75 | return num_hydrogens; | 
|---|
|  | 76 | } | 
|---|
|  | 77 |  | 
|---|
|  | 78 | static candidates_t getCandidatesInVicinity( | 
|---|
|  | 79 | const BondGraph &BG, | 
|---|
|  | 80 | const atom * const Walker) | 
|---|
|  | 81 | { | 
|---|
|  | 82 | const std::vector< atom *> atoms_in_vicinity = World::getInstance().getAllAtoms( | 
|---|
| [9782e20] | 83 | AtomsWithinDistanceOf(MAX_DISTANCE, Walker->getPosition())); | 
|---|
| [5fa1e86] | 84 | candidates_t candidates; | 
|---|
|  | 85 | for (std::vector<atom *>::const_iterator iter = atoms_in_vicinity.begin(); | 
|---|
|  | 86 | iter != atoms_in_vicinity.end(); ++iter) { | 
|---|
|  | 87 | const atom *OtherWalker = *iter; | 
|---|
|  | 88 | // skip hydrogens | 
|---|
|  | 89 | if (OtherWalker->getElementNo() == 1) | 
|---|
|  | 90 | continue; | 
|---|
|  | 91 | // skip atoms that are bonded already | 
|---|
|  | 92 | if (OtherWalker->IsBondedTo(Walker)) | 
|---|
|  | 93 | continue; | 
|---|
| [9782e20] | 94 | const double distance = OtherWalker->getPosition().distance(Walker->getPosition()); | 
|---|
| [5fa1e86] | 95 | range<double> typical_distance = BG.getMinMaxDistance(Walker, OtherWalker); | 
|---|
|  | 96 | if (typical_distance.isInRange(distance)) { | 
|---|
|  | 97 | int num_hydrogens = getNumberOfHydrogenAtoms(OtherWalker); | 
|---|
|  | 98 | candidates.push_back( std::make_pair(const_cast<atom *>(OtherWalker), num_hydrogens)); | 
|---|
|  | 99 | } | 
|---|
|  | 100 | } | 
|---|
|  | 101 | return candidates; | 
|---|
|  | 102 | } | 
|---|
|  | 103 |  | 
|---|
|  | 104 | static int getTotalBondDegree( | 
|---|
|  | 105 | const atom * Walker) | 
|---|
|  | 106 | { | 
|---|
|  | 107 | int num_degreed_bonds = 0; | 
|---|
|  | 108 | const BondList& ListOfBonds = Walker->getListOfBonds(); | 
|---|
|  | 109 | for (BondList::const_iterator bonditer = ListOfBonds.begin(); | 
|---|
|  | 110 | bonditer != ListOfBonds.end(); ++bonditer) { | 
|---|
|  | 111 | num_degreed_bonds += (*bonditer)->getDegree(); | 
|---|
|  | 112 | } | 
|---|
|  | 113 | return num_degreed_bonds; | 
|---|
|  | 114 | } | 
|---|
|  | 115 |  | 
|---|
|  | 116 | class CandidatesAtValence { | 
|---|
|  | 117 | public: | 
|---|
|  | 118 | typedef std::multimap<int, atom*> num_hydrogens_atom_map_t; | 
|---|
|  | 119 | typedef std::vector< std::pair<atom *, int> > candidates_hydrogens_to_remove_t; | 
|---|
|  | 120 | typedef std::vector<candidates_hydrogens_to_remove_t> all_candidates_t; | 
|---|
|  | 121 |  | 
|---|
|  | 122 | int max_hydrogens; | 
|---|
|  | 123 |  | 
|---|
|  | 124 | CandidatesAtValence(const int _max_hydrogens) : max_hydrogens(_max_hydrogens) {} | 
|---|
|  | 125 |  | 
|---|
|  | 126 | void setMaxHydrogens(const int _max_hydrogens) { | 
|---|
|  | 127 | max_hydrogens = _max_hydrogens; | 
|---|
|  | 128 | } | 
|---|
|  | 129 |  | 
|---|
|  | 130 | all_candidates_t operator()( | 
|---|
|  | 131 | const int open_valence, | 
|---|
| [9782e20] | 132 | const num_hydrogens_atom_map_t& sorted_candidates) const | 
|---|
| [5fa1e86] | 133 | { | 
|---|
|  | 134 | all_candidates_t all_candidates; | 
|---|
|  | 135 | if (sorted_candidates.empty()) | 
|---|
|  | 136 | return all_candidates; | 
|---|
|  | 137 |  | 
|---|
|  | 138 |  | 
|---|
|  | 139 | /* pick from beginning (i.e. smallest valence first) and gather all | 
|---|
|  | 140 | * possible candidate sets. | 
|---|
|  | 141 | */ | 
|---|
|  | 142 | int remaining_valence = open_valence; | 
|---|
| [9782e20] | 143 | num_hydrogens_atom_map_t::const_iterator iter = sorted_candidates.begin(); | 
|---|
| [5fa1e86] | 144 | candidates_hydrogens_to_remove_t candidate_set; | 
|---|
|  | 145 | exploreCandidateSetRecursively( | 
|---|
|  | 146 | iter, sorted_candidates.end(), | 
|---|
|  | 147 | candidate_set, remaining_valence, | 
|---|
|  | 148 | all_candidates); | 
|---|
|  | 149 |  | 
|---|
|  | 150 | for (all_candidates_t::const_iterator all_iter = all_candidates.begin(); | 
|---|
|  | 151 | all_iter != all_candidates.end(); ++all_iter) { | 
|---|
|  | 152 | LOG(2, "DEBUG: Candidate set is " << candidateToString(*all_iter)); | 
|---|
|  | 153 | } | 
|---|
|  | 154 |  | 
|---|
|  | 155 | return all_candidates; | 
|---|
|  | 156 | } | 
|---|
|  | 157 |  | 
|---|
|  | 158 | void exploreCandidateSetRecursively( | 
|---|
|  | 159 | num_hydrogens_atom_map_t::const_iterator iter, | 
|---|
|  | 160 | num_hydrogens_atom_map_t::const_iterator iter_end, | 
|---|
|  | 161 | candidates_hydrogens_to_remove_t candidate_set, | 
|---|
|  | 162 | int remaining_valence, | 
|---|
|  | 163 | all_candidates_t& all_candidates | 
|---|
|  | 164 | ) const | 
|---|
|  | 165 | { | 
|---|
|  | 166 | const int & num_hydrogens = iter->first; | 
|---|
|  | 167 | atom * const walker = iter->second; | 
|---|
|  | 168 | ++iter; | 
|---|
|  | 169 | const int usable_valence = | 
|---|
|  | 170 | std::min( | 
|---|
|  | 171 | std::min(num_hydrogens, remaining_valence), | 
|---|
|  | 172 | max_hydrogens | 
|---|
|  | 173 | ); | 
|---|
|  | 174 | candidate_set.push_back( | 
|---|
|  | 175 | std::make_pair(walker, 0) | 
|---|
|  | 176 | ); | 
|---|
|  | 177 | for (int i=0;i<= usable_valence; ++i) { | 
|---|
|  | 178 | candidate_set.back().second = i; | 
|---|
|  | 179 | // check whether we are complete with the candidate_set | 
|---|
|  | 180 | if (remaining_valence - i <= 0) { | 
|---|
|  | 181 | // we have a complete set | 
|---|
|  | 182 | ASSERT(remaining_valence -i == 0, | 
|---|
|  | 183 | "exploreCandidateSetRecursively() - usable valence "+toString(i) | 
|---|
|  | 184 | +" must always be less or equal remaining valence "+toString(remaining_valence)); | 
|---|
|  | 185 | all_candidates.push_back(candidate_set); | 
|---|
|  | 186 | } else { | 
|---|
|  | 187 | // not complete, need to recurse further if we can | 
|---|
|  | 188 | if (iter != iter_end) | 
|---|
|  | 189 | exploreCandidateSetRecursively( | 
|---|
|  | 190 | iter, | 
|---|
|  | 191 | iter_end, | 
|---|
|  | 192 | candidate_set, | 
|---|
|  | 193 | remaining_valence-i, | 
|---|
|  | 194 | all_candidates); | 
|---|
|  | 195 | } | 
|---|
|  | 196 | } | 
|---|
|  | 197 | } | 
|---|
|  | 198 |  | 
|---|
|  | 199 | static std::string candidateToString( | 
|---|
|  | 200 | const candidates_hydrogens_to_remove_t &candidates) | 
|---|
|  | 201 | { | 
|---|
|  | 202 | std::stringstream output; | 
|---|
|  | 203 | for (candidates_hydrogens_to_remove_t::const_iterator iter = candidates.begin(); | 
|---|
|  | 204 | iter != candidates.end(); ++iter) | 
|---|
|  | 205 | output << "[" << *iter->first << "," << iter->second << "], "; | 
|---|
|  | 206 | return output.str(); | 
|---|
|  | 207 | } | 
|---|
|  | 208 |  | 
|---|
|  | 209 |  | 
|---|
|  | 210 | }; | 
|---|
|  | 211 |  | 
|---|
|  | 212 | /** =========== define the function ====================== */ | 
|---|
|  | 213 | ActionState::ptr AtomBondifyAction::performCall() { | 
|---|
|  | 214 | // ensure we have one selected atom | 
|---|
|  | 215 | World &world = World::getInstance(); | 
|---|
|  | 216 | const BondGraph &BG = *world.getBondGraph(); | 
|---|
|  | 217 |  | 
|---|
|  | 218 | // check precondition | 
|---|
|  | 219 | const std::vector<atom *> atoms = world.getSelectedAtoms(); | 
|---|
|  | 220 | if (atoms.size() != 1) { | 
|---|
|  | 221 | STATUS("Exactly one atom must be selected for bondify."); | 
|---|
|  | 222 | return Action::failure; | 
|---|
|  | 223 | } | 
|---|
|  | 224 | atom *Walker = atoms[0]; | 
|---|
|  | 225 |  | 
|---|
|  | 226 | // check whether atom has unoccupied valence orbitals | 
|---|
|  | 227 | const int num_valence = Walker->getType()->getNoValenceOrbitals(); | 
|---|
|  | 228 |  | 
|---|
|  | 229 | // Look at all bond neighbors | 
|---|
|  | 230 | const int num_degreed_bonds = getTotalBondDegree(Walker); | 
|---|
|  | 231 | int open_valence = num_valence - num_degreed_bonds; | 
|---|
|  | 232 | LOG(1, "INFO: We have " << open_valence << " unoccupied  valence"); | 
|---|
|  | 233 |  | 
|---|
|  | 234 | if (open_valence <= 0) { | 
|---|
| [9782e20] | 235 | STATUS("Nothing to do. Atom is saturated already, we need some unoccupied valence orbitals to have something to work on."); | 
|---|
| [5fa1e86] | 236 | return Action::success; | 
|---|
|  | 237 | } | 
|---|
|  | 238 |  | 
|---|
|  | 239 | // gather all atoms in vicinity | 
|---|
|  | 240 | candidates_t candidates = getCandidatesInVicinity(BG, Walker); | 
|---|
|  | 241 |  | 
|---|
|  | 242 |  | 
|---|
|  | 243 | /** We revert the map's key and values, as we want the entries | 
|---|
| [9782e20] | 244 | * sorted by the number of hydrogens. | 
|---|
| [5fa1e86] | 245 | */ | 
|---|
|  | 246 | std::multimap<int, atom*> sorted_candidates; | 
|---|
|  | 247 | for (candidates_t::const_iterator iter = candidates.begin(); | 
|---|
|  | 248 | iter != candidates.end(); ++iter) { | 
|---|
|  | 249 | if (iter->second > 0) { | 
|---|
|  | 250 | sorted_candidates.insert( std::make_pair(iter->second, iter->first) ); | 
|---|
|  | 251 | } | 
|---|
|  | 252 | } | 
|---|
|  | 253 |  | 
|---|
|  | 254 | /** | 
|---|
|  | 255 | * Try to find the maximum number of bonds to use instead of having to | 
|---|
|  | 256 | * add hydrogens in the end. | 
|---|
|  | 257 | */ | 
|---|
|  | 258 | CandidatesAtValence::all_candidates_t all_candidates; | 
|---|
|  | 259 | CandidatesAtValence candidateGetter(params.max_hydrogens.get()); | 
|---|
|  | 260 | ++open_valence; | 
|---|
|  | 261 | while ((all_candidates.size() == 0) && (--open_valence > 0)) { | 
|---|
|  | 262 | all_candidates = candidateGetter(open_valence, sorted_candidates); | 
|---|
|  | 263 | } | 
|---|
|  | 264 | if (all_candidates.empty()) { | 
|---|
|  | 265 | STATUS("Could not find any suitable candidate sets."); | 
|---|
|  | 266 | return Action::success; | 
|---|
|  | 267 | } | 
|---|
|  | 268 |  | 
|---|
|  | 269 | // pick one set at random | 
|---|
|  | 270 | const std::string oldtype = RandomNumberDistributionFactory::getConstInstance().getCurrentTypeName(); | 
|---|
|  | 271 | RandomNumberDistributionFactory::getInstance().setCurrentType("uniform_01"); | 
|---|
|  | 272 | const RandomNumberGenerator& rng = RandomNumberGeneratorFactory::getConstInstance().makeRandomNumberGenerator(); | 
|---|
|  | 273 | int id = (int)(rng()*(all_candidates.size()-1)); | 
|---|
|  | 274 | RandomNumberDistributionFactory::getInstance().setCurrentType(oldtype); | 
|---|
|  | 275 | CandidatesAtValence::candidates_hydrogens_to_remove_t &picked_set = all_candidates[id]; | 
|---|
|  | 276 | LOG(1, "INFO: Picked candidate set is " << CandidatesAtValence::candidateToString(picked_set)); | 
|---|
|  | 277 |  | 
|---|
|  | 278 | // execute that | 
|---|
| [9782e20] | 279 | std::vector<AtomicInfo> removedHydrogens; | 
|---|
|  | 280 | std::vector<BondInfo> addedBonds; | 
|---|
| [5fa1e86] | 281 | int removed_hydrogens = 0; | 
|---|
|  | 282 | int added_bonds = 0; | 
|---|
|  | 283 | for (CandidatesAtValence::candidates_hydrogens_to_remove_t::iterator iter = picked_set.begin(); | 
|---|
|  | 284 | iter != picked_set.end(); ++iter) { | 
|---|
|  | 285 | atom * const other_walker = iter->first; | 
|---|
|  | 286 | const int &num_hydrogens = iter->second; | 
|---|
|  | 287 |  | 
|---|
|  | 288 | // remove the hydrogens closest to Walker | 
|---|
|  | 289 | typedef std::map<double, atom *> closest_map_t; | 
|---|
|  | 290 | closest_map_t closest_map; | 
|---|
|  | 291 | const BondList& ListOfBonds = other_walker->getListOfBonds(); | 
|---|
|  | 292 | for (BondList::const_iterator bonditer = ListOfBonds.begin(); | 
|---|
|  | 293 | bonditer != ListOfBonds.end(); ++bonditer) { | 
|---|
|  | 294 | atom * const hydrogen = (*bonditer)->GetOtherAtom(other_walker); | 
|---|
|  | 295 | if (hydrogen->getElementNo() != 1) | 
|---|
|  | 296 | continue; | 
|---|
|  | 297 | closest_map.insert( | 
|---|
|  | 298 | std::make_pair( | 
|---|
|  | 299 | Walker->getPosition().distance(hydrogen->getPosition()), | 
|---|
|  | 300 | hydrogen | 
|---|
|  | 301 | ) | 
|---|
|  | 302 | ); | 
|---|
|  | 303 | } | 
|---|
|  | 304 | ASSERT( closest_map.size() >= (size_t)num_hydrogens, | 
|---|
|  | 305 | "AtomBondifyAction::performCall() - Atom "+toString(*other_walker) | 
|---|
|  | 306 | +" has less hydrogens "+toString(closest_map.size()) | 
|---|
|  | 307 | +" than expected "+toString(num_hydrogens)); | 
|---|
|  | 308 | closest_map_t::iterator removeiter = closest_map.begin(); | 
|---|
|  | 309 | for (int i=0;i<num_hydrogens; ++i) { | 
|---|
|  | 310 | ASSERT( removeiter != closest_map.end(), | 
|---|
|  | 311 | "AtomBondifyAction::performCall() - already at end of closest map?"); | 
|---|
|  | 312 | LOG(2, "DEBUG: Removing hydrogen at distance " << removeiter->first | 
|---|
|  | 313 | << " to Walker at " << Walker->getPosition()); | 
|---|
| [9782e20] | 314 | removedHydrogens.push_back(AtomicInfo(*removeiter->second)); | 
|---|
| [5fa1e86] | 315 | world.destroyAtom(removeiter->second); | 
|---|
|  | 316 | removeiter->second = NULL; | 
|---|
|  | 317 | ++removeiter; | 
|---|
|  | 318 | ++removed_hydrogens; | 
|---|
|  | 319 | } | 
|---|
|  | 320 |  | 
|---|
|  | 321 | // add the bond | 
|---|
|  | 322 | bond::ptr new_bond = Walker->addBond(other_walker); | 
|---|
| [9782e20] | 323 | addedBonds.push_back(BondInfo(new_bond)); | 
|---|
| [5fa1e86] | 324 | new_bond->setDegree(num_hydrogens); | 
|---|
|  | 325 | ++added_bonds; | 
|---|
|  | 326 | LOG(2, "DEBUG: Added new bond " << *new_bond); | 
|---|
|  | 327 | } | 
|---|
|  | 328 | ASSERT( removed_hydrogens == open_valence, | 
|---|
|  | 329 | "AtomBondifyAction::performCall() - candidate set has not the number of hydrogens " | 
|---|
|  | 330 | +toString(removed_hydrogens)+" to match the unoccupied valence "+toString(open_valence)); | 
|---|
|  | 331 |  | 
|---|
|  | 332 | LOG(1, "INFO: Removed " << removed_hydrogens << " hydrogen atoms and added " | 
|---|
|  | 333 | << added_bonds << " new bonds."); | 
|---|
|  | 334 |  | 
|---|
| [9782e20] | 335 | return ActionState::ptr(new AtomBondifyState(removedHydrogens, addedBonds, params)); | 
|---|
| [5fa1e86] | 336 | } | 
|---|
|  | 337 |  | 
|---|
|  | 338 | ActionState::ptr AtomBondifyAction::performUndo(ActionState::ptr _state) { | 
|---|
| [9782e20] | 339 | AtomBondifyState *state = assert_cast<AtomBondifyState*>(_state.get()); | 
|---|
| [5fa1e86] | 340 |  | 
|---|
| [9782e20] | 341 | AddAtomsFromAtomicInfo(state->removedHydrogens); | 
|---|
|  | 342 | RemoveBondsFromBondInfo(state->addedBonds); | 
|---|
| [5fa1e86] | 343 |  | 
|---|
| [9782e20] | 344 | return ActionState::ptr(_state); | 
|---|
| [5fa1e86] | 345 | } | 
|---|
|  | 346 |  | 
|---|
|  | 347 | ActionState::ptr AtomBondifyAction::performRedo(ActionState::ptr _state){ | 
|---|
| [9782e20] | 348 | AtomBondifyState *state = assert_cast<AtomBondifyState*>(_state.get()); | 
|---|
| [5fa1e86] | 349 |  | 
|---|
| [9782e20] | 350 | RemoveAtomsFromAtomicInfo(state->removedHydrogens); | 
|---|
|  | 351 | AddBondsFromBondInfo(state->addedBonds); | 
|---|
| [5fa1e86] | 352 |  | 
|---|
| [9782e20] | 353 | return ActionState::ptr(_state); | 
|---|
| [5fa1e86] | 354 | } | 
|---|
|  | 355 |  | 
|---|
|  | 356 | bool AtomBondifyAction::canUndo() { | 
|---|
| [9782e20] | 357 | return true; | 
|---|
| [5fa1e86] | 358 | } | 
|---|
|  | 359 |  | 
|---|
|  | 360 | bool AtomBondifyAction::shouldUndo() { | 
|---|
| [9782e20] | 361 | return true; | 
|---|
| [5fa1e86] | 362 | } | 
|---|
|  | 363 | /** =========== end of function ====================== */ | 
|---|