custodian icon indicating copy to clipboard operation
custodian copied to clipboard

VASP command with more than 1 line

Open JaGeo opened this issue 2 years ago • 8 comments

It's currently not possible to use a VASP (or Lobster) command including more than 1 line (e.g., to change the number of threads within one job script).

I did not find a solution that is easy to implement yet and wanted to ask if there is an interest in such a feature and/or if someone has ideas on how to implement it.

Can anyone think of a command that could easily replace this one here? https://github.com/materialsproject/custodian/blob/202b941ab9c5d8fc3b9b9e5bb26a9e280ba1e0ea/custodian/vasp/jobs.py#L878

JaGeo avatar Jul 19 '23 13:07 JaGeo

What's the issue with a multiline VASP command?

janosh avatar Jul 22 '23 19:07 janosh

It's not working to provide a command longer than 1 line with popen.

JaGeo avatar Jul 22 '23 19:07 JaGeo

What if we refactor cmd to be list instead of str?

janosh avatar Jul 22 '23 19:07 janosh

Would this work in combination with a typical configuration file?

JaGeo avatar Jul 22 '23 20:07 JaGeo

Sorry, didn't read carefully. Actually cmd is already a list

https://github.com/materialsproject/custodian/blob/202b941ab9c5d8fc3b9b9e5bb26a9e280ba1e0ea/custodian/vasp/jobs.py#L863

janosh avatar Jul 22 '23 20:07 janosh