Skip to content

Olympusrawdev #44

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

Merged
merged 1 commit into from
Mar 30, 2016
Merged

Olympusrawdev #44

merged 1 commit into from
Mar 30, 2016

Conversation

kwhopper
Copy link
Collaborator

Implements another Olympus subifd (Raw Development, 0x2030) as a directory. Cleans-up more format 13 errors from the images project.

@drewnoakes
Copy link
Owner

Looks great.

Before I merge it, can you remove the first two commits just to keep things clearer? You can do that via an interactive rebase. Let me know if it's something you'd like some help with. Also, I've fetched your commits to my machine as a backup, so don't worry about losing anything.

Use DateTimeOffset instead of DateTime

Revert "Use DateTimeOffset instead of DateTime"

This reverts commit 0ec0dca.

Olympus Raw Development subifd (0x2030) directory
@kwhopper
Copy link
Collaborator Author

That was... odd, but I think it's ok now. If not, let me know.

@drewnoakes
Copy link
Owner

Perfect. The build failed due to an error on the AppVeyor side it seems. Have triggered a new build and will merge if/when it passes.

@drewnoakes drewnoakes merged commit 2e4ba16 into drewnoakes:master Mar 30, 2016
@kwhopper kwhopper deleted the olympusrawdev branch March 30, 2016 22:01
drewnoakes added a commit to drewnoakes/metadata-extractor-images that referenced this pull request Apr 20, 2016
- Raw Development (drewnoakes/metadata-extractor-dotnet#44)
- Raw Development 2 (drewnoakes/metadata-extractor-dotnet#45)
- Image Processing (drewnoakes/metadata-extractor-dotnet#46)
- Focus Info (drewnoakes/metadata-extractor-dotnet#47)

Addresses "Invalid TIFF tag format code 13 for tag 0x20*0" issues.

Many thanks to @kwhopper for these PRs!
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