Ticket #754 (closed enhancement: fixed)

Opened 2 years ago

Last modified 19 months ago

[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: trunk
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

Changed 2 years ago by jorgearevalo

  • status changed from new to assigned

Changed 19 months ago by dustymugs

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.

Changed 19 months ago by dustymugs

  • summary changed from [raster] Standardize the index nomenclature for rt_core functions to [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.

Changed 19 months ago by dustymugs

  • status changed from assigned to closed
  • resolution set to fixed

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.