Opened 14 years ago
Closed 5 years 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 by , 14 years ago
Keywords: | jp2mrsid added |
---|---|
Milestone: | 1.7.2 |
comment:2 by , 14 years ago
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 by , 9 years ago
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 by , 5 years ago
Milestone: | → closed_because_of_github_migration |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
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.
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.