geotrellis
geotrellis copied to clipboard
colormap.render is broken for some tile types
Tile requires a foreach method. For most tiles - ArrayTiles to be sure - a line scan from top left to bottom right is carried out. This is usually fine. ColorMap.render, however, assumes this behavior and can cause extremely hard to diagnose bugs as a result. For instance, the current PaddedTile implementation will cause tile data to be shifted up and to the left (as far as the tile boundary), pushing out any NoData values if ColorMap.render is called.
- Maybe there's no good argument for colormaps having a render method and we should remove it
- Perhaps the implementation of
foreachviolates expected semantics and should be corrected - Both 1 and 2?