search

MineralsCloud / Express.jl

Express: a high-level, extensible workflow framework for accelerating ab initio calculations for the materials science community
https://mineralscloud.github.io/Express.jl/
GNU General Public License v3.0
25 stars 1 forks source link

Change `outdir` to `outdir * mktempdir()` #45

Closed singularitti closed 4 years ago

singularitti commented 4 years ago

The current implementation is just mktempdir(), which requires users to set the TMPDIR environement variables, etc. (See http://docs.julialang.org/en/v1/base/file/#Base.Filesystem.tempdir) to not occupy system /tmp dir, which is often not owned by users but the system administrators on a supercomputer.

I propose take the control.outdir as prefix, then create a mktempdir() by mktempdir(). Should be easier to set.

singularitti commented 4 years ago

Fixed. @ChaoxuanG9 Update if you are interested.