troosh / openjpeg

Automatically exported from code.google.com/p/openjpeg
Other
0 stars 0 forks source link

misleading info in JP2 box lead to wrong number of components #110

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The decoding of jpxstream.data : as indicated by Winfried, 1.3 and 1.4 give 
different results (respectively 1 and 3 components). In this case, 1.3 is 
right, the info in the jp2 boxes is misleading 1.4 (and was not taken into 
account in 1.3, which explains the regression. Kakadu gives also only one 
component.

Original issue reported on code.google.com by anto...@gmail.com on 23 Jan 2012 at 8:22

Attachments:

GoogleCodeExporter commented 9 years ago
(see issue #104 for comments history)

Original comment by anto...@gmail.com on 23 Jan 2012 at 8:23

GoogleCodeExporter commented 9 years ago
This must have been fixed in 1.5. So I would be tempted to simply closing this 
issue.

Original comment by mathieu.malaterre on 29 May 2012 at 1:24

GoogleCodeExporter commented 9 years ago

Original comment by mathieu.malaterre on 25 Feb 2014 at 3:57

GoogleCodeExporter commented 9 years ago
I must say I do not understand how kakadu decide not to apply PCLR. Does it 
check for the invalid value of PREC (I.5.3.3) ?

Original comment by mathieu.malaterre on 27 Feb 2014 at 2:04

GoogleCodeExporter commented 9 years ago
This issue was updated by revision r2501.

Original comment by mathieu.malaterre on 27 Feb 2014 at 2:17

GoogleCodeExporter commented 9 years ago
opj_jp2_apply_pclr is properly applied on file9.jp2 as should.

Original comment by mathieu.malaterre on 27 Feb 2014 at 2:18

GoogleCodeExporter commented 9 years ago
antonin, I believe applying PCRL is correct. I have been using file9.jp2 
(conformance) and kakadu seems to have a similar behavior : if output target is 
single channel (eg. pgm) then a single channel is extracted, if output target 
is three channel then PCLR is applied (eg. ppm).

Could you please refine why you think behavior in 1.3 was correct ?

Original comment by mathieu.malaterre on 28 Feb 2014 at 10:34

GoogleCodeExporter commented 9 years ago
antonin, I am closing this one for now. If you believe I misunderstood 
something please update the description. thx

Original comment by mathieu.malaterre on 3 Mar 2014 at 4:16

GoogleCodeExporter commented 9 years ago

Original comment by anto...@gmail.com on 24 Mar 2014 at 3:57