Changes between Initial Version and Version 1 of Ticket #605


Ignore:
Timestamp:
Apr 2, 2007, 9:36:59 PM (17 years ago)
Author:
warmerdam
Comment:

This needs to be reviewed and considered for 1.5.0. NITF is important to our sponsors.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #605

    • Property Priority high → normal
    • Property Status new → assigned
    • Property Milestone → 1.5.0

  • Ticket #605 – Description

    initial v1  
    1 {{{
     1
    22Frank,
    33
    4 I have just begun using GDAL to work with NITF images.  Nice package.  Thankyou.
    5  I needed a few new features and added them.  Attached is a patch file you can
    6 do with as you will.  The changes do this:
     4I have just begun using GDAL to work with NITF images.  Nice package.  Thankyou.   I needed a few new features and added them.  Attached is a patch file you can do with as you will.  The changes do this:
    75
    8 - Add NITF_ICORDS to the dataset's metadata.
    9 - Add all ICHIPB fields to the metadata.
    10 - Read RPC00A exactly as RPC00B is read.  (I've never been able to see any
    11 difference between the A and B versions.  The gov't's
    12 STDI-0002-version-2.1-16-Nov-2000.pdf document says they differ in some
    13 coefficient ordering, but comparing with STDI-0002-version-1.0-25-Aug-1998.pdf,
    14 I see no difference at all.)
     6 $ Add NITF_ICORDS to the dataset's metadata.
     7 $ Add all ICHIPB fields to the metadata.
     8 $ Read RPC00A exactly as RPC00B is read.  (I've never been able to see any difference between the A and B versions.  The gov't's
     9STDI-0002-version-2.1-16-Nov-2000.pdf document says they differ in some coefficient ordering, but comparing with  STDI-0002-version-1.0-25-Aug-1998.pdf, I see no difference at all.)
    1510
    16 I think you might want to move the ICORDS change to a different spot and change
    17 the comments.  There are also /*DRM:*/ comments you'll want to remove.  They
    18 mark every change.
     11I think you might want to move the ICORDS change to a different spot and change the comments.  There are also /*DRM:*/ comments you'll want to remove.  They mark every change.
    1912
    20 FYI, I needed NITF_ICORDS to interface with some annotation feature of IMAGINE
    21 with a NITF reader module.
     13FYI, I needed NITF_ICORDS to interface with some annotation feature of IMAGINE with a NITF reader module.
    2214
    2315Here are some possible bug/deficiencies I noticed while poking around:
    2416
    25 - The NITF_MIN/MAX_LAT/LONG metadata items look wrong.  The RPC region usually
    26 straddles LAT/LONG_OFFSET by LAT/LONG_SCALE.  However, the metadata items begin
    27 at LAT/LONG_OFFSET and extend from there by 2*LAT/LONG_OFFSET
    28 - The RPC projection code doesn't know about ICHIPB and so will incorrectly
    29 project points for a NITF file with ICHIPB.
    30 - The ERR_BIAS and ERR_RAND RPC00B fields aren't put in the dataset's metadata.
     17 * The NITF_MIN/MAX_LAT/LONG metadata items look wrong.  The RPC region usually straddles LAT/LONG_OFFSET by LAT/LONG_SCALE.  However, the metadata items begin at LAT/LONG_OFFSET and extend from there by 2*LAT/LONG_OFFSET
     18 * The RPC projection code doesn't know about ICHIPB and so will incorrectly project points for a NITF file with ICHIPB.
     19 * The ERR_BIAS and ERR_RAND RPC00B fields aren't put in the dataset's metadata.
    3120
    3221Thanks again for GDAL,
     22
    3323Doug
    34 }}}