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

Catlearn MLNEB() initializes without assigning the correct calculator to the images of is_endpoint, fs_endpoint #98

Open
ssyrnyk opened this issue Aug 10, 2022 · 2 comments

Comments

@ssyrnyk
Copy link

ssyrnyk commented Aug 10, 2022

It seems that right now, when you create a mlneb object, it doesnt assign the calculator of the images to be the calculator passed in the ase_calc variable.

On my machine, I hackily patched this by just putting
for image in is_endpoint:\n image.calc = ase_calc
but IDK if this is a safe way to do it, or if its actually the correct solution.

@mhangaard
Copy link
Collaborator

mhangaard commented Nov 14, 2022

@ssyrnyk thanks for posting. Can you tell me which version of ase you are using? And a minimal example to produce your error would also be helpful.

@ssyrnyk
Copy link
Author

ssyrnyk commented Dec 2, 2022 via email

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