Skip to content
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

Dealing with hydride ligands #29

Open
lmoranglez opened this issue Oct 7, 2024 · 1 comment
Open

Dealing with hydride ligands #29

lmoranglez opened this issue Oct 7, 2024 · 1 comment

Comments

@lmoranglez
Copy link

I am currently using your code to build models with hydride ligands. I noticed that the 'relax' parameter is set to 'False' when hydrides are detected as SMILES (in io_process_input.py, lines 427 and 446). Could you please clarify the reasoning behind this decision? Does setting the 'relax' parameter to 'False' help to avoid significant convergence issues during the xtb optimization step?
Thank you for your assistance.

@mgt16-LANL
Copy link
Collaborator

Hi lmoranglez! Your intuition about it is generally correct. With XTB it would more often than not tend to move the hydride off the metal center and run into XTB calculations crashing ~1/2 the time. I am happy to add an option to override this default, like I had done for the oxo groups if you think it would be useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants