Opened 13 years ago

Closed 11 years ago

#826 closed enhancement (fixed)

[raster] By default, raster2pgsql should not pad tiles with NODATA

Reported by: pracine Owned by: Bborie Park
Priority: high Milestone: PostGIS 2.1.0
Component: raster Version: master
Keywords: history Cc:

Description (last modified by Bborie Park)

This would be as part of the changes described in #2122.

Change History (15)

comment:1 by pracine, 13 years ago

-Another interesting option would allow not loading tiles containing only nodata -Another interesting option would trim complete row or columns of nodata values at the borders of loading tiles

comment:2 by pracine, 13 years ago

Milestone: PostGIS 2.0.0PostGIS Raster Future

comment:3 by Bborie Park, 12 years ago

Summary: [raster] There should be an option in raster2pgsql.py to prevent completing tiles with nodata[raster] There should be an option in raster2pgsql to prevent completing tiles with nodata

comment:4 by pracine, 12 years ago

Milestone: PostGIS Raster FuturePostGIS Future

comment:5 by Bborie Park, 11 years ago

Milestone: PostGIS FuturePostGIS 2.1.0
Owner: changed from pracine to Bborie Park
Priority: lowhigh
Status: newassigned

comment:6 by Bborie Park, 11 years ago

Summary: [raster] There should be an option in raster2pgsql to prevent completing tiles with nodata[raster] raster2pgsql should not pad tiles with NODATA

comment:7 by Bborie Park, 11 years ago

Description: modified (diff)

comment:8 by pracine, 11 years ago

So I guess the default behaviour now will be to NOT pad. Do we need an option to pad? I would say yes…

comment:9 by Bborie Park, 11 years ago

But what reason is there to pad? Especially if the specs indicate that the width/height of the right and bottom edge tiles can be smaller?

comment:10 by pracine, 11 years ago

The code for padding is there already. It would be a waste to just throw it away. Some people might not mind about keeping the former raster extent and want a nice and clean raster coverage with all the tiles the same size. It make client read more simple. Mat wanted simplicity AND keeping the original extent. You can't have both. And "The reality is that none of us will ever know how PostGIS raster will be used" :-)

comment:11 by Bborie Park, 11 years ago

Actually, no. There was no code FOR padding tiles. I had to add code to my git repo to get no padding.

comment:12 by Bborie Park, 11 years ago

As a followup, there IS code in ST_Tile though that made sure to support padding when generating tiles.

comment:13 by Bborie Park, 11 years ago

Gah. I'm tired of this. I'll add a -P flag indicating that tiles should be padded if needed.

comment:14 by Bborie Park, 11 years ago

Summary: [raster] raster2pgsql should not pad tiles with NODATA[raster] By default, raster2pgsql should not pad tiles with NODATA

comment:15 by Bborie Park, 11 years ago

Keywords: history added
Resolution: fixed
Status: assignedclosed

Added in r10829.

Note: See TracTickets for help on using tickets.