Open GoogleCodeExporter opened 9 years ago
@bo,
All images are not decoded properly using the latest trunk (r2949).
image 1.jp2 appears as fully black. The other are "blurred", on a black
background which isn't right.
"Blurred" issue : Number of tile part specified is not right (same issue as
issue 254 or issue 423).
cmap box doesn't appear in the jp2 header box & is not read at all by OpenJPEG.
cmap box is "invalid" (not according to standard but according to what it is
intended to show) :
Red channel shall use palette mapping using palette column 0 : OK
Green channel shall use direct mapping : KO, should be palette mapping using palette column 1
Blue channel shall use direct mapping : KO, should be palette mapping using palette column 2
With all of the above manually corrected, images are properly decoded.
I will let someone else (Antonin ?) have a look at this because I might be
missing something regarding ISO-15444 (which ever part/amendment/...).
Another thing is the values for precedence & approx in the colr box which
aren't what's expected (but ignored by OpenJPEG)
Original comment by m.darb...@gmail.com
on 4 Dec 2014 at 8:48
Attachments:
@darbois, thank you so much for the quick debugging.
Original comment by bo...@foxitsoftware.com
on 4 Dec 2014 at 9:11
Original comment by m.darb...@gmail.com
on 5 Dec 2014 at 11:25
This is the exact same issue as issue 235.
I'm not marking this one as duplicate just yet.
Original comment by m.darb...@gmail.com
on 6 Dec 2014 at 10:58
Here's a patch to get those images to decode properly (with patch from issue
254 applied as well).
Not sure we wan't this in the trunk like this but it gives the idea of what's
going on.
Solves the issue 235 as well.
Original comment by m.darb...@gmail.com
on 6 Dec 2014 at 11:08
Attachments:
Forgot to mention that it's been tested against the test suite with no
regressions. Nevertheless, it shall be reviewed.
Original comment by m.darb...@gmail.com
on 6 Dec 2014 at 11:09
'xpdf version 3.03' does show all images clear and sharp.
winfried
Original comment by szukw...@arcor.de
on 8 Dec 2014 at 4:22
Is the patch in trunk now? Please let me know when I can update the code,
thanks!
Original comment by bo...@foxitsoftware.com
on 20 Dec 2014 at 12:54
Original comment by mathieu.malaterre
on 7 Jan 2015 at 4:43
@matthieu could you clarify this bug please (eg. the title). What remains to be
done ? Only patch from comment #5 is needed ?
Original comment by mathieu.malaterre
on 7 Jan 2015 at 5:44
@mathieu,
Those files are not valid jp2 files for multiple reasons (3 issues, see comment
#2), not sure how we could modify the title in this case.
The patch from comment #5 does not correct the issue 254 which comes with its
own patch.
Original comment by m.darb...@gmail.com
on 7 Jan 2015 at 11:15
Original issue reported on code.google.com by
bo...@foxitsoftware.com
on 4 Dec 2014 at 6:20Attachments: