Skip to content

Commit 0ccae86

Browse files
kbattocchistar1327p
authored andcommitted
Pin cython<3
Signed-off-by: Keith Battocchi <kebatt@microsoft.com> Signed-off-by: star1327p <star1327p@gmail.com>
1 parent 9c568e4 commit 0ccae86

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

.github/workflows/publish-documentation.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ jobs:
6363
python-version: 3.8 # because of our supported TensorFlow versions, must build on 3.6-3.8
6464
- run: python -m pip install --upgrade pip && pip install --upgrade setuptools
6565
name: Ensure latest pip and setuptools
66-
- run: pip install -U cython && pip install -e .[all] ${{ inputs.use_lkg && '-r lkg.txt' }}
66+
- run: pip install -e .[all] ${{ inputs.use_lkg && '-r lkg.txt' }}
6767
name: Install econml[all]
6868
- run: sudo apt-get -yq install graphviz
6969
name: Install graphviz

.github/workflows/publish-package.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ jobs:
9191
python-version: 3.8 # because of our supported TensorFlow versions, must build on 3.6-3.8
9292
- run: python -m pip install --upgrade pip && pip install --upgrade setuptools
9393
name: Ensure latest pip and setuptools
94-
- run: pip install -U cython && pip install -e .[all] ${{ inputs.use_lkg && '-r lkg.txt' }}
94+
- run: pip install -e .[all] ${{ inputs.use_lkg && '-r lkg.txt' }}
9595
name: Install econml[all]
9696
- run: python setup.py sdist
9797
name: Build sdist

pyproject.toml

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ requires = [
44
"wheel",
55
"oldest-supported-numpy",
66
"scipy",
7-
"cython"
7+
"cython<3" # Our native code has several incompatibilities with Cython 3
88
]
99
build-backend = "setuptools.build_meta"
1010

0 commit comments

Comments
 (0)