Opened 12 years ago

Closed 10 years ago

#3682 closed enhancement (fixed)

ENVI metadata in machine parseable form

Reported by: verosk Owned by: warmerdam
Priority: normal Milestone: 1.10.0
Component: GDAL_Raster Version: unspecified
Severity: normal Keywords: gdal metadata envi
Cc:

Description

Dear GDAL people,

I'm trying to use GDAL/Python to read and process ENVI data produced by our airborne sensor. It already saved me lot of my time.

I realized that there is no easy(portable) way how to extract metadata of ENVI-saved data, e.g. wavelength, band width, names and other values. But these properties are sometimes crucial for processing of remote sensed imagery.

I suggest these metadata should be stored in Metadata structure of GDALObject in the similar way to current way of saving metadata ("Band_XX"), but in machine parseable format.

I'm able to prepare the patch for C++ code, but I'm not sure if there is a chance of accepting this type of patch. So I'm politely asking, may I prepare patch for GDAL, or should I solve this problem in separate way outside of the GDAL library?

Thank you for your responses.

--Veros

Attachments (1)

diff.txt (1.2 KB ) - added by verosk 12 years ago.
Patch to envidataset.cpp - reading wavelengths to metadata

Download all attachments as: .zip

Change History (8)

comment:1 by warmerdam, 12 years ago

Věroš,

I'm sure we would be interested in such a submission. Please be clear that such a code submission will be assumed to be under the MIT/X open source license used for the rest of GDAL. It would be helpful if you also provide a smallish data sample with the metadata. I also think it would be more appropriate to attach the metadata to the band's themselves if it is band specific rather than attaching it to the dataset with band specific names.

by verosk, 12 years ago

Attachment: diff.txt added

Patch to envidataset.cpp - reading wavelengths to metadata

comment:2 by verosk, 12 years ago

Attached patch to envidataset.cpp for reading wavelengths to bands object metadata.

Test data are available on http://truhla.lep.brno.cas.cz/~veros/gdal_3682_testdata.tar.gz

comment:3 by verosk, 12 years ago

Cc: chaitanya removed

I would like to politely ask if there is any progress in processing of this ticket. The patch is already attached for several weeks.

If there is some problem with the patch, I can prepare a new better version.

Thank you for your response.

--Veros

comment:4 by warmerdam, 12 years ago

Milestone: 1.8.0
Status: newassigned

Veros,

I will try to ensure the patch is reviewed and hopefully applied before the 1.8 release. Is there any particular reason you removed Chaitanya as a cc: on this ticket?

comment:5 by verosk, 12 years ago

Thank you for your reply. I wasn't sure whether the patch is lost.

Removal of chaitanya in cc: is my error. Sorry for that, I'm still not very familiar with Trac.

--Veros

comment:6 by robintw, 10 years ago

Has there been any progress on this? The last discussion seemed to be around two years ago, and it was suggested that it would be implemented by 1.8.0.

I am currently running 1.9.2, and the test provided by Veros doesn't seem to work.

Does this mean that this enhancement has not yet been implemented? If so, is it likely to be implemented fairly soon? Like Veros, I need to be able to get access to the Wavelengths list in the ENVI header as part of my processing of remotely-sensed imagery.

comment:7 by Even Rouault, 10 years ago

Milestone: 1.8.11.10.0
Resolution: fixed
Status: assignedclosed

trunk r25537 "ENVI: report wavelength and wavelength_units as band metadata (#3682)"

Note: See TracTickets for help on using tickets.