Bring python binding back to work

.github/workflows/continuous.yml

@@ -67,4 +67,4 @@ jobs:
       - name: Tests
         if: matrix.name == 'Linux' || matrix.name == 'MacOS' || (matrix.config == 'Debug' && runner.os == 'Windows')
         run: |
-          python test/test_bending.py
+          python test/test_basic.py

CMakeLists.txt

@@ -1,6 +1,6 @@
 ################################################################################
 # Check required CMake version
-set(REQUIRED_CMAKE_VERSION "3.14.0")
+set(REQUIRED_CMAKE_VERSION "3.25.0")
 cmake_minimum_required(VERSION ${REQUIRED_CMAKE_VERSION})
 
 if(INPUT_POLYFEMPY_DATA_ROOT)
@@ -13,11 +13,18 @@ if(NOT EXISTS ${POLYFEMPY_DATA_ROOT})
 endif()
 
 project(polyfempy)
+
+set(CMAKE_CXX_STANDARD 17)
+set(CMAKE_CXX_STANDARD_REQUIRED ON)
+set(CMAKE_CXX_EXTENSIONS OFF)
+
 ################################################################################
 
 list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake")
 list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake/recipes/")
 
+include(polyfem_cpm_cache)
+
 # Color output
 include(UseColors)
 
@@ -51,11 +58,9 @@ include(polyfem_data)
 ################################################################################
 # Subdirectories
 ################################################################################
-add_library(polyfempy MODULE src/binding.cpp src/raster.cpp)
+pybind11_add_module(polyfempy)
+add_subdirectory(src)
+target_include_directories(polyfempy PRIVATE "${CMAKE_CURRENT_SOURCE_DIR}/src")
 target_link_libraries(polyfempy PRIVATE polyfem::polyfem pybind11::module pybind11::json)
 
 set_target_properties(polyfempy PROPERTIES PREFIX "${PYTHON_MODULE_PREFIX}" SUFFIX "${PYTHON_MODULE_EXTENSION}")
-
-
-
-

README.md

@@ -7,30 +7,21 @@
 [![Anaconda-Server Badge](https://anaconda.org/conda-forge/polyfempy/badges/platforms.svg)](https://anaconda.org/conda-forge/polyfempy)
 [![Anaconda-Server Badge](https://anaconda.org/conda-forge/polyfempy/badges/installer/conda.svg)](https://conda.anaconda.org/conda-forge)
 
-
-
-
-The python bindings are in alpha. Expect a lot of API changes and possible bugs. Use at your own peril!
+The Python bindings are in alpha. Expect a lot of API changes and possible bugs. Use at your own peril!
 
 <br/>
-I am making efforts to provide a simple python interface to Polyfem.
-
-For doing so I am maintaining  a *conda* package which can be easily installed [https://anaconda.org/conda-forge/polyfempy](https://anaconda.org/conda-forge/polyfempy).
-
-Note that the conda deployment is slow and this tutorial will follow the deployment version.
+To use the Python bindings, clone the current repository and use anaconda to install:
 
-If you hare in a hurry for the juicy latest feature you can clone the repository [Polyfem-python](https://github.com/polyfem/polyfem-python) and use `pip` to install:
 ```
-python setup.py install
-```
-and
-```
-python setup.py test
-```
-for testing.
+conda create -n diffipc python=3.9
+conda activate diffipc
+conda install numpy scipy conda-forge::cython pytorch::pytorch
 
-For python documentation [https://polyfem.github.io/python/](https://polyfem.github.io/python/).
+# optional
+export N_THREADS=16
 
-For python jupyter notebook [https://polyfem.github.io/python_examples/](https://polyfem.github.io/python_examples/).
+cd polyfem-python/
+pip install . -v
+```
 
 For full documentation see [https://polyfem.github.io/](https://polyfem.github.io/).

cmake/recipes/CPM.cmake

@@ -0,0 +1,33 @@
+set(CPM_DOWNLOAD_VERSION 0.39.0)
+
+if(CPM_SOURCE_CACHE)
+  set(CPM_DOWNLOAD_LOCATION "${CPM_SOURCE_CACHE}/cpm/CPM_${CPM_DOWNLOAD_VERSION}.cmake")
+elseif(DEFINED ENV{CPM_SOURCE_CACHE})
+  set(CPM_DOWNLOAD_LOCATION "$ENV{CPM_SOURCE_CACHE}/cpm/CPM_${CPM_DOWNLOAD_VERSION}.cmake")
+else()
+  set(CPM_DOWNLOAD_LOCATION "${CMAKE_BINARY_DIR}/cmake/CPM_${CPM_DOWNLOAD_VERSION}.cmake")
+endif()
+
+# Expand relative path. This is important if the provided path contains a tilde (~)
+get_filename_component(CPM_DOWNLOAD_LOCATION ${CPM_DOWNLOAD_LOCATION} ABSOLUTE)
+
+function(download_cpm)
+  message(STATUS "Downloading CPM.cmake to ${CPM_DOWNLOAD_LOCATION}")
+  file(DOWNLOAD
+       https://github.com/cpm-cmake/CPM.cmake/releases/download/v${CPM_DOWNLOAD_VERSION}/CPM.cmake
+       ${CPM_DOWNLOAD_LOCATION}
+  )
+endfunction()
+
+if(NOT (EXISTS ${CPM_DOWNLOAD_LOCATION}))
+  download_cpm()
+else()
+  # resume download if it previously failed
+  file(READ ${CPM_DOWNLOAD_LOCATION} check)
+  if("${check}" STREQUAL "")
+    download_cpm()
+  endif()
+  unset(check)
+endif()
+
+include(${CPM_DOWNLOAD_LOCATION})

cmake/recipes/polyfem.cmake

@@ -7,13 +7,14 @@ endif()
 
 message(STATUS "Third-party: creating target 'polyfem::polyfem'")
 
-include(FetchContent)
-FetchContent_Declare(
-    polyfem
-    GIT_REPOSITORY https://github.com/polyfem/polyfem.git
-    GIT_TAG 12ac634833f91a3946cff26db01972fdb2ec3214
-    GIT_SHALLOW FALSE
-)
-FetchContent_MakeAvailable(polyfem)
-
+# include(FetchContent)
+# FetchContent_Declare(
+#     polyfem
+#     GIT_REPOSITORY https://github.com/polyfem/polyfem.git
+#     GIT_TAG 07ee824f836c445699bbc47ee6f19afbfe39bad4
+#     GIT_SHALLOW FALSE
+# )
+# FetchContent_MakeAvailable(polyfem)
 
+include(CPM)
+CPMAddPackage("gh:polyfem/polyfem#e8bd3d3")

cmake/recipes/polyfem_cpm_cache.cmake

@@ -0,0 +1,24 @@
+#
+# Copyright 2021 Adobe. All rights reserved.
+# This file is licensed to you under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License. You may obtain a copy
+# of the License at http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software distributed under
+# the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS
+# OF ANY KIND, either express or implied. See the License for the specific language
+# governing permissions and limitations under the License.
+#
+
+if(DEFINED ENV{CPM_SOURCE_CACHE})
+    set(CPM_SOURCE_CACHE_DEFAULT $ENV{CPM_SOURCE_CACHE})
+else()
+    # Set CPM cache folder if unset
+    file(REAL_PATH "~/.cache/CPM" CPM_SOURCE_CACHE_DEFAULT EXPAND_TILDE)
+endif()
+
+set(CPM_SOURCE_CACHE
+    ${CPM_SOURCE_CACHE_DEFAULT}
+    CACHE PATH "Directory to download CPM dependencies"
+)
+message(STATUS "Using CPM cache folder: ${CPM_SOURCE_CACHE}")

cmake/recipes/polyfem_data.cmake

@@ -7,7 +7,7 @@ include(FetchContent)
 FetchContent_Declare(
     polyfem_data
     GIT_REPOSITORY https://github.com/polyfem/polyfem-data
-    GIT_TAG 29a46df1fd90c237a82c219f346a956e72bd17d3
+    GIT_TAG f2089eb6eaa22071f7490e0f144e10afe85d4eba
     GIT_SHALLOW FALSE
     SOURCE_DIR ${POLYFEMPY_DATA_ROOT}
 )

cmake/recipes/pybind11.cmake

@@ -1,28 +1,11 @@
-#
-# Copyright 2020 Adobe. All rights reserved.
-# This file is licensed to you under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License. You may obtain a copy
-# of the License at http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software distributed under
-# the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS
-# OF ANY KIND, either express or implied. See the License for the specific language
-# governing permissions and limitations under the License.
-#
+# pybind11 (https://github.com/pybind/pybind11)
+# License: BSD-style
 if(TARGET pybind11::pybind11)
     return()
 endif()
 
 message(STATUS "Third-party: creating target 'pybind11::pybind11'")
 
-include(FetchContent)
-FetchContent_Declare(
-    pybind11
-    GIT_REPOSITORY https://github.com/pybind/pybind11.git
-    GIT_TAG v2.9.2
-    GIT_SHALLOW FALSE
-)
-
 if (POLICY CMP0094)  # https://cmake.org/cmake/help/latest/policy/CMP0094.html
     cmake_policy(SET CMP0094 NEW)  # FindPython should return the first matching Python
 endif ()
@@ -39,7 +22,8 @@ endif ()
 # Pybind11 still uses the deprecated FindPythonInterp. So let's call CMake's
 # new FindPython module and set PYTHON_EXECUTABLE for Pybind11 to pick up.
 # This works well with conda environments.
-find_package(Python REQUIRED COMPONENTS Interpreter Development)
+find_package(Python COMPONENTS Interpreter Development.Module REQUIRED)
 set(PYTHON_EXECUTABLE ${Python_EXECUTABLE})
 
-FetchContent_MakeAvailable(pybind11)
+include(CPM)
+CPMAddPackage("gh:pybind/pybind11@2.13.6")