-
Notifications
You must be signed in to change notification settings - Fork 150
Numba full backend support and required dependency #811
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
base: main
Are you sure you want to change the base?
Conversation
2eb7fe7 to
1f093cd
Compare
|
Locally I ran all the tests in Numba caching is baaaaaaaaaaaaaaaaaaaaaaaaaaaaaaad |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
5c4491d to
a1e3775
Compare
This comment was marked as outdated.
This comment was marked as outdated.
6f6e8bb to
53d9a26
Compare
53d9a26 to
9076c99
Compare
9076c99 to
0af7cb7
Compare
|
After caching the test file now runs in 50s after caching vs 6s before the PR, so only 8x slower now :( |
0af7cb7 to
17ef6a4
Compare
bd3da41 to
5f17484
Compare
463cba4 to
3367ea8
Compare
|
The build test is failing with llvm/numba, seems to be trying to build wheel for free threaded python? Do we have a way to skip that, if that's even the problem. llvmlite/numba don't yet support it (and probably neither do I even if we built for it) |
|
@ricardoV94, just pushed a commit that hopefully solves it |
e686b48 to
0b8ed71
Compare
Numba rewrites these to integers for specialized implementation
Mark overly specific tests as xfail
0b8ed71 to
bfd59a0
Compare
|
@maresb / @lucianopaz I requested your review basically for the last 2 commits, where we make numba default dependency and run it on the CI The prior commits is just tweaking tests so they pass with both backends. I've gone over them with @jessegrabowski and asked him to review those. Feel free to look if you want but nothing terribly interesting or critical there |
|
@maresb I force-pushed above your commit, sorry didn't see you had come to the rescue already. should we go with |
bfd59a0 to
4789ae2
Compare
4789ae2 to
78cde62
Compare

What is broken / not supported by Numba:
The plan now
Immediate action
I'll be cherry picking the countless fixes into their own PRs. I need help with review as both me and @jessegrabowski are pretty much at capacityDone