Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3174 closed defect (fixed)

Gdal envi driver can no longer read complex dataset

Reported by: jmichel Owned by: warmerdam
Priority: normal Milestone: 1.7.0
Component: GDAL_Raster Version: 1.6.2
Severity: critical Keywords: envi, complex
Cc:

Description (last modified by warmerdam)

When trying to read complex data with envi header using gdal 1.6.2, the following error occurs:

ERROR 1: The file does not have a value for the data type
that is recognised by the GDAL ENVI driver. 

This come from the following commented lines (frmts/raw/envidataset.cpp:1791):

/*  Removed JBP, ITT-VIS 11/29/2007.  Complex data does not display
properly in the ArcGIS application so we are removing it from the
supported types list. This is a bit of a big hammer as it may remove
functionality from some direct GDAL users, but this data is extremely
problematic from an ArcGIS standpoint.   */
/*          case 6:
            eType = GDT_CFloat32;
            break;

            case 9:
            eType = GDT_CFloat64;
            break;
*/

No wonder why I can not use my dataset anymore. I must say that I'm very surprised that every gdal users looses a valuable (SAR users will agree for sure) supported dataset types just to accommodate one software which is not able to display complex data.

Attachments (2)

complex32Bits (234.4 KB) - added by jmichel 12 years ago.
The complex 32 bits raw file
complex32Bits.hdr (197 bytes) - added by jmichel 12 years ago.
The complex 32 bits envi header

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by warmerdam

Component: defaultGDAL_Raster
Description: modified (diff)
Keywords: driver removed
Milestone: 1.6.31.7.0
Priority: highestnormal
Status: newassigned

Julien,

Do you have any, hopefully modestly sized and public examples of this data we could use for testing?

Changed 12 years ago by jmichel

Attachment: complex32Bits added

The complex 32 bits raw file

Changed 12 years ago by jmichel

Attachment: complex32Bits.hdr added

The complex 32 bits envi header

comment:2 Changed 12 years ago by jmichel

Warmerdam,

Please find attached the requested file.

Regards,

Julien

comment:3 Changed 12 years ago by warmerdam

Resolution: fixed
Status: assignedclosed

I have re-enabled complex support in trunk (r18557) and 1.6 branch (r18558). The test file seems to only have values with an imaginary component.

Sorry for taking so long on this.

comment:4 Changed 12 years ago by warmerdam

Patch ported into 1.6-esri tree (r18587), according to Peng Gao ArcGIS 10 handled complex pixel types fine.

Note: See TracTickets for help on using tickets.