Skip to content

Make numba an optional dependency #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
JayFoxRox opened this issue Nov 22, 2019 · 3 comments
Open

Make numba an optional dependency #4

JayFoxRox opened this issue Nov 22, 2019 · 3 comments

Comments

@JayFoxRox
Copy link

I tried to install this, but I get the following, while installing numba (during installation of its llvmlite dependency):

    RuntimeError: Building llvmlite requires LLVM 7.0+ Be sure to set LLVM_CONFIG to the right executable path.

This is because my system only has LLVM 9 which isn't supported by llvmlite (used in numba) yet.
Additionally, I use Python 3.8, which also isn't supported by numba yet.

I checked the source-code for this project, and it only seems to use numba to improve performance. This project itself doesn't seem to have anything that prevents Python 3.8 or would even require the LLVM dependency (other numba).

I assume that even reduced performance would be fine for me. However, due to the numba dependency, I can't run this project at all. If numba was an optional dependency, this wouldn't be an issue.

@yxdragon
Copy link
Member

yxdragon commented Dec 8, 2019

the numba.jit is a decorator function, you can give a python format.

# from numba import jit
def jit(f): return f
...

that is will be ok!

and I did not update this repo for a long time, since numba has update much, I would try to review sknw soon!

@yxdragon
Copy link
Member

@JayFoxRox Hi, now sknw can work with the new version numba, have a try!

@JayFoxRox
Copy link
Author

I don't need to use sknw anymore, so I can't / won't test this.

I had already locally applied the same fix you had suggested to work around the issue (actually I came up with it myself independently; I had the same code in a custom numba/__init__.py so it would automatically work).

I still think that numba should be an optional dependency; it just adds a lot of dependencies and bloat for little benefit (for some users).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants