ryanqq / openjpeg

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

Wrong encoding of small tiles with high level number #239

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Take a large image (say 3840x2160) and define the tile size a bit smaller to 
make one large tile and three smaller (say -t 3810,2130) and opj_compress with 
-n 6 or higher.
2. When DWT division comes to input width=1 (OPJ_INT32 rw in dwt.c, line 390) 
or  the output for the tile is visibly incorrect.
3. There is a similar problem when tile height comes to 1 and then there is 
another DWT level so it repeats the computation with height 1 (it should break 
in that case). 

What is the expected output? What do you see instead?
The output j2k image is visibly malformed on the small tiles. 

What version of the product are you using? On what operating system?
r2345, Ubuntu Linux, 3.2.0-48-generic, x86_64

Please provide any additional information below.

Original issue reported on code.google.com by The.Bo...@gmail.com on 10 Sep 2013 at 8:00

GoogleCodeExporter commented 9 years ago

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

GoogleCodeExporter commented 9 years ago
Seems like I can reproduce it over here:

$ ./bin/opj_compress -i ../../data/input/nonregression/ElephantDream_4K.tif -o 
issue239.jp2 -n 7 -t 4046,2130

[INFO] tile number 1 / 4
[INFO] tile number 2 / 4
WARNING: No incltree created.
WARNING: No imsbtree created.
WARNING: No incltree created.
WARNING: No imsbtree created.
WARNING: No incltree created.
WARNING: No imsbtree created.
[INFO] tile number 3 / 4
[INFO] tile number 4 / 4
WARNING: No incltree created.
WARNING: No imsbtree created.
WARNING: No incltree created.
WARNING: No imsbtree created.
WARNING: No incltree created.
WARNING: No imsbtree created.
Generated outfile issue239.jp2

Original comment by mathieu.malaterre on 4 Mar 2014 at 10:07

GoogleCodeExporter commented 9 years ago

Original comment by mathieu.malaterre on 12 Mar 2014 at 10:36