-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
[NO MRG] Run tests requiring sparse
#121
base: main
Are you sure you want to change the base?
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Good that you ended up retesting this! Seems the failures that I kept seeing over the last 1-2 years when I tried this on the arrow-cpp feedstock aren't there anymore. I also checked that the skipped section does not contain what's visible on main, namely:
|
Am seeing a different error in the macOS
This could just be a fluke given we are not seeing it other jobs. So probably worth restarting first Though if this keeps happening, maybe we can try Edit: Restarted and it passed |
So the situation is reproducible with arrow v14 (segfaults if enabling |
Thanks for investigating, Axel! 🙏 Was wondering if we would learn where the fix occurred One thing that sticks out when looking at the diff between the branches Is it possible to use Clang 16 with Arrow 14 builds? If so, do we still see the error in that case? |
Yeah, I'm testing that hypothesis (arrow v14 + clang 16; arrow v15 + clang 15). It could be that it's an interaction between llvm 14 & 15 only, i.e. it doesn't seem to affect later llvm versions for now. That said, that this happens at all is an argument in favour of caution (and thus building against static libllvm), because we don't understand yet understand under what conditions the failure mode(s) can occur. |
Confirmed that llvm 15 is the one causing issues (note current
|
Try to reproduce Numba + Arrow + LLVM + sparse issue: conda-forge/llvmlite-feedstock#72 (comment)
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)