Skip to content

Commit

Permalink
Update CI to also run python tests w
Browse files Browse the repository at this point in the history
ith numpy 1
  • Loading branch information
jparismorgan committed Oct 15, 2024
1 parent 45853f2 commit 712a276
Showing 1 changed file with 54 additions and 0 deletions.
54 changes: 54 additions & 0 deletions .github/workflows/ci-python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ jobs:
os: [ubuntu-latest]
python-version: ["3.9"]
runs-on: ${{ matrix.os }}
continue-on-error: true
steps:
- name: Install OpenBLAS
run: sudo apt install libopenblas-dev
Expand All @@ -29,6 +30,59 @@ jobs:
- name: Build and test python
run: |
pip install .[test]
pip list
cd apis/python
pytest -n logical --durations=0
# TODO: fix editable on linux
#pip uninstall -y tiledb.vector_search
#pip install -e .
#pytest
pip install -r test/ipynb/requirements.txt
export TILEDB_REST_TOKEN=$TILEDB_CLOUD_HELPER_VAR
pytest -n logical --durations=0 --nbmake test/ipynb
env:
TILEDB_CLOUD_HELPER_VAR: ${{ secrets.TILEDB_CLOUD_HELPER_VAR }}
shell: bash -el {0}
- name: Check tiledb-vector-search version
run: |
python -c "from tiledb.vector_search.version import version; print(version)"
# This is a temporary job where we will build with numpy2, but run with numpy1.
run-tests-numpy-1:
strategy:
matrix:
os: [ubuntu-latest]
python-version: ["3.9"]
runs-on: ${{ matrix.os }}
continue-on-error: true
steps:
- name: Install OpenBLAS
run: sudo apt install libopenblas-dev
- uses: actions/checkout@v3
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v2
with:
python-version: ${{ matrix.python-version }}
- name: Print Python version
run: |
which python
which pip
python --version
- name: Build and test python
run: |
# This will build with numpy 2.
pip install .[test]
pip list
# Then we will uninstall numpy 2 and install numpy 1.
pip uninstall -y numpy
pip install numpy==1.25.0
pip list
cd apis/python
pytest -n logical --durations=0
# TODO: fix editable on linux
Expand Down

0 comments on commit 712a276

Please sign in to comment.