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.

Sign up for GitHub

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

Jump to bottom

Parse orientation from XMP tag contents #6463

Merged
merged 7 commits into from
Jul 30, 2022
Merged

Parse orientation from XMP tag contents #6463

merged 7 commits into from
Jul 30, 2022

Conversation

bigcat88
Copy link
Contributor

Fixes #6462 .

@bigcat88
Copy link
Contributor Author

For me it looks like this fail test is not related to this PR...

@radarhere
Copy link
Member

It's an intermittent failure in our CIs. I've rerun the test, and it passes now.

@radarhere radarhere mentioned this pull request Jul 27, 2022
Merged
@radarhere radarhere mentioned this pull request Jul 28, 2022
Merged
@bigcat88
Merge pull request #2 from radarhere/xmp-tags-orientation
5d8dacf 
Check that orientation is still absent after reloading Exif
@radarhere radarhere added the automerge Automatically merge PRs that are ready label Jul 30, 2022
@radarhere radarhere changed the title XMP tags orientation fix Parse orientation from XMP tag contents Jul 30, 2022
@mergify mergify bot merged commit 363586b into python-pillow:main Jul 30, 2022
@bigcat88 bigcat88 deleted the xmp-tags-orientation branch May 11, 2023 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@radarhere radarhere radarhere approved these changes

Assignees
No one assigned
Labels
automerge Automatically merge PRs that are ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

XMP tag orientation generated by exiftool is not handled
2 participants
@bigcat88 @radarhere