chore: Pillow/numpy/robotics extras を bump し Python 3.13/3.14 を CI に追加#275
Open
rikunosuke wants to merge 7 commits into
Open
chore: Pillow/numpy/robotics extras を bump し Python 3.13/3.14 を CI に追加#275rikunosuke wants to merge 7 commits into
rikunosuke wants to merge 7 commits into
Conversation
Adds smoke tests for __export_index_color_image and __create_image_with_annotation so Pillow major-version bumps surface API breakage in CI. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Phase B of #273. Pillow 11/12 wheels are required for Python 3.13/3.14. Used APIs (Image.open/new/fromarray/composite, convert, putpalette, save, ImageDraw, ImageColor) have no breaking changes across 10 -> 12. Covered by tests/test_pillow_exports.py. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds smoke tests for _build_episode_map, _convert_episode_frames, get_episode_indices, and check_dependencies so pandas/pyarrow version bumps surface API breakage in CI. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Part of #273 Phase B. These lower bounds are required for numpy 2 compatibility and Python 3.13/3.14 wheel availability, in preparation for Phase C. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- numpy: >=1.26.0,<2.0.0 -> >=1.26.0,<3.0.0 (1.x/2.x both supported) - requires-python: >=3.8 -> >=3.10 - release.yml: Python 3.8 -> 3.10 - test.yml: matrix Python 3.10-3.14 x numpy 1.26 / 2.x (3.13/3.14 excluded for numpy 1.26 since no wheels exist) - test.yml: also install robotics extras to exercise pandas/pyarrow numpy 2.0 removed APIs (np.float_, np.NaN, np.product, etc.) are not used. Used APIs (np.array, np.vstack, np.uint8, np.int32) are stable across 1/2. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Bumps Pillow / numpy / robotics extras to versions that support Python 3.13–3.14, raises the minimum Python to 3.10, expands the CI matrix accordingly, and adds smoke tests so future major-version bumps of Pillow / pandas / pyarrow are caught by CI.
Changes:
- Update
requires-pythonto>=3.10; widennumpyto<3.0.0, bumpPillowto>=11.0.0,<13.0.0, and bump robotics extras (pandas>=2.2.2,pyarrow>=18.0.0). - Expand
test.ymlmatrix to Python 3.10–3.14 × numpy 1.26/2.x (excluding 3.13/3.14 + numpy 1.26), installroboticsextras, and pin numpy per matrix; bumprelease.ymlPython to 3.10. - Add
tests/test_pillow_exports.pyandtests/test_lerobot_v3_parquet.pysmoke tests covering Pillow / pandas / pyarrow usage.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Raise min Python to 3.10; widen numpy/Pillow; bump robotics extras. |
| requirements.txt | Mirror numpy/Pillow bumps in requirements file. |
| .github/workflows/test.yml | Expand matrix to Python 3.10–3.14 × numpy 1.26/2.x; install robotics extras. |
| .github/workflows/release.yml | Bump release Python from 3.8 to 3.10. |
| tests/test_pillow_exports.py | New smoke tests for Pillow-using code paths. |
| tests/test_lerobot_v3_parquet.py | New smoke tests for pandas/pyarrow code paths in lerobot v3. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
#273 の Phase B + Phase C を 1 PR にまとめて対応。
>=10.0.0,<11.0.0→>=11.0.0,<13.0.0(Python 3.13/3.14 wheel 対応)>=1.26.0,<2.0.0→>=1.26.0,<3.0.0(1.x / 2.x 両対応)>=3.8→>=3.10(Python 3.8/3.9 サポート終了)pandas>=2.0.0→>=2.2.2、pyarrow>=14.0.0→>=18.0.0(numpy 2 / Python 3.13+ 対応)3.8→3.10Python 3.10–3.14 × numpy 1.26 / 2.xに拡張(3.13/3.14 + numpy 1.26 は wheel 未提供のため exclude)。robotics extras も install して pandas/pyarrow も実行。変更点
pyproject.toml/requirements.txtrequires-python>=3.8>=3.10numpy>=1.26.0,<2.0.0>=1.26.0,<3.0.0Pillow>=10.0.0,<11.0.0>=11.0.0,<13.0.0pandas(robotics)>=2.0.0>=2.2.2pyarrow(robotics)>=14.0.0>=18.0.0.github/workflows/test.ymlコードへの影響
np.float_,np.NaN,np.product等)は未使用np.array,np.vstack,np.uint8,np.int32,Image.open/new/fromarray/composite,pd.read_parquet,iterrows等)は対象範囲で安定テスト計画
pytest全 40 件パス🤖 Generated with Claude Code