Sean Gillies

Results 512 comments of Sean Gillies
trafficstars

I did some background reading and am more qualified to discuss this now. https://cran.r-project.org/web/packages/gdalraster/vignettes/raster-attribute-tables.html is a nice intro. From watching some QGIS videos it seems that map-makers appreciate raster attribute...

@groutr aren't structured arrays primarily for record or row-oriented data? A raster attribute table looks fundamentally column-oriented, and that's reflected in the GDAL model.

Thanks for the comments @ebkurtz, @groutr. We'll have to implement much of the GDAL RAT API just to read and write them, yes? I don't think it would be much...

@pwheimann thanks for the report! I'll modify the test at https://github.com/Toblerity/Fiona/blob/main/tests/test_datetime.py#L832 to try to reproduce this.

@jayrobwilliams Thanks for the report! I agree 100%. However, this would need to be implemented in the GDAL project. In this case, Rasterio is propagating an error message from GDAL....

@konstntokas I can reproduce this with 1.4b1. If I use `endpoint_url="dataspace.copernicus.eu"` then I do get a working request. I'm not sure where the bug is yet.

@konstntokas can you see if using `eodata.dataspace.copernicus.eu` as endpoint and setting `AWS_VIRTUAL_HOSTING=False` as in https://github.com/rasterio/rasterio/issues/1293#issuecomment-5026516 helps? If you have any ideas for how to make this experience better, I would...

@ryandrake08 thanks for the report! The 4 vertex requirement is inherited from GeoJSON: https://datatracker.ietf.org/doc/html/rfc7946#section-3.1.6. Like you point out, we could relax that requirement. I'd be more than happy to review...

@xinluo2018 can you explain what the problems are? Otherwise, I don't know if this work solves them.

Have you tried this out, @vincentsarago? Does it do what it advertises? @xinluo2018 I will try this out, but first I need to make a 1.4.2 release.