Skip to content

Commit

Permalink
Returning GetBondedAtomsSmart default to octahedral
Browse files Browse the repository at this point in the history
  • Loading branch information
aarongarrison committed Apr 16, 2024
1 parent 391d51c commit 79272c1
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions molSimplify/Classes/mol3D.py
Original file line number Diff line number Diff line change
Expand Up @@ -1979,7 +1979,7 @@ def getBondedAtomsOct(self, ind, CN=6, debug=False, flag_loose=False, atom_speci
'Error, mol3D could not understand connectivity in mol')
return nats

def getBondedAtomsSmart(self, idx, oct=False, strict_cutoff=False, catom_list=None):
def getBondedAtomsSmart(self, idx, oct=True, strict_cutoff=False, catom_list=None):
"""
Get the atoms bonded with the atom specified with the given index, using the molecular graph.
Creates graph if it does not exist.
Expand Down Expand Up @@ -5726,7 +5726,8 @@ def get_geometry_type_distance(self, max_dev=1e6, close_dev=1e-2,
if len(first_shell.findMetal()) > 1:
raise ValueError('Multimetal complexes are not yet handled.')
elif len(first_shell.findMetal(transition_metals_only=transition_metals_only)) == 1:
num_coord = len(first_shell.getBondedAtomsSmart(first_shell.findMetal(transition_metals_only=transition_metals_only)[0]))
#Use oct=False to ensure coordination number based on radius cutoffs only
num_coord = len(first_shell.getBondedAtomsSmart(first_shell.findMetal(transition_metals_only=transition_metals_only)[0]), oct=False)
else:
raise ValueError('No metal centers exist in this complex.')
if catoms_arr is not None and len(catoms_arr) != num_coord:
Expand Down

0 comments on commit 79272c1

Please sign in to comment.