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

NXP-15437: Picture orientation is not detected properly #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ThibArg
Copy link
Contributor

@ThibArg ThibArg commented Oct 6, 2014

Adding the -auto-orient parameter to the ImageMagick command line arguments. Cf http://www.imagemagick.org/script/command-line-options.php#auto-orient

The potential problem is the one described in IM doc...

This operator reads and resets the EXIF image profile setting 'Orientation' and then performs the appropriate 90 degree rotation on the image to orient the image, for correct viewing.

This EXIF profile setting is usually set using a gravity sensor in digital camera, however photos taken directly downward or upward may not have an appropriate value. Also images that have been orientation 'corrected' without reseting this setting, may be 'corrected' again resulting in a incorrect result

...so using -auto-orient could, maybe, lead to incorrect orientation on some pictures?

@efge
Copy link
Collaborator

efge commented Oct 7, 2014

It's better than not having it. A piece of software that rotated an image without updating the EXIF profile is broken, we can't deal with everything.

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