search

XENONnT / utilix

Package for XENON members to easily interface with RunDB, Midway batch queue, maybe Rucio (others?).
0 stars 6 forks source link

Incorrect use of `os.path.expanduser` in batchq in TMPDIR["dali"] #90

Closed Microwave-WYB closed 7 months ago

Microwave-WYB commented 7 months ago

os.path.expanduser should be used with ~. It expands the ~ symbol to the current user's home directory. However, in the current code:

    "dali": os.path.expanduser(f"/dali/lgrandi/{USER}/tmp"),

The path is an absolute path, so os.path.expanduser will have no effect. This will not lead to errors but should be replaced with an f-string.

yuema137 commented 7 months ago

Already fixed.