-
Notifications
You must be signed in to change notification settings - Fork 47
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
working directory of Custodian does not match with that of VASP #2323
Comments
@honghuikim: thank you for this important bug report. This looks like a regression in behavior (ie it worked in a prior version but not now). I will ensure this is fixed in the afternoon and will add a test to make sure it does not happen again. |
@Andrew-S-Rosen Thank you for the quick fix. Do you know what version of Quacc worked before? It could be a problem with my environment tbh. I can run the same test on that version. |
I must admit, I'm not sure! It could also (potentially) be a bug with a recent version of custodian. I'll have to track down the ultimate cause. Doing |
I see. Actually, I had a similar problem before the recent update of Quacc (probably a month ago), but didn't take a look since it wasn't a big deal and I thought there was something to do with the old |
Looks like there is some digging for me to do when I'm back at my machine...! |
Thank you! Take your time. I'll also dig some more meantime. |
@honghuikim: It looks like this is a Custodian error. I determined this was likely the case because the Custodian files (e.g. We can further confirm this with the following minimal example I have reported upstream: materialsproject/custodian#340. I won't be able to solve this one immediately, but I think it's safe to say the problem is unfortunately further within the depths of the quacc ecosystem. In the meantime, unfortunately the only suggestion I have is to |
Looks like it's been patched. Doing I'll keep this issue report open until Custodian has a new build on PyPI. |
@Andrew-S-Rosen: Thank you very much for reporting the issue in Custodian. I just checked that it works now with the updated Custodian. Thank you! |
Closing as patched upstream. |
Details about the quacc environment
What is the issue?
When running VASP with Custodian, Custodian does not save the corrected input files (e.g., INCAR, KPOINTS) in the working directory of VASP.
It seems Custodian saves them in the base working directory where the calculation script is submitted. I have checked that this bug still remains the same when I'm not using workflow engines. I guess directory of the temporary VASP working directory (e.g.,
tmp-quacc-xxx
) is not transferred to Custodian in Vasp._run.In the below example, all 5 corrections from custodian are
Algo=Normal
, but it's not applied on thefailed-quacc-2024-07-02-06-06-37-930779-39481/INCAR
while it's applied on the INCAR (./INCAR
) in the base working directory.How can we easily reproduce the issue?
The text was updated successfully, but these errors were encountered: