Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#5386 closed enhancement (fixed)

JP2KAK support for NITF driver

Reported by: bugbuster Owned by: warmerdam
Priority: normal Milestone: 1.11.0
Component: GDAL_Raster Version: 1.9.2
Severity: normal Keywords: NITF JPEG2000 Kakadu
Cc:

Description (last modified by bugbuster)

GDAL does not completely support NITF "Jpeg2000 compression format" based on Kakadu (JP2KAK driver). As it is with GDAL 1.9.2, it is possible to use "IC=C8" compression flag and use JP2KAK driver. However, there is no way to give any option. In particular, tiling is necesary when dealing with large raster images (e.g. 40000 X 40000).

I added JP2KAK into nitfdataset.cpp file , taking "ECW support" as an example : 1) New function NITFKAKOptions() : option filtering to pass specific options to JP2KAK driver 2) In NITFCreateCopy() : added a test for JP2KAK when initializing poJ2KDriver

Would it be possible to add it to a future version ? For us, it's particularly interesting since : 1) we only have JP2KAK driver to deal with JPEG2000 raster 2) we are dealing large raster images

Thanks for all the great job you're doing !

Attachments (3)

patch_ticket_5386.7z (772 bytes ) - added by bugbuster 10 years ago.
Proposed patch to nitfdataset.cpp
nitfdataset.cpp.patch (4.0 KB ) - added by bugbuster 10 years ago.
Proposed patch for source code
frmt_nitf.html.patch (1.6 KB ) - added by bugbuster 10 years ago.
Proposed patch for HTML documentation

Download all attachments as: .zip

Change History (10)

by bugbuster, 10 years ago

Attachment: patch_ticket_5386.7z added

Proposed patch to nitfdataset.cpp

comment:1 by bugbuster, 10 years ago

Description: modified (diff)

comment:2 by Even Rouault, 10 years ago

Could you also update frmt_nitf.html to document that new capability ? And attaching as a .patch directly in Trac makes it easier to review the patch.

in reply to:  2 comment:3 by bugbuster, 10 years ago

Replying to rouault: I just added the Create() case. Attached are the proposed patchs for both source code and html documentation. Let me know if it's OK. Sincerly Philippe

comment:4 by Even Rouault, 10 years ago

Well, did you test your patch... ? The JP2KAK driver doesn't support Create(), just CreateCopy().

by bugbuster, 10 years ago

Attachment: nitfdataset.cpp.patch added

Proposed patch for source code

by bugbuster, 10 years ago

Attachment: frmt_nitf.html.patch added

Proposed patch for HTML documentation

in reply to:  4 comment:5 by bugbuster, 10 years ago

Shame on me ! My test program was only testing CreateCopy() method. I modified the proposed patch (and html file) accordingly.

comment:6 by Even Rouault, 10 years ago

Milestone: 2.0
Resolution: fixed
Status: newclosed

trunk r26980 "NITF: allow JP2KAK to be used as the JPEG2000 compression engine in the CreateCopy() case (patch by bugbuster, #5386)"

comment:7 by Even Rouault, 10 years ago

Milestone: 2.01.11.0
Note: See TracTickets for help on using tickets.