-
Notifications
You must be signed in to change notification settings - Fork 7
Use JULIA_CPU_TARGET #2646
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
Use JULIA_CPU_TARGET #2646
Conversation
Rather than the kwarg in JuliaC, let's try if using the environment variable JULIA_CPU_TARGET works better. Related to #2644
|
I'm running a few personal builds on Teamcity using the main branch with different JULIA_CPU_TARGETs set as an env variable. Let's see what happens. |
|
Hmm on the latest commit the Windows Ribasim-API tests still fail with Also curious that this takes much longer to compile on Windows, 100 minutes for the compile step. |
|
On Teamcity you should
Some patterns:
On Teamcity Linux, this PR builds and all tests are green(!). On main, with setting On Teamcity Windows, this PR fails horribly, Rust can't find the procedure, and neither can Python in the BMI tests (note that my last commit never touched the Python code, so that could be fair). On main, my personal |
|
Thanks, yes it seems a bit all over the place with possibly multiple bugs in JuliaC / PackageCompiler / julia. We can wait a little, but may need to start thinking about reverting stuff. Ideally only JuliaC -> PackageCompiler would be enough, but perhaps we need to go back to Julia 1.11 as well. |
If we wait, we should use the old release/executable for the DSD. So no new features since then. |
|
I want to do a new release before the pizza course. I was thinking to wait max a week before reverting. EDIT: reverted in 84c5182 |
Rather than the kwarg in JuliaC, let's try if using the environment variable JULIA_CPU_TARGET works better.
Related to #2644
The native binaries are for
skylake-avx512. Most people including me don't haveavx512, hence I get: