-
Notifications
You must be signed in to change notification settings - Fork 114
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
Conversation
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
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
There was a problem hiding this 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?
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>
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>
@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? |
Closing in favour of: #684 |
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 theDirectory
class to be able to retrieve theField
and itsDataType
instance.