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

Add ICC Profile name for JPEG and TIFF files. #249

Merged
merged 6 commits into from
May 11, 2017

Conversation

tledoux
Copy link
Contributor

@tledoux tledoux commented May 3, 2017

Validate ICC profile using java standard ICC_Profile class
Decode v2 and v4 ICC Profiles description according to
Specification ICC.1:2001-04 http://www.color.org/ICC_Minor_Revision_for_Web.pdf
Specification ICC.1:2010-12 http://www.color.org/ICC1-V41.pdf
Add unit test for v2 and v4 ICC profiles.
Test output of iccProfileName in NisoImageMetadata v0.2, v1.0 and v2.0.
Extract ICCProfile from TIFF tag ICCProfile.
Extracy ICCProfile from APP2 segment in JPEG.

Fixes #61
Fixes #62

Validate ICC profile using java standard ICC_Profile class
Decode v2 and v4 ICC Profiles description according to
  Specification ICC.1:2001-04 http://www.color.org/ICC_Minor_Revision_for_Web.pdf
  Specification ICC.1:2010-12 http://www.color.org/ICC1-V41.pdf
Add unit test for v2 and v4 ICC profiles.
Test output of iccProfileName in NisoImageMetadata v0.2, v1.0 and v2.0.
Extract ICCProfile from TIFF tag ICCProfile.
Extracy ICCProfile from APP2 segment in JPEG.

Fixes openpreserve#61
Fixes openpreserve#62
@codecov
Copy link

codecov bot commented May 3, 2017

Codecov Report

Merging #249 into integration will increase coverage by 0.14%.
The diff coverage is 69.13%.

Impacted file tree graph

@@               Coverage Diff                @@
##             integration    #249      +/-   ##
================================================
+ Coverage          40.36%   40.5%   +0.14%     
- Complexity          3128    3136       +8     
================================================
  Files                392     392              
  Lines              31400   31474      +74     
  Branches            6114    6123       +9     
================================================
+ Hits               12674   12748      +74     
+ Misses             16349   16337      -12     
- Partials            2377    2389      +12
Impacted Files Coverage Δ Complexity Δ
...du/harvard/hul/ois/jhove/module/tiff/TiffTags.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...edu/harvard/hul/ois/jhove/handler/TextHandler.java 0.55% <0%> (ø) 2 <0> (ø) ⬇️
.../edu/harvard/hul/ois/jhove/handler/XmlHandler.java 55% <100%> (+0.55%) 177 <0> (+1) ⬆️
...edu/harvard/hul/ois/jhove/module/tiff/TiffIFD.java 37.27% <54.54%> (+0.02%) 186 <0> (ø) ⬇️
...a/edu/harvard/hul/ois/jhove/module/JpegModule.java 34.88% <61.29%> (+1.04%) 47 <2> (+3) ⬆️
...a/edu/harvard/hul/ois/jhove/NisoImageMetadata.java 69.67% <83.33%> (+1.61%) 157 <3> (+3) ⬆️
...ava/edu/harvard/hul/ois/jhove/module/tiff/IFD.java 57.73% <0%> (+1.37%) 63% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d336df8...126cda2. Read the comment docs.

@tledoux tledoux added ready and removed in progress labels May 3, 2017
@tledoux tledoux added this to the JHOVE hack day activities milestone May 3, 2017
@tledoux tledoux self-assigned this May 3, 2017
@tledoux tledoux added in progress and removed ready labels May 3, 2017
@tledoux tledoux requested a review from carlwilson May 9, 2017 15:19
Copy link
Member

@carlwilson carlwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tledoux, saw your message and had already had a look and have approved this now.

@carlwilson carlwilson merged commit 6d79018 into openpreserve:integration May 11, 2017
@tledoux tledoux mentioned this pull request May 12, 2017
@tledoux tledoux deleted the issue62 branch May 12, 2017 11:26
rgfeldman added a commit to rgfeldman/jhove that referenced this pull request Apr 10, 2019
rgfeldman added a commit to rgfeldman/jhove that referenced this pull request Apr 10, 2019
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.

None yet

2 participants