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

Support long & short values for TIFF EXIF orientation tag. #549

Closed
wants to merge 1 commit into from

Conversation

La0
Copy link

@La0 La0 commented Dec 30, 2021

This pull request fixes the issue #548 by inspecting the EXIF field's data type before converting it to Orientation.

I added a getField method in the Directory class to be able to retrieve the Field and its DataType instance.

@mrghosti3 mrghosti3 mentioned this pull request Mar 3, 2023
mrghosti3 added a commit to mrghosti3/cantaloupe that referenced this pull request Mar 21, 2023
Thanks to original contributor Bastien Abadie <bastien@nextcairn.com>

Original pr: cantaloupe-project#549

Changes are ported over to 'release/5.0' branch for fix cantaloupe-project#548
mrghosti3 added a commit to mrghosti3/cantaloupe that referenced this pull request Mar 21, 2023
Thanks to original contributor Bastien Abadie <bastien@nextcairn.com>

Original pr: cantaloupe-project#549

Changes are ported over to 'release/5.0' branch for fix cantaloupe-project#548
Copy link
Contributor

@janhoy janhoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, hope this gets merged soon!

Can you fix the conflicts and ping the maintainers to merge this for next release?

mrghosti3 added a commit to mrghosti3/cantaloupe that referenced this pull request Apr 13, 2023
Original pr: cantaloupe-project#549

Changes are ported over to 'release/5.0' branch for fix cantaloupe-project#548

Co-authored-by: mrghosti3 <mysteriousghst@gmail.com>
mrghosti3 added a commit to mrghosti3/cantaloupe that referenced this pull request Feb 16, 2024
Original pr: cantaloupe-project#549

Changes are ported over to 'release/5.0' branch for fix cantaloupe-project#548

Co-authored-by: mrghosti3 <mysteriousghst@gmail.com>
@glenrobson glenrobson requested a review from jcoyne July 31, 2024 15:38
@jcoyne
Copy link
Contributor

jcoyne commented Jul 31, 2024

@La0 This seems really useful and we're interested in merging this. Are you still interested in contributing (sorry for the very long delay). Are you able to rebase this and fix the conflicts? Can you add a test?

janhoy added a commit to janhoy/cantaloupe that referenced this pull request Aug 9, 2024
@glenrobson
Copy link
Contributor

Closing in favour of: #684

@glenrobson glenrobson closed this Aug 14, 2024
janhoy added a commit to janhoy/cantaloupe that referenced this pull request Aug 22, 2024
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.

4 participants