Opened 16 years ago

Closed 15 years ago

#1567 closed enhancement (fixed)

Patch for HFA no-data value support

Reported by: bclaywell Owned by: Mateusz Łoskot
Priority: high Milestone: 1.5.0
Component: GDAL_Raster Version: 1.4.0
Severity: normal Keywords: HFA
Cc: warmerdam, Markus Neteler


Enclosed is a patch against the 1.4.0 HFA driver source implementing no-data value support. Only partial BASEDATA support in HFAField::SetInstValue() has been implemented to support this, due to time crunch issues on my end; specifically, only 1x1 BASEDATA arrays of type double are handled. Setting and retrieving the no-data value has been tested and verified in at least simple cases to work correctly; the HFA statistics generation routines correctly honor the set no-data value; and gdalinfo correctly reports the set no-data value.

Attachments (1)

hfa-no-data-20070409.patch (7.4 KB ) - added by bclaywell 16 years ago.

Download all attachments as: .zip

Change History (9)

by bclaywell, 16 years ago

Attachment: hfa-no-data-20070409.patch added

comment:1 by bclaywell, 16 years ago

Milestone: 1.5.0
Version: 1.4.0

comment:2 by bclaywell, 16 years ago

Version: 1.4.0

comment:3 by Mateusz Łoskot, 16 years ago

Cc: Mateusz Łoskot added

comment:4 by warmerdam, 16 years ago

Cc: warmerdam added; Mateusz Łoskot removed
Owner: changed from warmerdam to Mateusz Łoskot


Could you incorporate this patch in trunk when time permits.

Be quite careful with the basedata portion of the patch (in hfafield.cpp) as I believe there has been other work in this area already recently.

I'm too chicken to incorporate this feature improvement in branches/1.4.

The HFA script(s) should be extended to test the new nodata support.

comment:5 by Markus Neteler, 15 years ago

Cc: Markus Neteler added

comment:6 by warmerdam, 15 years ago

Priority: normalhigh

comment:7 by Mateusz Łoskot, 15 years ago

Status: newassigned

comment:8 by Mateusz Łoskot, 15 years ago

Resolution: fixed
Status: assignedclosed
Version: 1.4.0

Patch applied to trunk along with NoData read/write tests in (r13278)

Note: See TracTickets for help on using tickets.