fileout_gdal seems to build broken JP2k files

Description

Lookng at the code in FONgTransform, it may be that it should be modified to follow the pattern used for the GeoTIFF code (instead of writing the info to a MEM object and then using CreateCopy to make the JP2k file, use the recipe that is used for GeoTIFF).

Also:

  • There are two copies of the return_temp_stream() method - one for the GeoTIFF code and one for the JP2k code. This should probably be turned into a utility function in the BES.

  • The tests for the JP2k output are pretty weak

Environment

None

Activity

Show:
James Gallagher
June 15, 2017, 8:56 PM

Worth noting, on the GDAL mailing list 15 June 2107

[gdal-dev] [FIXED] Re: OpenJPEG lossless encode broken

On vendredi 17 février 2017 14:02:37 CEST Aaron Boxer wrote:

> See https://github.com/uclouvain/openjpeg/issues/892 for
further details.

> Since problems occur unpredictably for various image
dimensions and number

> of encoder decompositions, safest approach is to avoid using
the codec for

> lossless until this issue is resolved.

FYI, this issue has been fixed in OpenJPEG master, as well as with
speed improvements. A new release will come in the following
weeks.

For more details, see

https://groups.google.com/forum/#!topic/openjpeg/2lgzhnoRTYA

Even

Done

Assignee

Slav Korolev

Reporter

James Gallagher

Labels

None

Time tracking

0m

Time remaining

16h

Components

Priority

Low