Change installation of fenics-basix using pip with compiled C++ version directly from repo

Author: hmella

CMakeLists.txt

@@ -27,6 +27,8 @@ if(UNIX AND NOT APPLE)
                     ninja-build
                     git
                     libopenmpi-dev
+                    libblas-dev
+                    liblapack-dev
                     screen
                     nano
                 RESULT_VARIABLE APT_RESULT
@@ -96,28 +98,26 @@ endif()
 find_package(Python REQUIRED COMPONENTS Interpreter Development.Module)
 find_package(pybind11 CONFIG REQUIRED)
 
-# --- Basix: use the basix that is installed in this Python env (fenics-basix wheel) ---
-execute_process(
-  COMMAND "${Python_EXECUTABLE}" -c
-    "import basix, os; print(os.path.join(os.path.dirname(basix.__file__), 'lib'))"
-  OUTPUT_VARIABLE BASIX_PY_LIBDIR
-  OUTPUT_STRIP_TRAILING_WHITESPACE
-  RESULT_VARIABLE _basix_rc
-)
-if(NOT _basix_rc EQUAL 0)
-  message(FATAL_ERROR
-    "Basix not importable in this Python env. Install: python -m pip install fenics-basix")
-endif()
+# ------------------------------------------------------------
+# FEniCS Basix (C++) automatic compilation via FetchContent
+# ------------------------------------------------------------
+find_package(BLAS REQUIRED)
+find_package(LAPACK REQUIRED)
 
-find_library(BASIX_LIB
-  NAMES basix
-  PATHS "${BASIX_PY_LIBDIR}"
-  NO_DEFAULT_PATH
-  REQUIRED
+include(FetchContent)
+FetchContent_Declare(
+    basix
+    GIT_REPOSITORY https://github.com/FEniCS/basix.git
+    GIT_TAG v0.10.0    # Matches the >=0.10.0 requirement in your pyproject.toml
+    SOURCE_SUBDIR cpp  # Tells CMake to look for CMakeLists.txt inside the cpp/ folder
 )
 
-message(STATUS "Basix wheel libdir: ${BASIX_PY_LIBDIR}")
-message(STATUS "Basix wheel lib:    ${BASIX_LIB}")
+FetchContent_GetProperties(basix)
+if(NOT basix_POPULATED)
+    message(STATUS "Fetching and building FEniCS Basix C++ library from GitHub...")
+    FetchContent_Populate(basix)
+    add_subdirectory(${basix_SOURCE_DIR}/cpp ${basix_BINARY_DIR})
+endif()
 
 # ------------------------------------------------------------
 # Eigen3 automatic installation (cached and robust)
@@ -168,7 +168,7 @@ foreach(mod ${MODULES})
             ${Python_INCLUDE_DIRS}
     )
 
-		target_link_libraries(${mod} PRIVATE "${BASIX_LIB}")
+    target_link_libraries(${mod} PRIVATE basix)
 
 		# Apply high-performance compile flags
     target_compile_options(${mod} PRIVATE
@@ -207,4 +207,4 @@ message(STATUS "  Pybind11 includes:   ${pybind11_INCLUDE_DIRS}")
 message(STATUS "  Eigen include dir:   ${EIGEN_INCLUDE_DIR}")
 message(STATUS "  Eigen unsupported:   ${EIGEN_UNSUPPORTED_DIR}")
 message(STATUS "  LTO enabled:         ${CMAKE_INTERPROCEDURAL_OPTIMIZATION}")
-message(STATUS "------------------------------------------------------------")
+message(STATUS "------------------------------------------------------------")

pyproject.toml

@@ -1,6 +1,5 @@
 [build-system]
 requires = ["scikit-build-core>=0.10",
-            "fenics-basix>=0.10.0",
             "pybind11",
             "numpy",
             "setuptools",
@@ -26,7 +25,6 @@ dependencies = [
   "scikit-image",
   "pint",
   "pymetis",
-  "fenics-basix>=0.10.0",
   "pynufft",
 ]
 keywords = ["MRI", "finite-element", "simulation", "pybind11", "C++"]

python/feelmri/__init__.py

@@ -3,11 +3,4 @@
 try:
     __version__ = version("feelmri")
 except PackageNotFoundError:
-    __version__ = "0.0.0"
-
-# Ensure basix (and its bundled libbasix) is loaded before feelmri extensions
-try:
-    import basix  # noqa: F401
-except Exception:
-    # If basix isn't installed, leave the error to the caller when they import features needing it
-    pass
+    __version__ = "0.0.0"