Skip to content

Add pillow to test dependencies - #139

Merged
gokulkrishna98 merged 1 commit into
apple:mainfrom
gokulkrishna98:dev/gokul/update_pil_dep
Oct 22, 2026
Merged

gokulkrishna98 merged 1 commit into
apple:mainfrom
gokulkrishna98:dev/gokul/update_pil_dep

Conversation

@gokulkrishna98

Copy link
Copy Markdown
Contributor

Summary

tests/test_image_io.py imports PIL directly, but pillow was only installed transitively. This made deptry fail:

tests/test_image_io.py:19:1: DEP003 'PIL' imported but it is a transitive dependency

Changes

  • Add pillow to the test optional dependencies.
  • Map the pillow distribution to its PIL import name in [tool.deptry].package_module_name_map.

Testing

  • deptry coreai_torch tests reports no dependency issues.

tests/test_image_io.py imports PIL directly, but pillow was only available
transitively, causing deptry to fail with DEP003. Declare it in the test
extras and map the pillow distribution to its PIL import name for deptry.
@gokulkrishna98
gokulkrishna98 merged commit 7a18c00 into apple:main Oct 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants