Skip to content

[unopy] Test the TestPyPI artifacts#549

Open
cvanaret wants to merge 33 commits intomainfrom
unopy_testpypi
Open

[unopy] Test the TestPyPI artifacts#549
cvanaret wants to merge 33 commits intomainfrom
unopy_testpypi

Conversation

@cvanaret
Copy link
Owner

@cvanaret cvanaret commented Mar 4, 2026

No description provided.

@amontoison
Copy link
Collaborator

The issue for Ubuntu is related to cxx11 string ABI:
https://docs.binarybuilder.org/stable/tricksy_gotchas/

We will also get the same error on windows (if we found a way to use unopy with CI on this platform).
I need to recompile UnoUtils.

@cvanaret
Copy link
Owner Author

cvanaret commented Mar 4, 2026

@amontoison how come the current artifacts from PyPI work fine, but the new TestPyPI artifacts fail? Did we change anything that would cause this cxx11 string ABI issue?

@amontoison
Copy link
Collaborator

amontoison commented Mar 4, 2026

I think it was always there.
But I checked the symbols in the precompiled library of UnoUtils and nothing is wrong.
The issue is with cibuildwheel.
The flag -D_GLIBCXX_USE_CXX11_ABI=0 is forced somewhere while we want -D_GLIBCXX_USE_CXX11_ABI=1.

@cvanaret
Copy link
Owner Author

cvanaret commented Mar 4, 2026

I see. Note: the TestPyPI artifacts were built with the cibuildwheel action, I'm now rebuilding in #480 with a direct call cibuildwheel --output-dir ${{github.workspace}}/wheelhouse, like we have in the main branch. Could be the source of the problem.

@amontoison
Copy link
Collaborator

The good news is that now I understand why we can't do a static linking of HiGHS on Windows, we have the same issue on Linux but the linker is more permissive and we only have the error at runtime.

@amontoison
Copy link
Collaborator

I also suggest to test all platforms (aarch64 linux and x64 mac are missing).

@amontoison
Copy link
Collaborator

@cvanaret If it can help:
ralna/GALAHAD@95441a4#diff-6b39a2f1a6399aa39e427fb4c11a7b552e615a2d83a3c36aa6dd6ddbbb8281f4

I remember that Jari spend a lot of time on this issue.

@cvanaret
Copy link
Owner Author

cvanaret commented Mar 5, 2026

@amontoison cool, that's exactly what Claude asked me to add to __init__py (33502d4) 😃

@cvanaret
Copy link
Owner Author

cvanaret commented Mar 6, 2026

@amontoison good news! If we exclude libhighs.dll from the wheels, unopy runs on Windows :)
https://github.com/cvanaret/Uno/actions/runs/22777727310/job/66075263282?pr=549

@amontoison
Copy link
Collaborator

We can maybe do a release of unopy for Windows without it and add it later?
We still need to understand why HiGHS 1.13 is failing with Uno

@cvanaret
Copy link
Owner Author

cvanaret commented Mar 6, 2026

We can maybe do a release of unopy for Windows without it and add it later? We still need to understand why HiGHS 1.13 is failing with Uno

Agreed. Then we can finally merge #480. I'll tag you when it's ready.

@amontoison
Copy link
Collaborator

@cvanaret Should we close this PR?

@cvanaret
Copy link
Owner Author

cvanaret commented Mar 7, 2026

@amontoison there is useful code to detect DLL errors in wheels. I would keep it open in case we need it when we add HiGHS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants