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

Proposal: shake geometry handler for difficult geometry convergence cases #24

Open
yakutovicha opened this issue Nov 29, 2022 · 3 comments
Assignees

Comments

@yakutovicha
Copy link
Member

yakutovicha commented Nov 29, 2022

Add random XYZ 5-10 * 10^-2 A displacement for the atoms usually helps to destroy the symmetry and forces gaussian not to take it into account. It often helps with geometry convergence.

@eimrek
Copy link
Member

eimrek commented Nov 29, 2022

my preference would be to implement things like this in a RobustGaussianSCF workchain and not try to bloat the plugin too much, as it makes the plugin harder to maintain, for potential users to read/understand the code, and to prevent possible unexpected steps to run. This workchain could be part of this repo though. But on the other hand, in principle, as a feature that is disabled by default, this could also fit directly in the plugin.

@yakutovicha yakutovicha changed the title Proposal: shake geometry handler in of difficult convergence cases Proposal: shake geometry handler for difficult geometry convergence cases Nov 29, 2022
@eimrek
Copy link
Member

eimrek commented Nov 29, 2022

now that i reconsider, in the baseworkchain it would fit well 👍

@cpignedoli
Copy link
Member

One should not forget to pay attention to constraints in case of shaking

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

3 participants