acreate

[moravveji]

Hi.

Apparently, with atools/1.4.5 the acreate command is built against a python version which does not recognize the "sys" package (or not imported). The main issue is reported in service desk ticket 31609268.

$ acreate single_RG.pbs > single_RG_puls.pbs Traceback (most recent call last):  File "/vsc-hard-mounts/leuven-apps/skylake/2018a/software/atools/1.4.5/bin/../lib/acreate.py", line 73, in <module>    sys.stderr.write(msg) NameError: name 'sys' is not defined

Thanks
Ehsan

[gjbex]

Hi @moravveji ,

Fixed in release 1.4.6, it was a missing import, every Python release I've worked with has a sys module ;)

I've also done a quick and dirty fix on genius into the existing 1.4.5 installation (yes, I'm lazy, but then again, I'm still on holidays ;)

Thanks, best regards, -gjb-

[moravveji]

Thanks for the quick fix, specially in between your holiday period.
So far, I naively thought that packages like sys, os, and friends ship with the basic Python installation (on the OS or as external builds); this example proved me wrong!
Here, atools is relying on the OS python, isn't it?

E.

[gjbex]

Hi Ehsan, Modules like sys, os & friends are all part of the Python standard library that comes with any Python install. However, in acreate.py, I forgot to import the sys module, hence the error. It went unnoticed for a long while since it is only used for sys.stderr. The Python installation used is determined by: conf/atools_python.sh For genius, it is indeed /usr/bin/python Best regards, -gjb-

…

On Thu, Jan 3, 2019 at 1:46 PM Ehsan Moravveji ***@***.***> wrote: Thanks for the quick fix, specially in between your holiday period. So far, I naively thought that packages like sys, os, and friends ship with the basic Python installation (on the OS or as external builds); this example proved me wrong! Here, atools is relying on the OS python, isn't it? E. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#23 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AElDOOYq9D7vqzBYxozugP0vSDL6LT3qks5u_fumgaJpZM4Zn6U8> .

-- dr. Geert Jan Bex HPC consultant/analist DOC T +32(0)11 26 82 31 www.uhasselt.be Universiteit Hasselt - Campus Diepenbeek Agoralaan Gebouw D - B-3590 Diepenbeek Kantoor D250a

[wpoely86]

I can highly recommend pyflakes/pylint/... integration in your editor. It points these kind of errors.

[gjbex]

Hi Ward, True, I have that on my laptop's VM, but not on the cluster since vim extensions can be a nuisance. If I get around to it, I should add CI to the project. Thanks, best regards, -gjb-

…

On Thu, Jan 3, 2019 at 2:45 PM wpoely86 ***@***.***> wrote: I can highly recommend pyflakes/pylint/... integration in your editor. It points these kind of errors. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#23 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AElDOHhUf9Bz4tO1zA_KDfJvxU0hYzEyks5u_glpgaJpZM4Zn6U8> .

-- dr. Geert Jan Bex HPC consultant/analist DOC T +32(0)11 26 82 31 www.uhasselt.be Universiteit Hasselt - Campus Diepenbeek Agoralaan Gebouw D - B-3590 Diepenbeek Kantoor D250a