Opened 13 years ago
Closed 13 years ago
#3383 closed defect (fixed)
Some CADRG crossing 180 have wrong extent.
Reported by: | gaopeng | Owned by: | warmerdam |
---|---|---|---|
Priority: | normal | Milestone: | 1.7.1 |
Component: | GDAL_Raster | Version: | 1.6.1 |
Severity: | normal | Keywords: | NITF CADRG |
Cc: |
Description
Some CADRG crossing 180 have wrong extent. Here are the good and bad extents for attached image:
GDAL: 174.375, 41.538, 517.5, 51.923 It should be: 174.375, 41.538, 191.25, 51.923
Attachments (2)
Change History (5)
by , 13 years ago
Attachment: | 0000M033.GN3 added |
---|
comment:1 by , 13 years ago
by , 13 years ago
Attachment: | nitf_fix3383.patch added |
---|
comment:2 by , 13 years ago
Keywords: | NITF CADRG added |
---|---|
Milestone: | 1.6.4 → 1.7.1 |
Status: | new → assigned |
With 1.6-esri, 1.7 and trunk I was seeing this for the geotransform:
Origin = (174.375000000000000,51.923076923076927) Pixel Size = (-0.223388671875000,-0.006760817307692) Corner Coordinates: Upper Left ( 174.3750000, 51.9230769) (174d22'30.00"E, 51d55'23.08"N) Lower Left ( 174.3750000, 41.5384615) (174d22'30.00"E, 41d32'18.46"N) Upper Right (-168.7500000, 51.9230769) (168d45'0.00"W, 51d55'23.08"N) Lower Right (-168.7500000, 41.5384615) (168d45'0.00"W, 41d32'18.46"N)
Which does not give the right side bound of 517.5 reported above. I'm not sure how you got that. But this is also wrong, with a negative x pixel size due to the dateline confusion. Even's patch seems to do the reasonable thing and I have applied it in trunk (r18740), 1.7 (r18741) and 1.6-esri (r18742).
The trunk change also includes an autotest with this file.
Gao, could you confirm that this commit fixes things for you, and close the ticket?
comment:3 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Yes. It fixed my problems.
Frank,
I also got into this issue some time ago with other similar CADRG products. I had done a broader fix in the past, but I had to make more narrow due to #2135. The proposed patch basically comes back to my original fix, but restrict it only to CADRG products. So hopefully it shouldn't cause regressions in other NITF profiles that can have non north-up images, and not to CADRG products themselves as, to the best of my knowledge, they have not fancy orientations.