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

using the '-n' option for an image with gps coordinates gives warnings #7

Closed
rempargo opened this issue Aug 30, 2016 · 1 comment
Closed

using the '-n' option for an image with gps coordinates gives warnings #7

rempargo opened this issue Aug 30, 2016 · 1 comment
Assignees
@morozgrafix
Labels
bug

Comments

@rempargo
Copy link

I'm running the following command with the extra argument -n to extract the real exif data (I want the integer value for the orientation tag).

Exiftool.new('some-image-file.jpg,'-n').to_hash

I see some warnings for both the 'GPSLatitude' & 'GPSLongitude':
:gps_latitude=>"Warning: Parsing '52.7353472222222' for attribute 'GPSLatitude' raised undefined methodsplit' for 52.7353472222222:Float"`

added an image for any testing:
2016-08-30 11 06 47
@rempargo rempargo changed the title using the '-n' option for an image with gps coordinates gies using the '-n' option for an image with gps coordinates gives warnings Aug 30, 2016
morozgrafix pushed a commit to morozgrafix/exiftool that referenced this issue Oct 5, 2016
Do not attempt to split raw numeric GPS coordinates
add8ec4 
Fixes exiftool-rb#7
If `-n` option is used, GPS coordinates come as numeric values and there is no need to attempt to split cardinal directions from the value.

Also added a test for this scenario.
@morozgrafix morozgrafix mentioned this issue Oct 5, 2016
Merged
@morozgrafix morozgrafix added the bug label Oct 5, 2016
@morozgrafix morozgrafix self-assigned this Oct 5, 2016
@mceachen
Copy link
Member

mceachen commented Oct 26, 2016 •
edited
Loading

Merged PR7 and released v1.0.0.

Thanks for opening the issue, @rempargo and the PR @morozgrafix ! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@morozgrafix morozgrafix

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mceachen @rempargo @morozgrafix