Andrew S. Rosen

Results 388 comments of 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.