-
-
Notifications
You must be signed in to change notification settings - Fork 13k
openai-whisper: use llvm@20
(pre-release testing)
#235317
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
@cho-m looks like the test failure was some sort of glitch that caused a timeout? Context: Numba release manager here, I am checking on dependents and the stability of the release candidate. |
Yeah, you can ignore the failure as it is unrelated to numba/llvmlite. Likely a slow CI runner. |
b9408df
to
fbeb122
Compare
|
||
# TODO: remove | ||
system libexec/"bin/python", "-m", "llvmlite.tests" | ||
system libexec/"bin/python", "-m", "numba.runtests", "-m" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To remove on a future run. Just checking our shared LLVM 20 works with llvmlite via test suite:
# TODO: remove | |
system libexec/"bin/python", "-m", "llvmlite.tests" | |
system libexec/"bin/python", "-m", "numba.runtests", "-m" |
FWIW: using a shared LLVM is somewhat discouraged and this setup has been removed in conda-forge: conda-forge/llvmlite-feedstock#100 -- in case you end up with user issues and weird segfaults, this may be something to consider. |
Testing pre-releases of
llvmlite
andnumba
CMake addition is for
llvmlite
. Same for environment variable changes:RC tracking issues:
Tested:
system libexec/"bin/python", "-m", "llvmlite.tests"
system libexec/"bin/python", "-m", "numba.runtests", "-m"