Fix build errors (#832)

* up the version to 0.15.5dev
* remove python setup.py build_ext --inplace from the github action and install instruction
* update python versions to test in github action files
* add [optional] to the graphviz installation step in the installation doc

Author: jeongyoonlee

.github/workflows/python-test.yaml

@@ -27,7 +27,6 @@ jobs:
             sudo apt install graphviz
             pip install --upgrade pip
             pip install --upgrade setuptools
-            python -m pip install .[test]
-            python setup.py build_ext --inplace
+            python -m pip install -e ".[test]"
       - name: Test with pytest
         run: pytest -vs tests/ --cov causalml/

.github/workflows/test-build-from-source.yml

@@ -19,17 +19,19 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest]
-        python-version: [3.7, 3.8, 3.9]
+        python-version: ["3.9", "3.10", "3.11", "3.12"]
         tf-label: ['', '-tf']
         include:
-          - python-version: 3.7
-            python-version-nd: 37
-          - python-version: 3.8
-            python-version-nd: 38
-          - python-version: 3.9
+          - python-version: "3.9"
             python-version-nd: 39
+          - python-version: "3.10"
+            python-version-nd: 310
+          - python-version: "3.11"
+            python-version-nd: 311
+          - python-version: "3.12"
+            python-version-nd: 312
           - tf-label: '-tf'
-            tf-label-pip: '[tf]'
+            tf-label-pip: ',tf'
 
     steps:
       - name: checkout repository
@@ -68,9 +70,7 @@ jobs:
         run: |
             pip install -U pip
             pip install -U setuptools
-            python -m pip install .${{ matrix.tf-label-pip}}
-            python -m pip install .[test]
-            python setup.py build_ext --inplace
+            python -m pip install -e ".[test${{ matrix.tf-label-pip}}]"
 
       - name: Test with pytest
         run: pytest -vs tests/ --cov causalml/

.github/workflows/test-conda-forge-install.yml

@@ -14,15 +14,17 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest]
-        python-version: [3.7, 3.8, 3.9]
+        python-version: ["3.9", "3.10", "3.11", "3.12"]
         include:
-          - python-version: 3.7
-            python-version-nd: 37
-          - python-version: 3.8
-            python-version-nd: 38
-          - python-version: 3.9
+          - python-version: "3.9"
             python-version-nd: 39
-
+          - python-version: "3.10"
+            python-version-nd: 310
+          - python-version: "3.11"
+            python-version-nd: 311
+          - python-version: "3.12"
+            python-version-nd: 312
+ 
     steps:
       - uses: actions/checkout@v2
       - name: Set up Python ${{ matrix.python-version }}

.github/workflows/test-pypi-install.yml

@@ -12,14 +12,16 @@ jobs:
       fail-fast: false
       matrix:
         os: [ubuntu-latest]
-        python-version: [3.7, 3.8, 3.9]
+        python-version: ["3.9", "3.10", "3.11", "3.12"]
         include:
-          - python-version: 3.7
-            python-version-nd: 37
-          - python-version: 3.8
-            python-version-nd: 38
-          - python-version: 3.9
+          - python-version: "3.9"
             python-version-nd: 39
+          - python-version: "3.10"
+            python-version-nd: 310
+          - python-version: "3.11"
+            python-version-nd: 311
+          - python-version: "3.12"
+            python-version-nd: 312
 
     steps:
       - uses: actions/checkout@v2

docs/installation.rst

@@ -42,7 +42,6 @@ Install ``causalml`` with ``tensorflow`` for ``DragonNet`` from ``PyPI``
 .. code-block:: bash
 
     pip install causalml[tf]
-    pip install -U numpy                            # this step is necessary to fix [#338](https://github.com/uber/causalml/issues/338)
 
 Install ``causalml`` with ``torch`` for ``CEVAE`` from ``PyPI``
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
@@ -55,37 +54,33 @@ Install ``causalml`` with ``torch`` for ``CEVAE`` from ``PyPI``
 Install from source
 -------------------
 
-Create a clean ``conda`` environment.
+[Optional] If you don't have Graphviz installed, you can install it using ``conda``, ``brew`` (on MacOS), or ``apt`` (on Linux).
 
 .. code-block:: bash
 
-    conda create -n causalml-py38 -y python=3.8
-    conda activate causalml-py38
-    conda install -c conda-forge cxx-compiler
     conda install python-graphviz
-    conda install -c conda-forge xorg-libxrender
-    conda install -c conda-forge libxcrypt
+    brew install graphviz           # MacOS
+    sudo apt-get install graphviz   # Linux
 
-Then:
+First, clone the repository and install the package:
 
 .. code-block:: bash
 
     git clone https://github.com/uber/causalml.git
     cd causalml
-    pip install .
-    python setup.py build_ext --inplace
+    pip install -e .
 
 with ``tensorflow`` for ``DragonNet``:
 
 .. code-block:: bash
 
-    pip install .[tf]
+    pip install -e ".[tf]"
 
 with ``torch`` for ``CEVAE``:
 
 .. code-block:: bash
 
-    pip install .[torch]
+    pip install -e ".[torch]"
 
 =======
 
@@ -100,6 +95,10 @@ Running Tests
 
 Make sure pytest is installed before attempting to run tests.
 
+.. code-block:: bash
+
+    pip install -e ".[test]"
+
 Run all tests with:
 
 .. code-block:: bash

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "causalml"
-version = "0.15.4"
+version = "0.15.5dev"
 description = "Python Package for Uplift Modeling and Causal Inference with Machine Learning Algorithms"
 readme = { file = "README.md", content-type = "text/markdown" }
 

setup.py

@@ -6,16 +6,17 @@
 try:
     from Cython.Build import cythonize
 except ImportError:
-    dist.Distribution().fetch_build_eggs(["cython<=0.29.34"])
+    dist.Distribution().fetch_build_eggs(["cython"])
     from Cython.Build import cythonize
 import Cython.Compiler.Options
 
 Cython.Compiler.Options.annotate = True
+
 try:
-    from numpy import get_include as np_get_include
+    import numpy as np
 except ImportError:
     dist.Distribution().fetch_build_eggs(["numpy"])
-    from numpy import get_include as np_get_include
+    import numpy as np
 
 # fmt: off
 cython_modules = [
@@ -34,7 +35,7 @@
         name,
         [source],
         libraries=[],
-        include_dirs=[np_get_include()],
+        include_dirs=[np.get_include()],
         extra_compile_args=["-O3"],
     )
     for name, source in cython_modules
@@ -51,5 +52,9 @@
 setup(
     packages=packages,
     ext_modules=cythonize(extensions, annotate=True, nthreads=nthreads),
-    include_dirs=[np_get_include()],
+    include_dirs=[np.get_include()],
+    setup_requires=[
+        "cython",
+        "numpy",
+    ],
 )