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

XMP in xmp field instead of being #71

Closed
tinco opened this issue Sep 13, 2023 · 1 comment
Closed

XMP in xmp field instead of being #71

tinco opened this issue Sep 13, 2023 · 1 comment

Comments

@tinco
Copy link

tinco commented Sep 13, 2023

Hi Remco,

We're using a Gem that's last been updated 13 years ago to parse the XMP from EXIFR, and randomly we've got some files that didn't work in the gem, and they way I fixed it was this little branch:

https://github.com/aeroscan-nl/xmp/blob/master/lib/xmp.rb#L71

I added some example files in the spec fixtures, feel free to do whatever you want with them if you're interested.

Do you have any idea why the XMP is in the xmp field for that specific jpg, but for the other ones it's in that app1s? Is this a valid JPG thing to happen or did one of our tools or the drone itself mangle this jpg?

@remvee
Copy link
Owner

remvee commented Sep 14, 2023

I don't know why the original amberbit xmp implementation did never considered the xmp tiff tag in the first place, I guess they did not need it for their purposes. Looking at the github insights of that project and forks, it seems the xmp gem does not have a proper maintainer. I did notice this RandieM/xmp@43dfec8 when browsing in the network graph of forks. Maybe contact the developer to see if you can find a good future for xmp gem together?

But honestly, I'm not the guy to ask. I've never used xmp myself and have no interest in it because I don't own any equipment supporting it. Maybe I should remove the reference to the amberbit repo from the readme because it's very outdated..

@remvee remvee closed this as completed Sep 14, 2023
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

No branches or pull requests

2 participants