Opened 10 years ago

Closed 13 months ago

#3401 closed enhancement (wontfix)

When using MrSID for JP2K, JP2 metadata files are ignored

Reported by: gaige Owned by: warmerdam
Priority: normal Milestone: closed_because_of_github_migration
Component: GDAL_Raster Version: svn-trunk
Severity: normal Keywords: jp2mrsid
Cc:

Description

Adding in the use of the jp2metadata code solves this problem.

frmts/mrsid/mrsiddataset.cpp:

39a40
> 
66a68
> #  include "gdaljp2metadata.h"
1139c1141
< 
---
> 	
1502a1505,1521
> #ifdef MRSID_J2K
> 	if (bIsJP2 && (iNumRecs==0)) {
>         GDALJP2Metadata oJP2Geo;
> 
>         if( oJP2Geo.ReadAndParse( poOpenInfo->pszFilename ) )
>         {
>             poDS->pszProjection = CPLStrdup(oJP2Geo.pszProjection);
>             poDS->bHasGeoTransform = oJP2Geo.bHaveGeoTransform;
>             memcpy( poDS->adfGeoTransform, oJP2Geo.adfGeoTransform, 
>                     sizeof(double) * 6 );
>         }
> 	}
> #endif

Change History (4)

comment:1 Changed 10 years ago by warmerdam

Keywords: jp2mrsid added
Milestone: 1.7.2

Are you suggesting that the MrSID SDK does not real GeoJP2 metadata boxes? GMLJP2 metadata boxes? It was my intention that we would let the mrsid sdk handle the metadata processing in this driver in order to remain true to the intent of the mrsid sdk. It had been my experience that there was pretty good support for geojp2 and gmljp2 metadata boxes in the mrsid sdk.

comment:2 Changed 10 years ago by gaige

I'll back out hte patch here locally and rerun my tests using the data that I have in my test set. My experience was that, for some subset of files, I was not getting georef'd data from MrSID JP2K, but was getting it for other JP2K implementations. However, I was forced to use JP2K because I had found some files which were crashing in other locations. I've filed bug reports with the upstreams on those, but haven't gotten any reply.

-Gaige

comment:3 Changed 5 years ago by Jukka Rahkonen

I believe that this ticket can be closed as invalid if we can prove that JP2MrSID driver can read GeoTIFF metadata from UUID box and GMLJP2 metadata from its own box.

comment:4 Changed 13 months ago by Even Rouault

Milestone: closed_because_of_github_migration
Resolution: wontfix
Status: newclosed

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub?. If you believe this ticket is still valid, you may file it to https://github.com/OSGeo/gdal/issues if it is not already reported there.

Note: See TracTickets for help on using tickets.