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

Enhancement of JPEG-hul module to handle the APP14 marker segment #518

Merged
merged 3 commits into from
Dec 10, 2019

Conversation

tledoux
Copy link
Contributor

@tledoux tledoux commented Nov 5, 2019

Calculate the appropriate colorspace depending on the number of components and the segments
Add the "Adobe JPEG" profile and the information on the corresponding ISO/IEC 10918-6:2013
Suppress the warning message about deprecated JPEG compression scheme coming from the TIFF module
Add more control on the validity of the format :

  • 3 components only for EXIF
  • 1, 3 or 4 components for other profiles
    Add a YCCK colorspace value to the niso property
    The JPEG module goes to version 1.5.2
    Fixes Possible issue with ColorSpace #297

Calculate the appropriate colorspace depending on the number of components and the segments
Add the "Adobe JPEG" profile and the information on the corresponding ISO/IEC 10918-6:2013
Suppress the warning message about deprecated JPEG compression scheme coming from the TIFF module
Add more control on the validity of the format :
- 3 components only for EXIF
- 1, 3 or 4 components for other profiles
Add a YCCK colorspace value to the niso property
The JPEG module goes to version 1.5.2
Fixes openpreserve#297
@tledoux tledoux requested a review from carlwilson November 5, 2019 19:08
@codecov
Copy link

codecov bot commented Dec 10, 2019

Codecov Report

Merging #518 into integration will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             integration     #518   +/-   ##
==============================================
  Coverage          49.44%   49.44%           
  Complexity           984      984           
==============================================
  Files                 56       56           
  Lines               7768     7768           
  Branches            1409     1409           
==============================================
  Hits                3841     3841           
  Misses              3457     3457           
  Partials             470      470

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 88e9211...005509c. Read the comment docs.

@carlwilson carlwilson merged commit adb8eea into openpreserve:integration Dec 10, 2019
@tledoux tledoux deleted the issue297 branch December 11, 2019 08:24
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.

Possible issue with ColorSpace
2 participants