Opened 14 years ago

Closed 12 years ago

#826 closed enhancement (fixed)

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

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

Description (last modified by dustymugs)

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

Change History (15)

comment:1 by pracine, 14 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, 14 years ago

Milestone: PostGIS 2.0.0 → PostGIS Raster Future

comment:3 by dustymugs, 13 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, 13 years ago

Milestone: PostGIS Raster Future → PostGIS Future

comment:5 by dustymugs, 12 years ago

Milestone: PostGIS Future → PostGIS 2.1.0
Owner: changed from pracine to dustymugs
Priority: low → high
Status: new → assigned

comment:6 by dustymugs, 12 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 dustymugs, 12 years ago

Description: modified (diff)

comment:8 by pracine, 12 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 dustymugs, 12 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, 12 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 dustymugs, 12 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 dustymugs, 12 years ago

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

comment:13 by dustymugs, 12 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 dustymugs, 12 years ago

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

comment:15 by dustymugs, 12 years ago

Keywords: history added
Resolution: → fixed
Status: assigned → closed

Added in r10829.

Note: See TracTickets for help on using tickets.