Andrew S. Rosen
Andrew S. Rosen
Looking at your PR, indeed, it does not seem like it would be related (although you never know). I remain just as surprised as you that this behavior would not...
@janosh --- As a follow-up question, if the VASP job errors out, wouldn't it not be running anymore?
> i was wondering under what circumstances Custodian might think VASP errored out but where the VASP process is actually still running. is that possible? It would need to be...
@janosh: Just a brief update from me. I just used Custodian 2025.10.11 and had a toy job hit the maximum number of errors. Everything aborted as expected, with no lingering...
Thanks for reporting. If anyone decides to prepare a ix for this issue, I would recommend it being a two-step fix in the handler: 1. Try `KBLOWUP=False` 2. Then try...
@shyuep: Any idea why a Mac build would fail but Ubuntu would pass? The error seems unrelated to this PR by the way --- but I'm stumped. Edit: Passes now.
My proposal is as follows. 1. Remove `"auto_nbands"` from the `error_msgs` list. It is not an error but rather just a useful warning from VASP. Keeping it here could cause...
I think this is an interesting idea (even though workflow tools often have similar hashing). We already have hashing of the `Atoms` object. It would be possible to do the...
@kumaranu: The `tests/requirements.txt` file cannot have the various packages you have added to it. It is meant to include a minimum set of packages corresponding to the `tests-core` CI run....
@kumaranu: Thanks. I will review the PR once you undo all the unrelated changes you've made (there are still many) and remove commented out code blocks.