Opened 4 years ago

Closed 4 years ago

Last modified 3 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 4 years ago.
Proposed patch to nitfdataset.cpp
nitfdataset.cpp.patch (4.0 KB) - added by bugbuster 4 years ago.
Proposed patch for source code
frmt_nitf.html.patch (1.6 KB) - added by bugbuster 4 years ago.
Proposed patch for HTML documentation

Download all attachments as: .zip

Change History (10)

Changed 4 years ago by bugbuster

Attachment: patch_ticket_5386.7z added

Proposed patch to nitfdataset.cpp

comment:1 Changed 4 years ago by bugbuster

Description: modified (diff)

comment:2 Changed 4 years ago by Even Rouault

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.

comment:3 in reply to:  2 Changed 4 years ago by bugbuster

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 Changed 4 years ago by Even Rouault

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

Changed 4 years ago by bugbuster

Attachment: nitfdataset.cpp.patch added

Proposed patch for source code

Changed 4 years ago by bugbuster

Attachment: frmt_nitf.html.patch added

Proposed patch for HTML documentation

comment:5 in reply to:  4 Changed 4 years ago by bugbuster

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

comment:6 Changed 4 years ago by Even Rouault

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 Changed 3 years ago by Even Rouault

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