-
Notifications
You must be signed in to change notification settings - Fork 74
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
AttributeError: module 'numba' has no attribute 'generated_jit' #430
Comments
I have encountered this as well
Discussion of the removal of As a workaround until this change, |
Doesn't work with latest numba pygae/clifford#430 numba/numba#9353
Just wanted to note that I ran into this issue as well. The workaround |
Maybe a dup of #380 which was filed back in 2020? However, no workaround details were posted there. Workaround details were posted above by mhartl ( So, we now have to pin to an old version of python, too :-( Here is the full recipe, including workarounds, that works for me right now on ubuntu 24.04, to run from my clone of the clifford repo:
|
Pending a fix, it would be great if a link to this bug could be added to https://clifford.readthedocs.io/en/latest/installation.html , to help people find the workaround more quickly. |
Apologies everyone, we have all been pretty tied up recently and havent been taking as much care of our packages as we should be! Thanks so much for the work around, I have a couple of days I can dedicate to this coming up, and definitely appreciative of any suggestions. I think we went a bit overboard in integrating numba very tightly into the codebase chasing execution speed, I'll have a look if we can decouple it a bit |
Done by avoid the numpy.result_type (not supported by numba). Instead do the array multiplication at-once, then use the result dtype. This requires some numpy array-indexing tricks.
I'm a newbie in numba and clifford, so keep in mind that the above commit could be just a stupid idea. By my understanding, the only reason for using the I'm using some simple numpy-array indexing tricks to do the multiplication of all components at-once. In BTW, these changes are made on top of some patches that makes CI github action to pass all tests (both existing code and the fix). However, the fixing of CI is another topic. Hope this would help! |
The test_function_cache is rewritten to use @njit, which may not make much sense for particular test.
Done by avoid the numpy.result_type (not supported by numba). Instead do the array multiplication at-once, then use the result dtype. This requires some numpy array-indexing tricks.
The test_function_cache is rewritten to use @njit, which may not make much sense for particular test.
I tried:
When I try to import clifford I get:
The text was updated successfully, but these errors were encountered: