Hi. Apparently, with atools/1.4.5 the acre

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

acreate about atools HOT 5 CLOSED

gjbex commented on May 29, 2024

acreate

from atools.

Comments (5)

gjbex commented on May 29, 2024

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-

from atools.

moravveji commented on May 29, 2024

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?

from atools.

gjbex commented on May 29, 2024

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

from atools.

wpoely86 commented on May 29, 2024

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

from atools.

gjbex commented on May 29, 2024

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

from atools.

acreate about atools HOT 5 CLOSED

Comments (5)

Related Issues (6)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent