Conversation
Co-authored-by: alexlib <747110+alexlib@users.noreply.github.com>
Co-authored-by: alexlib <747110+alexlib@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates dependency versions across the OpenPIV project to use the latest stable releases, with a major upgrade to NumPy 2.x from the previous 1.21.6 requirement. The changes improve performance, security, and compatibility with modern Python versions.
Changes:
- Updated all dependency minimum version requirements in both
setup.pyandpyproject.tomlto latest stable versions - Upgraded NumPy from >=1.21.6 to >=2.0.0 (installed 2.2.6)
- Updated Python version classifiers to reflect support for Python 3.10-3.13 (removed 3.7-3.9)
- Removed outdated numpy requirement from documentation requirements file
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| setup.py | Updated all dependency version constraints and Python classifiers to support 3.10-3.13 |
| pyproject.toml | Updated dependencies to match setup.py, changed from caret (^) to >= syntax for consistency, added Python 3.13 classifier |
| poetry.lock | Automatically generated lock file with updated package versions including numpy 2.2.6, scipy 1.13.0, matplotlib 3.8.4, scikit-image 0.23.1, and imageio 2.37.2 |
| openpiv/docs/requirements.txt | Removed outdated numpy>=1.9 requirement to avoid conflicts |
| 'Programming Language :: Python :: 3.10', | ||
| 'Programming Language :: Python :: 3.11', | ||
| 'Programming Language :: Python :: 3.12', | ||
| 'Programming Language :: Python :: 3.13', |
There was a problem hiding this comment.
Python 3.13 classifier added but not tested in CI. The GitHub Actions workflow in .github/workflows/testing.yml only tests Python 3.10, 3.11, and 3.12. To properly support Python 3.13, it should be added to the CI test matrix to verify compatibility with all dependencies, especially numpy 2.x and scipy 1.13.0.
| "Programming Language :: Python :: 3.10", | ||
| "Programming Language :: Python :: 3.11", | ||
| "Programming Language :: Python :: 3.12", | ||
| "Programming Language :: Python :: 3.13", |
There was a problem hiding this comment.
Python 3.13 classifier added but not tested in CI. The GitHub Actions workflow in .github/workflows/testing.yml only tests Python 3.10, 3.11, and 3.12. To properly support Python 3.13, it should be added to the CI test matrix to verify compatibility with all dependencies, especially numpy 2.x and scipy 1.13.0.
Dependency Update Plan
setup.pydependencies to latest versions:setup.pyPython version classifiers (3.10, 3.11, 3.12, 3.13)pyproject.tomldependencies to match setup.pyopenpiv/docs/requirements.txtto remove outdated numpy requirementSummary
Successfully updated all dependencies to latest versions:
All 203 tests pass successfully with no deprecation warnings related to numpy 2.x. The codebase is fully compatible with NumPy 2.x without requiring any code changes.
Original prompt
This pull request was created from Copilot chat.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.