Opened 13 years ago

Closed 13 years ago

#754 closed enhancement (fixed)

[raster] Band index in rt_core functions must be 0-based

Reported by: jorgearevalo Owned by: jorgearevalo
Priority: low Milestone: PostGIS 2.0.0
Component: raster Version: master
Keywords: index Cc:

Description

Most of the functions that receive as argument a band index in rt_core expect 0-based index. But function "rt_raster_dump_as_wktpolygons" expects a 1-based index, and MapAlgebra (under construction) too. All the core functions should follow the same principles. The easier way is to ensure all the functions follow the 0-based scheme

Change History (4)

comment:1 by jorgearevalo, 13 years ago

Status: newassigned

comment:2 by Bborie Park, 13 years ago

rt_raster_has_no_band() also is using a 1-based index. There are no mapalgebra functions in rt_core so that isn't a problem.

comment:3 by Bborie Park, 13 years ago

Summary: [raster] Standardize the index nomenclature for rt_core functions[raster] Band index in rt_core functions must be 0-based

Standardized 0-based index for band number to rt_raster_dump_as_wktpolygons() in r7997.

comment:4 by Bborie Park, 13 years ago

Resolution: fixed
Status: assignedclosed

Changed rt_raster_has_no_band() to be 0-based in r7998. There are no more functions within rt_core/* that do not use 0-based band indices.

Note: See TracTickets for help on using tickets.