-
-
Notifications
You must be signed in to change notification settings - Fork 482
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
build/pkgs/sage_numerical_backends_coin
, sage_numerical_backends_cplex
, sage_numerical_backends_gurobi
: Update to 10.4 (includes fix for Cython 3)
#38319
Conversation
Documentation preview for this PR (built with commit 9ff0a10; changes) is ready! 🎉 |
27a0ff7
to
28a9bd0
Compare
@mkoeppe: I don't understand what changed here. Does this add the missing noexcept qualifiers? |
Yes, the new release includes this change: sagemath/sage-numerical-backends-coin@bf1e1d3 |
build/pkgs/sage_numerical_backends_coin
: Update to 10.4 (includes fix for Cython 3)build/pkgs/sage_numerical_backends_coin
, sage_numerical_backends_cplex
: Update to 10.4 (includes fix for Cython 3)
build/pkgs/sage_numerical_backends_coin
, sage_numerical_backends_cplex
: Update to 10.4 (includes fix for Cython 3)build/pkgs/sage_numerical_backends_coin
, sage_numerical_backends_cplex
, sage_numerical_backends_gurobi
: Update to 10.4 (includes fix for Cython 3)
I can successfully compile this PR on macOS 14.5, but I fail to install
I have CPLEX Studio 2211 and variable |
Thanks for testing! Does |
|
Thanks. Please try with this version |
|
Not sure how much of this is just from using build isolation (there should have been a fallback to no isolation). Let's try with explicitly using |
same result
|
Thanks for your patience with this. Here's another version, in which I added |
Same error. Should I try a distclean ? so far I only recompile with the added PR before triying to install cplex. |
|
unfortunately, it fails with the same error :( |
Could you share your |
Here it is config.log |
Unfortunately I cannot reproduce this problem. I just installed CPLEX_Studio2211 and installation went through without problem for me |
@maxale Would you be interested in testing this? |
I've tested this on CPLEX and Gurobi backends, and they are installed fine (provided that the corresponding environment variables are set). |
Thanks for testing. Let's get this in please. |
This works perfectly for me as well. I started from scratch, downloading free versions of cplex and gurobi, and could test examples using coin, cplex, gurobi solvers, respectively. |
I tried again with sage 10.4.rc3, M1 Max, macOS 14.5, and CPLEX_Studio2211. |
These packages have been broken since our switch to Cython 3.
Switching to new releases:
sage_numerical_backends_gurobi
package stalls #38268Changes include
noexcept
fixes for Cython 3.📝 Checklist
⌛ Dependencies