Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#5104 closed enhancement (fixed)

Iris driver does not add height information

Reported by: rveciana Owned by: warmerdam
Priority: low Milestone: 1.11.0
Component: default Version: 1.10.0
Severity: normal Keywords: IRIS
Cc:

Description

It would be good to have some metadata in each band indicating the height of the data represented. This only affects the CAPPI product, since is the only one where height changes in each layer.

I have changed the driver file so it adds the "height" metadata in each layer when the product is CAPPI.

The band creation is moved after the metadata creation to detect the product type before creating the band.

Attachments (1)

irisdataset.cpp (41.7 KB) - added by rveciana 4 years ago.
The irisdataset.cpp driver file

Download all attachments as: .zip

Change History (5)

Changed 4 years ago by rveciana

Attachment: irisdataset.cpp added

The irisdataset.cpp driver file

comment:1 Changed 4 years ago by Even Rouault

Milestone: 1.10.12.0
Resolution: fixed
Status: newclosed

trunk r26065 "IRIS: add height information on bands; rename dataset metadata item CAPPI_HEIGHT --> CAPPI_BOTTOM_HEIGHT (#5104)"

I just pushed it in trunk. Changing the name of a metadata item in a stable branch is normally not appropriate, but as it is a new and likely niche driver, I could also push it in 1.10 if you insist.

comment:2 Changed 4 years ago by rveciana

Actually is not a change in a name, but adding a new label to the metadata. The behaviour of the existing features does not change. Anyway, if it's better to wait to the new version, just do what you consider is the best, please.

comment:3 in reply to:  2 Changed 4 years ago by rveciana

Replying to rveciana:

Actually is not a change in a name, but adding a new label to the metadata. The behaviour of the existing features does not change. Anyway, if it's better to wait to the new version, just do what you consider is the best, please.

-->Actualization: It's true, there is one label that changes the name, to avoid confusion. So maybe it's better to wait to the 2.0 version.

comment:4 Changed 4 years ago by Even Rouault

Milestone: 2.01.11.0
Note: See TracTickets for help on using tickets.