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

limited tif support #322

Closed
gcode-importer opened this issue Mar 31, 2014 · 6 comments
Closed

limited tif support #322

gcode-importer opened this issue Mar 31, 2014 · 6 comments

Comments

@gcode-importer
Copy link

Originally reported on Google Code with ID 322


TIF support is currently provided by libtiff but this has some limitations, in particular
when dealing with non-8 or non-16 bits j2k images.

For instance, file6.jp2 from conformance test set is 12 bits per component, but OpenJPEG
produces a decoded tif file with 16 bits per component. This is wrong : the tiff file
should be 12 bits too.

TIF support should be improved/extended to support this kind of cases.


Reported by detonin on 2014-03-31 13:05:19

@gcode-importer
Copy link
Author

To rephrase the comment, openjpeg/tiff support is limited. libtiff does handle 12bits
quite nicely.

Reported by malaterre on 2014-04-01 08:27:09

  • Labels added: Conformance

@gcode-importer
Copy link
Author

Well, it's not what I have been told but that would be a rather good news, meaning that
we do not have to implement inhouse tiff support.

Do you have any idea of the workload to modify imagetotif() function so as to correctly
use the TIF library and output 12-bits image when it's needed ?

Reported by detonin on 2014-04-01 14:44:07

@gcode-importer
Copy link
Author

Well, it's not what I have been told but that would be a rather good news, meaning that
we do not have to implement inhouse tiff support.

Do you have any idea of the workload to modify imagetotif() function so as to correctly
use the TIF library and output 12-bits image when it's needed ?

Reported by detonin on 2014-04-01 14:45:10

@gcode-importer
Copy link
Author

Reported by detonin on 2014-04-01 14:45:31

@gcode-importer
Copy link
Author

@antonin,

I tested what I found in https://code.google.com/p/openjpeg/source/browse/tags/version.1.4/codec/convert.c
& it seemed to work.

However, 12bits TIFF handling disappeared in r745

This is somewhat related to issue 264.
I guess it would be good to get a status on this one before doing any work here.

Reported by mayeut on 2014-10-31 23:24:52

mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 14, 2015
mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 15, 2015
mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 16, 2015
mayeut added a commit to mayeut/openjpeg-data that referenced this issue Jul 16, 2015
mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 17, 2015
mayeut added a commit to mayeut/openjpeg-data that referenced this issue Jul 17, 2015
mayeut added a commit to mayeut/openjpeg that referenced this issue Jul 17, 2015
mayeut added a commit to mayeut/openjpeg-data that referenced this issue Jul 17, 2015
@mayeut
Copy link
Collaborator

mayeut commented Jul 17, 2015

Closed by PR #535

@mayeut mayeut closed this as completed Jul 17, 2015
@mayeut mayeut added this to the OPJ v2.1.1 milestone Aug 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants