Update GitHub Actions for recent TensorFlow and PyTorch versions - #115
Conversation
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
d3cd98b to
b407784
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR updates the testing infrastructure to support newer versions of TensorFlow and PyTorch while dropping support for Python 3.8 and adding Python 3.11 support. The changes focus on modernizing the CI/CD pipeline and ensuring compatibility with the latest library versions.
- Upgrade TensorFlow support from versions 2.4-2.11 to 2.11-2.15
- Upgrade PyTorch support from versions 1.7-2.0 to 1.13-2.6
- Drop Python 3.8 support and add Python 3.11 support
- Version bump from 0.3.0 to 0.4.0
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/python-tests-torch.yml | Updates PyTorch testing matrix to use Python 3.9-3.11 with newer PyTorch versions |
| .github/workflows/python-tests-tf.yml | Updates TensorFlow testing matrix to use Python 3.9-3.11 with newer TensorFlow versions |
| .github/workflows/python-linters.yml | Updates linting workflow to support Python 3.9-3.11 |
| .github/workflows/python-coverage-report.yml | Updates coverage testing to use Python 3.10 with TensorFlow 2.15 and PyTorch 2.2 |
| setup.cfg | Comprehensive update of tox environments with new library versions and dependency specifications |
| setup.py | Removes faiss_gpu dependency and bumps version to 0.4.0 |
| oodeel/init.py | Updates package version to 0.4.0 |
| .bumpversion.cfg | Updates version configuration to 0.4.0 |
| docs/index.md | Updates compatibility table with new Python and library versions |
| README.md | Updates compatibility table with new Python and library versions |
| tests/tests_torch/datasets/test_torch_datahandler.py | Fixes batch size assertions to handle smaller datasets |
| tests/tests_tensorflow/datasets/test_tf_datahandler.py | Fixes batch size assertions to handle smaller datasets |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| torch-version: 2.6 | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 |
There was a problem hiding this comment.
The Python setup step was removed but shell configuration bash -el {0} suggests conda/mamba environment usage. Without the Python setup step, the workflow may fail if the default Python version doesn't match the matrix requirements.
| - uses: actions/checkout@v3 | |
| - uses: actions/checkout@v3 | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v4 | |
| with: | |
| python-version: ${{ matrix.python-version }} |
| tf-version: 2.15 | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 |
There was a problem hiding this comment.
The Python setup step was removed but shell configuration bash -el {0} suggests conda/mamba environment usage. Without the Python setup step, the workflow may fail if the default Python version doesn't match the matrix requirements.
| - uses: actions/checkout@v3 | |
| - uses: actions/checkout@v3 | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v4 | |
| with: | |
| python-version: ${{ matrix.python-version }} |
| torch-version: "2.2" | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 |
There was a problem hiding this comment.
The Python setup step was removed but shell configuration bash -el {0} suggests conda/mamba environment usage. Without the Python setup step, the workflow may fail if the default Python version doesn't match the matrix requirements.
| - uses: actions/checkout@v3 | |
| - uses: actions/checkout@v3 | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v4 | |
| with: | |
| python-version: ${{ matrix.python-version }} |
| extend-ignore = | ||
| # See https://github.com/PyCQA/pycodestyle/issues/373 | ||
| E203, | ||
| E203 |
There was a problem hiding this comment.
The trailing comma was removed from the extend-ignore configuration. This change makes it harder to add new ignore codes in the future and goes against common formatting practices for configuration lists.
| E203 | |
| E203, |
b407784 to
a4ed251
Compare
| max-parallel: 4 | ||
| matrix: | ||
| python-version: ["3.8", "3.9", "3.10"] | ||
| python-version: ["3.9", "3.10", "3.11"] |
There was a problem hiding this comment.
Not so important, but Python 3.9 support will end very soon (in the next weeks). We could already test with 3.10 to 3.12 (or even 3.13).
Enhance the testing workflow by including support for newer versions of TensorFlow and PyTorch in GitHub Actions. This update ensures compatibility with Python 3.11 and 3.12, along with their respective library versions.