Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix loading IPTC images and update test #7667

Merged
merged 4 commits into from
Jan 1, 2024
Merged

Conversation

nulano
Copy link
Contributor

@nulano nulano commented Dec 31, 2023

Follow up to #7661, originally created as radarhere#22

I added a byte for the image data to also test the load method, but found that, at least on Windows, this fails due to the unusual use of a temporary file, so I replaced it with io.BytesIO.

I also realigned the test bytes to avoid breaking in the middle of a dataset.

nulano and others added 2 commits December 31, 2023 13:53
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
@radarhere radarhere merged commit 4f17b60 into python-pillow:main Jan 1, 2024
56 checks passed
@nulano nulano deleted the iptc branch January 1, 2024 09:35
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.

3 participants