Skip to content

[CI] Compile Uno with SPRAL#588

Open
amontoison wants to merge 11 commits into
mainfrom
am/upgrade_uno_utils
Open

[CI] Compile Uno with SPRAL#588
amontoison wants to merge 11 commits into
mainfrom
am/upgrade_uno_utils

Conversation

@amontoison
Copy link
Copy Markdown
Collaborator

@amontoison amontoison commented Mar 13, 2026

It will help to add SPRAL / SSIDS in unopy.

Update:

  • We need to recompile Hwloc with -fPIC on Linux if we want to use a static library...
  • We need to add an option -DSPRAL_INCLUDE_DIR, it was working without it thanks to -DMUMPS_INCLUDE_DIR and -DMETIS_INCLUDE_DIR. We use the same folder for all headers with our UnoUtils.
  • We have the following warning when we compile the interface of MUMPS:
 [06:07:19] /workspace/srcdir/Uno/uno/ingredients/subproblem_solvers/MUMPS/MUMPSSolver.cpp:134:30: warning: implicit conversion changes signedness: 'const int' to 'size_t' (aka 'unsigned long') [-Wsign-conversion]
[06:07:19]   134 |       return this->workspace.n - this->number_zero_eigenvalues();
[06:07:19]       |              ~~~~~~~~~~~~~~~~^ ~
[06:07:20] 1 warning generated.
``

@amontoison amontoison self-assigned this Mar 13, 2026
@amontoison amontoison added CI Continuous Integration build labels Mar 13, 2026
@amontoison amontoison requested a review from cvanaret March 13, 2026 06:23
@cvanaret
Copy link
Copy Markdown
Owner

@amontoison was SPRAL compiled with OpenMP in UnoUtils?

@amontoison
Copy link
Copy Markdown
Collaborator Author

amontoison commented Mar 13, 2026

Yes, LLVM OpenMP on Mac et GNUOpenMP and Windows / Linux.
You need -lomp or -lgomp.
I think you automatically add OpenMP in the dependency, but it is maybe only for MUMPS.

@cvanaret cvanaret force-pushed the am/upgrade_uno_utils branch from 2c68b00 to 8b584f4 Compare March 13, 2026 17:42
@amontoison amontoison force-pushed the main branch 5 times, most recently from 5a5ca03 to 21482ad Compare April 15, 2026 06:33
@amontoison amontoison force-pushed the am/upgrade_uno_utils branch 2 times, most recently from 4c712ce to 757c652 Compare April 15, 2026 21:42
@cvanaret cvanaret force-pushed the am/upgrade_uno_utils branch from 358a0fe to 22aa59b Compare April 16, 2026 09:17
@cvanaret
Copy link
Copy Markdown
Owner

cvanaret commented Apr 16, 2026

We're making progress!

TODO:

  • some builds: undefined reference to METIS and OMP
  • the unopy example fails with unopy.cpython-312-x86_64-linux-gnu.so: undefined symbol: hwloc_topology_set_type_filter and so on
  • the wheel builds fail with undefined symbol: hwloc_topology_set_type_filter and so on
  • the macOS wheel builds fail because libhwloc.a cannot be found
  • the repair for macOS wheel builds fails (cannot find libhwloc.15.a)

@cvanaret cvanaret force-pushed the am/upgrade_uno_utils branch from 3800c95 to 9a9485b Compare April 22, 2026 11:45
@cvanaret cvanaret force-pushed the am/upgrade_uno_utils branch from 9a9485b to a179591 Compare May 1, 2026 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build CI Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants