Skip to content
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

Fix Deployment for macOS x86_64 wheels on PyPi #40

Merged
merged 7 commits into from
May 17, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 9 additions & 5 deletions .github/workflows/wheels.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@ jobs:
- uses: actions/checkout@v3
############################# LINUX WHEELS #############################
# In case of Linux we need to install compiler and build tools before building the wheels
# We further only build the manylinux wheels, but not the musllinux wheels
# We only build the manylinux wheels, but not the musllinux wheels (due to some compile problems)
# We set-up QEMU to enable aarch64 builds in the GitHub Runner (which is x86_64 based)
- name: Set up QEMU
uses: docker/setup-qemu-action@v3
with:
Expand All @@ -41,8 +42,10 @@ jobs:
output-dir: dist
if: matrix.os == 'ubuntu-latest'
############################# MACOS WHEELS #############################
# Building on macOS requires an installation of gcc since the default clang compiler
# lacks certain features required for building the package
# We use Apple Clang, the macOS GitHub Runner is nowadays arm64 based
# The GCC compiler installable via brew does not support cross-compiling for x86_64. Hence, also Apple Claang
# For the x86_64, we set the MACOSX_DEPLOYMENT_TARGET='10.13' (released 2017) in order to have support for C++17
# We don't need this for the arm64 stuff since it works (and macOS on arm64 cam after C++17)
- name: Build wheels (macOS ARM)
uses: pypa/[email protected]
env:
Expand All @@ -56,8 +59,8 @@ jobs:
- name: Build wheels (macOS x86_64)
uses: pypa/[email protected]
env:
CIBW_BEFORE_BUILD: brew install ninja gcc@12
CIBW_ENVIRONMENT: "CC=gcc-12 CXX=g++-12"
CIBW_BEFORE_BUILD: brew install ninja
CIBW_ENVIRONMENT: 'MACOSX_DEPLOYMENT_TARGET="10.13"'
CIBW_ARCHS_MACOS: "x86_64"
CIBW_TEST_COMMAND: 'python -c "import polyhedral_gravity"'
with:
Expand All @@ -66,6 +69,7 @@ jobs:
if: matrix.os == 'macos-latest'
############################# WINDOWS WHEELS #############################
# Set up the Visual Studio environment on Windows (required, so that CMake finds the compiler)
# We use the Microsoft Visual Studio Compiler to compile the wheel
- uses: ilammy/msvc-dev-cmd@v1
if: matrix.os == 'windows-latest'
- name: Build wheels (Windows)
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -258,12 +258,12 @@ The project uses the following dependencies,
all of them are **automatically** set-up via CMake:

- GoogleTest (1.13.0 or compatible), only required for testing
- spdlog (1.11.0 or compatible), required for logging
- spdlog (1.13.0 or compatible), required for logging
- tetgen (1.6 or compatible), required for I/O
- yaml-cpp (0.7.0 or compatible), required for I/O
- yaml-cpp (0.8.0 or compatible), required for I/O
- thrust (2.1.0 or compatible), required for parallelization and utility
- xsimd (11.1.0 or compatible), required for vectorization of the `atan(..)`
- pybind11 (2.10.4 or compatible), required for the Python interface, but not the C++ standalone
- pybind11 (2.12.0 or compatible), required for the Python interface, but not the C++ standalone

The module will be build using a C++17 capable compiler,
CMake. Just execute the following command in
Expand Down
7 changes: 3 additions & 4 deletions cmake/pybind11.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ include(FetchContent)

message(STATUS "Setting up pybind11")

find_package(pybind11 2.10.4 QUIET)
find_package(pybind11 2.12.0 QUIET)

if (${pybind11_FOUND})

Expand All @@ -12,11 +12,10 @@ else()

message(STATUS "Using pybind11 from git repository")

#Fetches the version 2.10.4 from the official github of pybind11
FetchContent_Declare(pybind11
GIT_REPOSITORY https://github.com/pybind/pybind11
GIT_TAG v2.10.4
)
GIT_TAG v2.12.0
)

FetchContent_MakeAvailable(pybind11)

Expand Down
7 changes: 3 additions & 4 deletions cmake/spdlog.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ include(FetchContent)

message(STATUS "Setting up spdlog")

find_package(spdlog 1.11.0 QUIET)
find_package(spdlog 1.13.0 QUIET)

if (${spdlog_FOUND})

Expand All @@ -12,11 +12,10 @@ else()

message(STATUS "Using spdlog from git repository")

#Fetches the version 1.11.0 for spdlog
FetchContent_Declare(spdlog
GIT_REPOSITORY https://github.com/gabime/spdlog.git
GIT_TAG v1.11.0
)
GIT_TAG v1.13.0
)

# Disable stuff we don't need
option(SPDLOG_BUILD_EXAMPLE "" OFF)
Expand Down
6 changes: 3 additions & 3 deletions cmake/tbb.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ include(FetchContent)

message(STATUS "Setting up tbb via CMake")

#Fetches the version v2021.5.0 from the official github of tbb
#Fetches the version v2021.12.0 from the official github of tbb
FetchContent_Declare(tbb
GIT_REPOSITORY https://github.com/oneapi-src/oneTBB.git
GIT_TAG v2021.5.0
)
GIT_TAG v2021.12.0
)

# Disable tests & and do not treat tbb-compile errors as warnings
option(TBB_TEST "Enable testing" OFF)
Expand Down
6 changes: 3 additions & 3 deletions cmake/yaml.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,18 @@ include(FetchContent)

message(STATUS "Setting up yaml-cpp")

find_package(yaml-cpp 0.7.0 QUIET)
find_package(yaml-cpp 0.8.0 QUIET)

if (${yaml-cpp_FOUND})

message(STATUS "Using existing yaml-cpp installation")

else()

#Fetches the version 0.7.0 for yaml-cpp
#Fetches the version 0.8.0 for yaml-cpp
FetchContent_Declare(yaml-cpp
GIT_REPOSITORY https://github.com/jbeder/yaml-cpp.git
GIT_TAG yaml-cpp-0.7.0
GIT_TAG 0.8.0
)

# Disable everything we don't need
Expand Down
8 changes: 4 additions & 4 deletions docs/background/mesh_integrity.rst
Original file line number Diff line number Diff line change
Expand Up @@ -54,13 +54,13 @@ The :code:`Polyhedron` class checks the pointing direction of every single
normal. This way, the `Polyhedron` ensures correct results even if a mistake occurs during the definition.

Since we are in 3D and might have concave and convex polyhedrons,
the viable option is the `Möller–Trumbore intersection algorithm <https://en.wikipedia.org/wiki/Möller–Trumbore_intersection_algorithm>`__.
It checks the amount of intersections the plane unit normal has with the polyhedron.
If its an even number, the normals is :code:`OUTWARDS` pointing, otherwise :code:`INWARDS`.
the most viable option is the `Möller–Trumbore intersection algorithm <https://en.wikipedia.org/wiki/Möller–Trumbore_intersection_algorithm>`__.
It checks the amount of intersections each plane unit normal has with the polyhedron.
If this is an even number, the normal is :code:`OUTWARDS` pointing, otherwise :code:`INWARDS`.
In the *current implementation*, we implement a naiv version
which takes :math:`O(n^2)` operations - which can get quite expensive for polyhedrons with many faces.

To make this as straightforward to use as possible, we provide four options
To make this as straightforward to use, we provide four options
for the construction of a polyhedron in the form of the enum :code:`PolyhedronIntegrity`:

+-----------------------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+----------------------------+----------------+
Expand Down
6 changes: 3 additions & 3 deletions docs/quickstart/installation.rst
Original file line number Diff line number Diff line change
Expand Up @@ -105,10 +105,10 @@ Dependencies (automatically set-up)
Dependencies (all of them are automatically set-up via :code:`CMake`):

- GoogleTest (1.13.0 or compatible), only required for testing
- spdlog (1.11.0 or compatible), required for logging
- spdlog (1.13.0 or compatible), required for logging
- tetgen (1.6 or compatible), required for I/O
- yaml-cpp (0.7.0 or compatible), required for I/O
- yaml-cpp (0.8.0 or compatible), required for I/O
- thrust (2.1.0 or compatible), required for parallelization and utility
- xsimd (11.1.0 or compatible), required for vectorization of the :code:`atan(..)`
- pybind11 (2.10.4 or compatible), required for the Python interface, but not the C++ standalone
- pybind11 (2.12.0 or compatible), required for the Python interface, but not the C++ standalone

2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ def build_extension(self, ext):
# --------------------------------------------------------------------------------
setup(
name="polyhedral_gravity",
version="3.1",
version="3.2",
author="Jonas Schuhmacher",
author_email="[email protected]",
description="Package to compute full gravity tensor of a given constant density polyhedron for arbitrary points "
Expand Down
Loading